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

Add feature to hide existing credentials #617

Merged
merged 1 commit into from
Jul 6, 2024
Merged

Conversation

Spomky
Copy link
Contributor

@Spomky Spomky commented Jul 6, 2024

The code changes enable suppressing the existing user credential details to enhance security. This feature introduces a preventive measure against username enumeration exploits by concealing the previously existing credentials. The flag 'hide_existing_credentials' has been added to facilitate this change, defaulted to false. This change is particularly important during the attestation ceremony performed by anonymous users.

Target branch: 4.9.x
Resolves issue # none

  • It is a Bug fix
  • It is a New feature
  • Breaks BC
  • Includes Deprecations

@Spomky Spomky added bug Something isn't working security labels Jul 6, 2024
@Spomky Spomky added this to the 4.9.0 milestone Jul 6, 2024
@Spomky Spomky self-assigned this Jul 6, 2024
The code changes enable suppressing the existing user credential details to enhance security. This feature introduces a preventive measure against username enumeration exploits by concealing the previously existing credentials. The flag 'hide_existing_credentials' has been added to facilitate this change, defaulted to false. This change is particularly important during the attestation ceremony performed by anonymous users.
@Spomky Spomky force-pushed the bugs/username-enumeration branch from 66a7463 to 64de11f Compare July 6, 2024 04:54
@Spomky Spomky merged commit a9d1352 into 4.9.x Jul 6, 2024
15 checks passed
@Spomky Spomky deleted the bugs/username-enumeration branch July 6, 2024 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working security
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant