-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Add WebAuthn support to GUI (fixes #8409) #8417
Closed
Closed
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
0ce39a7
Extract function createSession
emlun 3995406
Move HTTPS enablement check inside redirectToHTTPSMiddleware
emlun dfe0592
Redirect to HTTPS earlier in middleware chain
emlun a9c1304
Add WebAuthn support
emlun 9785197
Extract struct splitMux
emlun 9dea90a
Add ExcludeCredentials
emlun f3e6231
Undo translations
emlun 9d6f49f
Rewrite GUIConfiguration changes in protobuf
emlun c33d9aa
Don't use backwards-incompatible template string syntax
emlun 83e79d5
Set excludeCredentials in frontend instead of backend for consistency…
emlun 31ab9c7
Move method configMuxBuilder.finish to config.Wrapper.Finish
emlun a2699a2
Validate WebAuthn signature counter
emlun bd23bb2
Use logger instead of fmt.Println to print errors
emlun 646b893
Add comment about missing transports field
emlun 95497b1
Revert "Extract struct splitMux"
emlun ee9bfa6
Move static assets under /static/ HTTP path
emlun a80dc40
Replace basic auth prompt with dedicated login endpoint
emlun bd8bada
Don't enable password auth if password is not set
emlun 0114db5
Unset password when set to empty
emlun 9bc23f5
Add more warnings about WebAuth host config
emlun a40c2ce
Simplify webauthMux to webauthnService
emlun a4d5124
Show warnings on login page when WebAuthn is not available
emlun ff66255
Show more WebAuthn error feedback
emlun 5ef2a58
Don't drown console with errors on login page
emlun caf5b49
Reload login page on auth error
emlun b5a1603
Add advanced setting sendBasicAuthPrompt to continue sending 401 resp…
emlun 0ddf472
Fix HTML tag mismatch
emlun b0b1420
Merge pull request #1 from emlun/webauthn-keep-basic-auth
emlun 2efcfee
Merge remote-tracking branch 'upstream/main' into webauthn
emlun f147065
Revert mux changes and moving static assets under /static/
emlun 7db89e9
Move /authn/ API paths under /rest/noauth/
emlun b8c5c74
Show error message if WebAuthn authentication is attempted without an…
emlun 919bf3e
Address review comments
emlun 456e968
Clean up new console.log statements
emlun File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an archived repo, with it's readme point out this as it's successor: https://github.com/go-webauthn/webauthn
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I noticed but haven't gotten around to migrating it yet. I'll put it on the todo list.