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

feat: CIE metadata endpoint and attributes #80

Merged
merged 2 commits into from
Jul 19, 2021
Merged

feat: CIE metadata endpoint and attributes #80

merged 2 commits into from
Jul 19, 2021

Conversation

peppelinux
Copy link
Member

Here a way to handle a CIE SP integration in the existing SPID SP.

  • feat: /cie/metadata/ endpoint
  • feat: SAML2 required attributes configured by matching on the metadata url path, if spid or cie

@peppelinux peppelinux requested a review from brunato July 18, 2021 00:05
@peppelinux
Copy link
Member Author

Maybe it would be better to have also the button "Entra con CIE" in the discovery page, insn't so @brunato?
it looks fun!

@@ -129,10 +129,6 @@ def test_get_config(self):
saml_config = get_config()
self.assertIsNone(saml_config.entityid)

request = self.factory.get("")
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@brunato is there some particular need to have this kind of check? It looks to me quite pedant, please tell me how you put it there, probably is useful for something.

as you can see I also changed this behaviour to our default config_loader,
tell me more when possibile, thx

@peppelinux peppelinux merged commit bfda394 into dev Jul 19, 2021
peppelinux pushed a commit that referenced this pull request Jul 19, 2021
* chore: spid metadata as django class view

* chore: README setup additional informations and tips

* - fix: spid-django example project now works also with django 2.2
- chore: removed spid-testenv2 from README
- feat: added 'allow_unsolicited` as customizable parameter in SAML_CONF  SETTING

* feat: CIE metadata endpoint and attributes (#80)

* feat: CIE metadata endpoint and attributes

* fix: removed useless test regarding the metadata web path

* v1.1.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants