Skip to content

Comments

Support authenticatorAttachment in PublicKeyCredential#250

Merged
emlun merged 4 commits intoYubico:mainfrom
luisgoncalves:feature/credential-authenticator-attachment
Dec 16, 2022
Merged

Support authenticatorAttachment in PublicKeyCredential#250
emlun merged 4 commits intoYubico:mainfrom
luisgoncalves:feature/credential-authenticator-attachment

Conversation

@luisgoncalves
Copy link
Contributor

@luisgoncalves luisgoncalves commented Dec 16, 2022

Fixes #249.

@emlun emlun self-requested a review December 16, 2022 12:10
@luisgoncalves
Copy link
Contributor Author

Failing at com.yubico.webauthn.data.JsonIoSpec > The class PublicKeyCredential allows and ignores an authenticatorAttachment attribute. I'll check.

@emlun
Copy link
Member

emlun commented Dec 16, 2022

Thanks but no need, I've already pushed a fix. 🙂

Copy link
Member

@emlun emlun left a comment

Choose a reason for hiding this comment

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

This looks really good, thanks!

Would you like to be credited in the release notes? And if so, by GitHub username or real name?

@luisgoncalves
Copy link
Contributor Author

Thanks for the help, and for the library.

Sure, why not :) use the GH username.

@emlun emlun merged commit 35c983d into Yubico:main Dec 16, 2022
@luisgoncalves luisgoncalves deleted the feature/credential-authenticator-attachment branch December 16, 2022 16:28
emlun added a commit that referenced this pull request Jan 11, 2023
New features:

- (Experimental) Added `authenticatorAttachment` property to response
  objects:
  - NOTE: Experimental features may receive breaking changes without a
    major version increase.
  - Added method `getAuthenticatorAttachment()` to
    `PublicKeyCredential` and corresponding builder method
    `authenticatorAttachment(AuthenticatorAttachment)`.
  - Added method `getAuthenticatorAttachment()` to
    `RegistrationResult` and `AssertionResult`, which echo
    `getAuthenticatorAttachment()` from the corresponding
    `PublicKeyCredential`.
  - Thanks to GitHub user luisgoncalves for the contribution, see
    #250

Other:

- Fixed the README description of SemVer exceptions: `@Deprecated`
  features are still part of the public API unless they also have an
  `EXPERIMENTAL:` tag in JavaDoc.
- Brought `com.yubico.webauthn` package JavaDoc up to date with new
  library features.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Support for PublicKeyCredential.authenticatorAttachment

2 participants