Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UM-Block Disposable Email Domains #91

Open
MissVeronica opened this issue May 2, 2024 · 1 comment · May be fixed by #89
Open

UM-Block Disposable Email Domains #91

MissVeronica opened this issue May 2, 2024 · 1 comment · May be fixed by #89
Assignees

Comments

@MissVeronica
Copy link
Contributor

Remove the email blocked list with the UM options key blocked_disposable_emails from um_options.

UM()->options()->update( 'blocked_disposable_emails', implode( PHP_EOL, $arr_email_formatted ) );

The blocked disposable email domains return from "CDN API URL" is approx 2970 Kbytes and my current um_options is 79 Kbytes.
Activating the "Block Disposable Email Domains" plugin will make the um_options 39 times larger with longer load times for each UM page or 97,4% of the um_options is Blocked Email Domains only needed at the Registration page.

Move the blocked_disposable_emails to its own option_name in the options table
and cleanup by removing blocked_disposable_emails from um_options.

Another issues with this blocked_disposable_emails is that phpMyAdmin will fail ( too large entry ) if you need to update anything in um_options and UM will crash.

@champsupertramp champsupertramp self-assigned this May 2, 2024
@champsupertramp champsupertramp linked a pull request May 2, 2024 that will close this issue
@champsupertramp
Copy link
Member

Hi @MissVeronica Thanks for letting us know. I've created a new pre-release version with fixes to this issue:
https://github.com/ultimatemember/Extended/releases/tag/2.1.0-beta.16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants