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

Refactor RegistrationsControllerConcern.verify_passkey_registration_challenge to break out credential integrity verification into its own before_action #31

Open
tcannonfodder opened this issue Jun 24, 2023 · 6 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@tcannonfodder
Copy link
Contributor

In #29, we refactored PasskeysControllerConcern.verify_passkey_challenge to move the integrity checks into their own before action: f1400cb (discussion where the idea came from: #29 (comment))

For consistency, we should have a similar pattern for RegistrationsControllerConcern.verify_passkey_registration_challenge. This will also involve updating the tests themselves to reflect the changes. See:

@johalloran01 this might be a good issue to tackle as well, if you want to take it 💪

@tcannonfodder tcannonfodder added enhancement New feature or request good first issue Good for newcomers labels Jun 24, 2023
@tcannonfodder tcannonfodder changed the title Refactor Refactor RegistrationsControllerConcern.verify_passkey_registration_challenge to break out credential integrity verification into its own before_action Jun 24, 2023
@Vagab
Copy link
Contributor

Vagab commented Jun 24, 2023

I’ll take it on

@tcannonfodder
Copy link
Contributor Author

@Vagab I definitely appreciate the gusto, but please wait for @johalloran01 to decide if it’s a project he’d like to tackle first. I’m helping him with mentoring/apprenticing; and trying to give him real-world issues to tackle as part of that process 😊

@johalloran01
Copy link
Contributor

@tcannonfodder Thanks for looking out, I want work on the other three tasks first then I would love to look at the solution. @Vagab you're clear for this if you wish to take it! 👍

@Vagab
Copy link
Contributor

Vagab commented Jun 25, 2023

sure, my pleasure

@Vagab
Copy link
Contributor

Vagab commented Jul 17, 2023

@tcannonfodder a bit swamped at work currently, sorry if I mislead anyone. If someone wants to take this issue - feel free. Otherwise, I'll work on it in a week or two.

@tcannonfodder
Copy link
Contributor Author

No worries! I know OSS work is free/spare time stuff; and I’ve been there before too numerous times. :) Just glad you’ve been able to help contribute so far! 💪

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants