Feat: only login activated users - Githubissues

uc-cdis / fence

AuthN/AuthZ OIDC Service

Apache License 2.0

37 stars 48 forks source link

Feat: only login activated users #1155

Closed pieterlukasse closed 1 week ago

pieterlukasse commented 2 months ago

Link to JIRA ticket if there is one: https://ctds-planx.atlassian.net/browse/VADC-1242

New Features

feat: ensure only activated users can login

Deployment changes

depends on a new configuration setting: AUTO_ACTIVATE_NEW_USER

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 9783944076

Details

0 of 0 changed or added relevant lines in 0 files are covered.
124 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.002%) to 75.049%

Files with Coverage Reduction	New Missed Lines	%
resources/ga4gh/passports.py	2	87.43%
auth.py	14	87.12%
config.py	14	75.44%
sync/sync_users.py	94	81.65%
<!--	Total:	124		-->

Totals
Change from base Build 9713845019:	-0.002%
Covered Lines:	7709
Relevant Lines:	10272

💛 - Coveralls

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 9784123176

Details

0 of 0 changed or added relevant lines in 0 files are covered.
124 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.002%) to 75.049%

Files with Coverage Reduction	New Missed Lines	%
resources/ga4gh/passports.py	2	87.43%
auth.py	14	87.12%
config.py	14	75.44%
sync/sync_users.py	94	81.65%
<!--	Total:	124		-->

Totals
Change from base Build 9713845019:	-0.002%
Covered Lines:	7709
Relevant Lines:	10272

💛 - Coveralls

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 9791563296

Details

0 of 0 changed or added relevant lines in 0 files are covered.
124 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.002%) to 75.049%

Files with Coverage Reduction	New Missed Lines	%
resources/ga4gh/passports.py	2	87.43%
auth.py	14	87.12%
config.py	14	75.44%
sync/sync_users.py	94	81.65%
<!--	Total:	124		-->

Totals
Change from base Build 9713845019:	-0.002%
Covered Lines:	7709
Relevant Lines:	10272

💛 - Coveralls

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 9791638922

Details

0 of 0 changed or added relevant lines in 0 files are covered.
122 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.002%) to 75.049%

Files with Coverage Reduction	New Missed Lines	%
resources/ga4gh/passports.py	2	87.43%
auth.py	12	87.12%
config.py	14	75.44%
sync/sync_users.py	94	81.65%
<!--	Total:	122		-->

Totals
Change from base Build 9713845019:	-0.002%
Covered Lines:	7709
Relevant Lines:	10272

💛 - Coveralls

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 9791719518

Details

0 of 0 changed or added relevant lines in 0 files are covered.
122 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.002%) to 75.049%

Files with Coverage Reduction	New Missed Lines	%
resources/ga4gh/passports.py	2	87.43%
auth.py	12	87.12%
config.py	14	75.44%
sync/sync_users.py	94	81.65%
<!--	Total:	122		-->

Totals
Change from base Build 9713845019:	-0.002%
Covered Lines:	7709
Relevant Lines:	10272

💛 - Coveralls

pieterlukasse commented 1 week ago

closing in favor of https://github.com/uc-cdis/fence/pull/1184