-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Prevent double-login for Git HTTP and LFS and simplify login #15303
Merged
Commits on Apr 6, 2021
-
Prevent double-login for Git HTTP and LFS and simplify login
There are a number of inconsistencies with our current methods for logging in for git and lfs. The first is that there is a double login process. This is particularly evident in 1.13 where there are no less than 4 hash checks for basic authentication due to the previous IsPasswordSet behaviour. This duplicated code had individual inconsistencies that were not helpful and caused confusion. This PR does the following: * Remove the specific login code from the git and lfs handlers except for the lfs special bearer token * Simplify the meaning of DisableBasicAuthentication to allow Token and Oauth2 sign-in. * The removal of the specific code from git and lfs means that these both now have the same login semantics and can - if not DisableBasicAuthentication - login from external services. Further it allows Oauth2 token authentication as per our standard mechanisms. * The change in the recovery handler prevents the service from re-attempting to login - primarily because this could easily cause a further panic and it is wasteful. Signed-off-by: Andrew Thornton <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 077f8bd - Browse repository at this point
Copy the full SHA 077f8bdView commit details -
Signed-off-by: Andrew Thornton <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 9f78255 - Browse repository at this point
Copy the full SHA 9f78255View commit details -
Signed-off-by: Andrew Thornton <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 7169169 - Browse repository at this point
Copy the full SHA 7169169View commit details
Commits on Apr 7, 2021
-
Configuration menu - View commit details
-
Copy full SHA for aae6847 - Browse repository at this point
Copy the full SHA aae6847View commit details -
actually get the user from the context if present
Signed-off-by: Andrew Thornton <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 58c821a - Browse repository at this point
Copy the full SHA 58c821aView commit details -
Merge branch 'disable-basic-authentication-2' of github.com:zeripath/…
…gitea into disable-basic-authentication-2
Configuration menu - View commit details
-
Copy full SHA for 7759de8 - Browse repository at this point
Copy the full SHA 7759de8View commit details -
Configuration menu - View commit details
-
Copy full SHA for ee27981 - Browse repository at this point
Copy the full SHA ee27981View commit details
Commits on Apr 8, 2021
-
Configuration menu - View commit details
-
Copy full SHA for 89fe166 - Browse repository at this point
Copy the full SHA 89fe166View commit details
Commits on Apr 10, 2021
-
Configuration menu - View commit details
-
Copy full SHA for 1540f68 - Browse repository at this point
Copy the full SHA 1540f68View commit details
Commits on Apr 15, 2021
-
Configuration menu - View commit details
-
Copy full SHA for fb27561 - Browse repository at this point
Copy the full SHA fb27561View commit details
Commits on May 9, 2021
-
Configuration menu - View commit details
-
Copy full SHA for 454efa8 - Browse repository at this point
Copy the full SHA 454efa8View commit details
Commits on May 10, 2021
-
Configuration menu - View commit details
-
Copy full SHA for ecfe6c4 - Browse repository at this point
Copy the full SHA ecfe6c4View commit details
Commits on May 12, 2021
-
Configuration menu - View commit details
-
Copy full SHA for bbd09e9 - Browse repository at this point
Copy the full SHA bbd09e9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6ec5e60 - Browse repository at this point
Copy the full SHA 6ec5e60View commit details -
Configuration menu - View commit details
-
Copy full SHA for b272dc8 - Browse repository at this point
Copy the full SHA b272dc8View commit details
Commits on May 15, 2021
-
Configuration menu - View commit details
-
Copy full SHA for 994c094 - Browse repository at this point
Copy the full SHA 994c094View commit details -
Configuration menu - View commit details
-
Copy full SHA for bd5b635 - Browse repository at this point
Copy the full SHA bd5b635View commit details
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.