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.

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

Initial implementation #1

Merged
merged 19 commits into from
Nov 17, 2020
Merged

Initial implementation #1

merged 19 commits into from
Nov 17, 2020

Conversation

duskobogdanovski
Copy link
Member

Initial implementation of CKAN SSO with saml2 authorization.
Supported features:

  1. Internal CKAN login/register as fallback;
  2. Automatically create users as sysadmins by providing a list of emails;
  3. Custom SP config settings.

@duskobogdanovski
Initial implementation of CKAN SSO with saml2 authorization
cb29e88
@duskobogdanovski
Code refactoring
9866d67
@duskobogdanovski
Add config for user attributes mapping and check if config exists on …
3f274f8 
…runtime
@duskobogdanovski
Move config from plugin to a separate module and make it to read the …
d0ac1c0 
…specific attributes from the CKAN config, add comments
@duskobogdanovski
Add function for generating password
11a2875
@duskobogdanovski
Use string formating for user fullname
0b6e49a
@duskobogdanovski
Add required dependencies and update docs
31c10a9
@duskobogdanovski
Add config to enable default ckan login and register as fallback
1b8b2fb
@duskobogdanovski
Update gonfig setting name
5db5ed5
@duskobogdanovski
Add exception handling on create and update user
63ca584
@duskobogdanovski
Add functionality that makes users sysadmins if user email is found i…
ddd2868 
…n a previously configured list of email addresses and oposite
@duskobogdanovski
Add additional configuration parameters
f9478d7
@duskobogdanovski
When internal login enabled there might be a regular CKAN user with s…
096cb8a 
…ame email like the incoming SAML user, in this case use that existing CKAN user and link it to the SAML user
@duskobogdanovski
Code refactoring
fb3f8ab
@duskobogdanovski
Add tests
0ca2238
@duskobogdanovski
Add tests
ca22d99
@duskobogdanovski
Fix travis build and PEP8 code style fixes
a571188
@duskobogdanovski
Bug fix, update docs
7b2e821
@duskobogdanovski
Add todo
9f32873
@mbocevski
Copy link
Member

Looks good!

@mbocevski mbocevski merged commit 21c68ce into main Nov 17, 2020
@mbocevski mbocevski deleted the initial-implementation branch November 17, 2020 12:38
duskobogdanovski pushed a commit that referenced this pull request Jul 1, 2022
@Jin-Sun-tts
Merge pull request #1 from GSA/fix-logout-issue
0493d1d 
added excpetion type for unspportedbinding.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blazhovsky blazhovsky Awaiting requested review from blazhovsky

@mbocevski mbocevski Awaiting requested review from mbocevski

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@duskobogdanovski @mbocevski