Skip to content
This repository has been archived by the owner on Feb 4, 2025. It is now read-only.

Custom challenge implementation with unit tests and refactor #2

Merged
merged 73 commits into from
Nov 26, 2024

Conversation

basvandriel
Copy link
Contributor

On top of this pull request on the original repository, a custom challenge was implemented too. For this, an JWT is passed through the challenge endpoint.

@basvandriel basvandriel requested a review from a team as a code owner November 21, 2024 10:01
@basvandriel
Copy link
Contributor Author

Alternatively, we could also create the original pull request for the unit tests and refactoring here and merge that first. That would lower the amount of changes.

@basvandriel
Copy link
Contributor Author

@gerbrand-bosch
Copy link
Contributor

Alternatively, we could also create the original pull request for the unit tests and refactoring here and merge that first. That would lower the amount of changes.

That would make the review a lot easier. I don't care that much about the tests and refactor for the sake of a PoC.

gerbrand-bosch
gerbrand-bosch previously approved these changes Nov 21, 2024
Copy link
Contributor

@gerbrand-bosch gerbrand-bosch left a comment

Choose a reason for hiding this comment

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

I see no issues to use this as a PoC implementation.

Co-authored-by: gerbrand-bosch <[email protected]>
@basvandriel basvandriel merged commit fbd1b79 into main Nov 26, 2024
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants