The Wayback Machine - https://web.archive.org/web/20220212221515/https://github.com/go-gitea/gitea/pull/16544
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

Jump to bottom

Add microsoft oauth2 providers #16544

Merged
merged 9 commits into from Aug 6, 2021
Merged

Add microsoft oauth2 providers #16544

merged 9 commits into from Aug 6, 2021

Conversation

@zeripath
Copy link
Contributor

@zeripath zeripath commented Jul 25, 2021 •
edited

Users commonly want to be able to use their AzureAD or MicrosoftOnline accounts with Gitea.

This PR adds the goth.Providers for Azure and microsoft online.

Fix #6866
Fix #12073
@zeripath zeripath force-pushed the add-microsoft-oauth2-providers branch from dd7497e to a1058ae Jul 25, 2021
@codecov-commenter
Copy link

@codecov-commenter codecov-commenter commented Jul 25, 2021 •
edited

Codecov Report

Merging #16544 (f629cd2) into main (6e6f23b) will increase coverage by 0.02%.
The diff coverage is 45.10%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #16544      +/-   ##
==========================================
+ Coverage   45.37%   45.39%   +0.02%     
==========================================
  Files         750      756       +6     
  Lines       85115    85247     +132     
==========================================
+ Hits        38622    38700      +78     
- Misses      40244    40283      +39     
- Partials     6249     6264      +15
Impacted Files Coverage Δ
routers/web/user/setting/security.go 30.30% <0.00%> (-2.49%) ⬇️
services/auth/source/oauth2/source_name.go 0.00% <0.00%> (ø)
services/auth/source/oauth2/source_register.go 0.00% <0.00%> (ø)
services/forms/auth_form.go 100.00% <ø> (ø)
services/auth/source/oauth2/urlmapping.go 27.58% <27.58%> (ø)
services/auth/source/oauth2/providers.go 26.66% <33.33%> (+21.49%) ⬆️
services/auth/source/oauth2/providers_simple.go 42.30% <42.30%> (ø)
services/auth/source/oauth2/providers_openid.go 53.33% <53.33%> (ø)
routers/web/admin/auths.go 48.72% <56.25%> (-1.46%) ⬇️
services/auth/source/oauth2/providers_custom.go 63.23% <63.23%> (ø)
... and 15 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6e6f23b...f629cd2. Read the comment docs.

@techknowlogick techknowlogick added this to the 1.16.0 milestone Jul 26, 2021
@zeripath zeripath mentioned this pull request Jul 27, 2021
Closed
@techknowlogick techknowlogick merged commit ab9bb54 into go-gitea:main Aug 6, 2021
2 checks passed
@zeripath zeripath deleted the add-microsoft-oauth2-providers branch Aug 6, 2021
@zeripath zeripath mentioned this pull request Aug 6, 2021
Closed
zeripath added a commit to zeripath/gitea that referenced this issue Aug 17, 2021
@zeripath @zeripath
Fix Github oauth provider configuration
1de92b1 
There was a regression in go-gitea#16544 whereby the default token url
for github was changed to the gitea one.

This PR restores this.

Signed-off-by: Andrew Thornton <[email protected]>
@zeripath zeripath mentioned this pull request Aug 17, 2021
Closed
zeripath pushed a commit that referenced this issue Aug 17, 2021
@techknowlogick
Use github token URL when fetching oauth token (#16709)
37d0d94 
Fixes regression from #16544
@zeripath zeripath mentioned this pull request Aug 17, 2021
Merged
zeripath added a commit to zeripath/gitea that referenced this issue Aug 20, 2021
@zeripath @zeripath
Fix regression from go-gitea#16544
734a902 
Signed-off-by: Andrew Thornton <[email protected]>
zeripath added a commit to zeripath/gitea that referenced this issue Aug 20, 2021
@zeripath @zeripath
Fix openidConnect source regression from go-gitea#16544
28499e4 
Unfortunately there is bug in go-gitea#16544 meaning that openid connects aren't
being matched properly as the capitalisation in that PR is incorrect.

This PR changes the capitalisation back to what is expected.

Signed-off-by: Andrew Thornton <[email protected]>
@zeripath zeripath mentioned this pull request Aug 20, 2021
Merged
zeripath added a commit to zeripath/gitea that referenced this issue Aug 21, 2021
@zeripath @zeripath
Fix regression from go-gitea#16544
945aa43 
Signed-off-by: Andrew Thornton <[email protected]>
zeripath added a commit to zeripath/gitea that referenced this issue Aug 21, 2021
@zeripath @zeripath
Fix regression from go-gitea#16544
ddc1966 
Signed-off-by: Andrew Thornton <[email protected]>
zeripath added a commit that referenced this issue Aug 22, 2021
@zeripath @zeripath
Fix openidConnect source regression from #16544 (#16759)
7f85610 
Unfortunately there is bug in #16544 meaning that openid connects aren't
being matched properly as the capitalisation in that PR is incorrect.

This PR changes the capitalisation back to what is expected.

Signed-off-by: Andrew Thornton <[email protected]>
@go-gitea go-gitea locked and limited conversation to collaborators Oct 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@techknowlogick techknowlogick

@lafriks lafriks

Assignees
No one assigned
Labels
kind/feature lgtm/done theme/authentication
Milestone
1.16.0
5 participants
@zeripath @codecov-commenter @techknowlogick @lafriks @GiteaBot