Skip to content

Add meonvr.com to disposable domains - #27

Closed
Rohan-Shinde24 wants to merge 1 commit into
utopia-php:mainfrom
Rohan-Shinde24:bugfix/add-meonvr-domain
Closed

Rohan-Shinde24 wants to merge 1 commit into
utopia-php:mainfrom
Rohan-Shinde24:bugfix/add-meonvr-domain

Conversation

@Rohan-Shinde24

Copy link
Copy Markdown

Summary

  • Add meonvr.com to the disposable email domain deny list.
  • Ensure email addresses from this domain are recognized as disposable.

Related Issue

Closes #26

Related to appwrite/appwrite#13740

Testing

  • The change is limited to data/disposable-domains.php.
  • Local Composer tests could not be run because PHP and Composer are not installed in the local environment.

@greptile-apps

greptile-apps Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 4/5

The PR should not merge until meonvr.com is recorded in the maintained source and the published list is regenerated, otherwise routine imports can undo the change.

Fix All in Claude CodeFindings

  1. P1 Classification Is Not Durable ▶
Fix with agent prompt
### Issue 1
data/disposable-domains.php:42505
This adds `meonvr.com` only to the generated runtime list. A standard disposable-domain import rebuilds this file from the manual and external sources, but the repository-controlled manual source does not contain this domain. If the external lists also omit it, the next import will silently remove the new disposable classification. Add the domain to the manual disposable-domain source and regenerate this file.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

This PR classifies meonvr.com as disposable by adding it to the published runtime domain list.

  • The classification is currently effective at runtime.
  • The entry was not added to the repository-controlled manual source, so a subsequent import can remove it.

Reviews (1) · Last reviewed commit: "Add meonvr.com to disposable domains"

'meomo.store',
'meong.store',
'meooovspjv.pl',
'meonvr.com',

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Classification Is Not Durable

This adds meonvr.com only to the generated runtime list. A standard disposable-domain import rebuilds this file from the manual and external sources, but the repository-controlled manual source does not contain this domain. If the external lists also omit it, the next import will silently remove the new disposable classification. Add the domain to the manual disposable-domain source and regenerate this file.

Knowledge Base Used: Domain classification data maintenance

Prompt To Fix With AI
This is a comment left during a code review.
Path: data/disposable-domains.php
Line: 42505

Comment:
**Classification Is Not Durable**

This adds `meonvr.com` only to the generated runtime list. A standard disposable-domain import rebuilds this file from the manual and external sources, but the repository-controlled manual source does not contain this domain. If the external lists also omit it, the next import will silently remove the new disposable classification. Add the domain to the manual disposable-domain source and regenerate this file.

**Knowledge Base Used:** [Domain classification data maintenance](https://app.greptile.com/appwrite/-/custom-context/knowledge-base/utopia-php/emails/-/docs/domain-classification-data.md)

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Fix in Claude Code Fix in Codex

@Rohan-Shinde24

Copy link
Copy Markdown
Author

Hi! I’m working on PR #27 for issue #26.
The automated review mentioned that meonvr.com should be added to the maintained source before the published list is regenerated.
Could you please let me know which file/source I should update and what regeneration process I should follow?
Thanks!

@ChiragAgg5k

Copy link
Copy Markdown
Member

Emails now lives in packages/emails in appwrite/appwrite (appwrite/appwrite#13929), and this repository is a read-only mirror. @Rohan-Shinde24, please re-open this there, adding the domain to data/disposable-domains-manual.php (disposable-domains.php is regenerated by import.php).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add meonvr.com to the disposable email domain deny list

2 participants