Skip to content

feat: add disclaimer to readme#2

Open
theseion wants to merge 3 commits into
mainfrom
add-warning
Open

feat: add disclaimer to readme#2
theseion wants to merge 3 commits into
mainfrom
add-warning

Conversation

@theseion
Copy link
Copy Markdown

No description provided.

@theseion theseion requested review from azurit and fzipi October 30, 2025 07:01
@azurit
Copy link
Copy Markdown
Member

azurit commented Oct 30, 2025

It sounds a bit too harsh, I’d say it more mildly. Also, i would recommend to apply rate limits on SMTP server side.

@azurit
Copy link
Copy Markdown
Member

azurit commented Oct 30, 2025

Can you complete also Testing section? I would just to run curl "http://localhost?test=/etc/passwd" and see if e-mail arrived. If not, see error logs. Thank you.

@theseion
Copy link
Copy Markdown
Author

I want the warning message to be clear. It's not harsh, IMO, just not unnecessarily friendly :).

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.

2 participants