Implement equals and hashCode in ImmutablePublicKeyCredentialUserEntity by aspan · Pull Request #18883 · spring-projects/spring-security · GitHub
Skip to content

Implement equals and hashCode in ImmutablePublicKeyCredentialUserEntity#18883

Merged
jzheaux merged 1 commit into
spring-projects:mainfrom
aspan:gh-18882
Mar 23, 2026
Merged

Implement equals and hashCode in ImmutablePublicKeyCredentialUserEntity#18883
jzheaux merged 1 commit into
spring-projects:mainfrom
aspan:gh-18882

Conversation

@aspan

@aspan aspan commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Mar 12, 2026
@jzheaux jzheaux self-assigned this Mar 21, 2026
@jzheaux jzheaux changed the title Implement equals and hashCode in ImmutablePublicKeyCredentialUserEntity closes gh-18882 Implement equals and hashCode in ImmutablePublicKeyCredentialUserEntity Mar 21, 2026
@jzheaux jzheaux added type: enhancement A general enhancement in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) and removed status: waiting-for-triage An issue we've not yet triaged labels Mar 21, 2026
@jzheaux jzheaux added this to the 7.1.0-RC1 milestone Mar 21, 2026

@jzheaux jzheaux left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the PR, @aspan! I've left some feedback inline.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Will you please use this.name/that.name and this.id/that.id? The reason is so that this class doesn't get surprised with overridden implementations, should this class ever become non-final.

@jzheaux jzheaux added the status: waiting-for-feedback We need additional information before we can continue label Mar 21, 2026
Signed-off-by: Andreas Asplund <andreas@asplund.biz>
@aspan

aspan commented Mar 21, 2026

Copy link
Copy Markdown
Contributor Author

Hi Thanks for the feedback. I've made the requested changes.

@spring-projects-issues spring-projects-issues added status: feedback-provided Feedback has been provided and removed status: waiting-for-feedback We need additional information before we can continue labels Mar 21, 2026
@jzheaux jzheaux removed the status: feedback-provided Feedback has been provided label Mar 23, 2026
@jzheaux jzheaux merged commit 330c565 into spring-projects:main Mar 23, 2026
7 checks passed
@jzheaux

jzheaux commented Mar 23, 2026

Copy link
Copy Markdown
Collaborator

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

Labels

in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants