Skip to content

Auth OIDC #6

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.

Sign up for GitHub

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
Open
wants to merge 78 commits into
base: dev
Choose a base branch
from
Open

Auth OIDC #6

wants to merge 78 commits into from

Conversation

sjanssen2
Copy link
Member

No description provided.

@coveralls
Copy link

coveralls commented Mar 20, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 13656403173

Details

  • 83 of 161 (51.55%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.2%) to 92.15%
Changes Missing Coverage Covered Lines Changed/Added Lines %
qiita_pet/webserver.py 2 3 66.67%
qiita_db/user.py 2 17 11.76%
qiita_pet/handlers/auth_handlers.py 23 85 27.06%
Totals Coverage Status
Change from base Build 13655898905: -0.2%
Covered Lines: 25015
Relevant Lines: 27146
💛 - Coveralls

sjanssen2 added 27 commits March 20, 2024 12:12
@sjanssen2
also provide a label for a speaking name of the identity provider
2840601
@sjanssen2
start implementing the OIDC dance
f1c9149
@sjanssen2
modal not necessary, if only one provider was defined
2eb6d08
@sjanssen2
error handling of provider not in config file
48ca02a
@sjanssen2
adding pycurl package to enable tornado curl_httpclients
dc4bd20
@sjanssen2
a new method to create a user, if information do not need to be enter…
e1f3c13 
…ed manually, but are obtained from an external identity provider
@sjanssen2
full OIDC dance implemented
48f09a5
@sjanssen2
add an admin page to activate users which requested authorization thr…
baf40df 
…ough OIDC
@sjanssen2
flake8
670a55a
@sjanssen2
adding menu entry for user authorization
091ffc6
@sjanssen2
do not expose traditional qiita internal user authentication, if OIDC…
1feefc0 
… is configured
@sjanssen2
use Qiita typical modal for OIDC login
29ce7dd
@sjanssen2
wrong menu entrie affected
2ca5bb8
@sjanssen2
always allow logout
1b787cb
@sjanssen2
improved error handling
88319b2
@sjanssen2
Merge branch 'dev' of https://github.com/qiita-spots/qiita into auth_…
02d9af0 
…oidc
@sjanssen2
revert: let user change their profile, but not password - if provided…
b1e1b6b 
… through OIDC
@sjanssen2
speaking button names + move into correct div to always get displayed
a7d3b84
@sjanssen2
use email from config + loop user_info from OIDC to fill DB
125835a
@sjanssen2
use OIDC info to prefil user information
5f28092
@sjanssen2
drop admin user authorization
19b4d7b
@sjanssen2
Merge branch 'dev' of https://github.com/qiita-spots/qiita into auth_…
33f2879 
…oidc
@sjanssen2
using the well-known json dict instead of manually providing multiple…
c9d413a 
… API endpoints through the config file
@sjanssen2
Merge branch 'dev' of https://github.com/qiita-spots/qiita into auth_…
6bfafcb 
…oidc_wellknown
@sjanssen2
flake8
9a5e7cc
@sjanssen2
flake8
b2fc279
@sjanssen2
add ability to display OIDC logos
5cc0896
sjanssen2 and others added 30 commits March 4, 2025 15:46
@sjanssen2
error handling of provider not in config file
51307d1
@sjanssen2
adding pycurl package to enable tornado curl_httpclients
7a0ec9f
@sjanssen2
a new method to create a user, if information do not need to be enter…
0c365a1 
…ed manually, but are obtained from an external identity provider
@sjanssen2
full OIDC dance implemented
e993a99
@sjanssen2
add an admin page to activate users which requested authorization thr…
ca5f7f6 
…ough OIDC
@sjanssen2
flake8
4d5c6a2
@sjanssen2
adding menu entry for user authorization
fd6d15e
@sjanssen2
do not expose traditional qiita internal user authentication, if OIDC…
9c8b824 
… is configured
@sjanssen2
use Qiita typical modal for OIDC login
a654e48
@sjanssen2
always allow logout
27f6d35
@sjanssen2
improved error handling
85bf1fa
@sjanssen2
revert: let user change their profile, but not password - if provided…
8a504cc 
… through OIDC
@sjanssen2
speaking button names + move into correct div to always get displayed
ef05eed
@sjanssen2
use email from config + loop user_info from OIDC to fill DB
a5270a0
@sjanssen2
use OIDC info to prefil user information
2efb70f
@sjanssen2
drop admin user authorization
c8b1198
@sjanssen2
using the well-known json dict instead of manually providing multiple…
3d6f718 
… API endpoints through the config file
@sjanssen2
flake8
3957030
@sjanssen2
flake8
648f2f9
@sjanssen2
add ability to display OIDC logos
73f92b9
@sjanssen2
add OIDC logo
0dc243d
@sjanssen2
fixing config manager tests
9b81163
@sjanssen2
Merge branch 'auth_oidc' of github.com:jlab/qiita into auth_oidc
bb03167
@sjanssen2
create neccessary "mountpoints" (qiita-spots#3462)
0a29ac2 
* add function that creates neccessary "mountpoints"

* code style

* addressing Antonio's suggestion to use get_mountpoint

* raise error if src dir is not present at expected location

* adding mountpoint directory FASTQ
@sjanssen2
Merge branch 'master' of github.com:qiita-spots/qiita into auth_oidc
a76288b
@sjanssen2
Merge branch 'dev' of github.com:qiita-spots/qiita into auth_oidc
8cc718f
@sjanssen2
fix sub-directory path (qiita-spots#3464)
5ab3ebb 
When I executed this code, I found that get_mount_point() returns complete paths, e.g. `/root/data/job` instead of "just" the sub-directory, i.e. `job`.
Thus `get_support_file` returns the dst directory, instead of the src directory :-/
@sjanssen2
Merge branch 'master' of github.com:qiita-spots/qiita into auth_oidc
89cab41
@sjanssen2 @antgonza
Fix multiple validation jobs (qiita-spots#3465)
d0e03de 
* Update CHANGELOG.md

* multiple validation jobs should be submitted as lead and dependent jobs, but the later must also made known by the DB

---------

Co-authored-by: Antonio Gonzalez <[email protected]>
@sjanssen2
Merge branch 'dev' of github.com:qiita-spots/qiita into auth_oidc
1a61930
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sjanssen2 @coveralls @antgonza