Email Change Security: Missing Verification and Notification Controls #5585
TheCyberDesk
started this conversation in
General Community
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I am moving the below to the discussion board following feedback from a security submission.
Summary
Pterodactyl's email change functionality lacks two critical security controls that are industry standard across major platforms: email verification and old email notification. While password re-entry is required (which is good), the absence of these controls creates two significant security risks that I believe warrant discussion and consideration for future versions.
The Issues
Issue 1: No Email Verification to New Address
When a user changes their email, the new address is immediately activated without any verification that the user actually controls that email address.
Security Implications:
support@hostingcompany.comto appear as legitimate staff)Issue 2: No Notification to Old Address
When an email is changed, the original email address receives zero notification about the change. The user whose account has been modified has no way of knowing their email was changed unless they happen to log in and check.
Security Implications:
1. Industry Standards and Guidelines
NIST SP 800-63B (Digital Identity Guidelines), Section 6.1.2.3:
OWASP Authentication Cheat Sheet:
CWE-620: Unverified Password Change:
CWE-451: User Interface Misrepresentation of Critical Information:
3. Real-World Attack Scenarios
Scenario A: Credential Stuffing Campaign
Scenario B: Social Engineering via Email Impersonation
support@hostingcompany.comEvery major platform implements email verification and notification.
Proposed Solution
I believe Pterodactyl should implement the following controls to align with industry standards:
Minimum Required:
1. Email Verification
2. Old Email Notification
Recommended:
3. Cancellation Window
4. Rate Limiting
5. UI Indicators
Benefits of Implementation
Beta Was this translation helpful? Give feedback.
All reactions