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

Paramater residentKey will be ignored when Initializing AuthenticatorSelectionCriteria #516

Closed
tomocrafter opened this issue Nov 15, 2023 · 4 comments · Fixed by #524
Closed
Assignees
Labels
bug Something isn't working
Milestone

Comments

@tomocrafter
Copy link

Version(s) affected

4.7.4

Description

Paramater residentKey will be ignored when Initializing AuthenticatorSelectionCriteria

How to reproduce

new AuthenticatorSelectionCriteria(null, "preferred", "required")->residentKey;

Possible Solution

This line should be $this->residentKey = $residentKey !== null ? $residentKey : ($requireResidentKey === true ? self::RESIDENT_KEY_REQUIREMENT_REQUIRED : self::RESIDENT_KEY_REQUIREMENT_PREFERRED);

$this->residentKey = $residentKey !== null ? $this->residentKey : ($requireResidentKey === true ? self::RESIDENT_KEY_REQUIREMENT_REQUIRED : self::RESIDENT_KEY_REQUIREMENT_PREFERRED);

Additional Context

No response

@Spomky
Copy link
Contributor

Spomky commented Nov 16, 2023

Hi,

Indeed, there is something wrong with this line.
I will address this issue ASAP

@Spomky Spomky self-assigned this Nov 16, 2023
@Spomky Spomky added the bug Something isn't working label Nov 16, 2023
@Spomky Spomky added this to the 4.7.5 milestone Nov 16, 2023
@tomocrafter
Copy link
Author

tomocrafter commented Nov 16, 2023

Hi, Thank you for response and commiting to this great library! It was hard to find solid library for webauthn in PHP so I am so pleasure to meet to this.

Thank you for looking into this problem as well, I think this can be fixed by my suggestion so Can I open the pull request with these changes?

@Spomky Spomky linked a pull request Nov 17, 2023 that will close this issue
4 tasks
@Spomky
Copy link
Contributor

Spomky commented Nov 17, 2023

Done. Will be tagged soon

@Spomky Spomky closed this as completed Nov 17, 2023
Copy link
Contributor

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants