Skip to content

v1.3.2 #298

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

Merged
merged 4 commits into from
Jun 20, 2021
Merged

v1.3.2 #298

merged 4 commits into from
Jun 20, 2021

Conversation

peppelinux
Copy link
Member

  • feat: Unsupported Binding exception in SLO - a workaround to go ahead when SLO is absent in IdP
  • custom sso POST template: changed from error to debug when theresn't any custom template to load (fallback to pysaml2 default)

sur.la.route and others added 3 commits June 15, 2021 20:32
changed from error to debug
There's the possibility that an IdP doesn't support the SLO, added an exception handling as a workaround for this.
Now we have in the log something like:

````
Unsupported binding: None (https://172.17.0.1:10000/Saml2IDP/metadata)
Error Handled - SLO - unsupported binding by IDP: None
Traceback (most recent call last):
  File "~/DEV/IdentityPython/env/lib/python3.8/site-packages/djangosaml2/views.py", line 601, in get
    result = client.global_logout(subject_id)
  File "~/DEV/IdentityPython/env/lib/python3.8/site-packages/saml2/client.py", line 210, in global_logout
    return self.do_logout(
  File "~/DEV/IdentityPython/env/lib/python3.8/site-packages/djangosaml2/overrides.py", line 25, in do_logout
    return super().do_logout(*args, **kwargs)
  File "~/DEV/IdentityPython/env/lib/python3.8/site-packages/saml2/client.py", line 259, in do_logout
    bindings_slo_supported = self.metadata.single_logout_service(
  File "~/DEV/IdentityPython/env/lib/python3.8/site-packages/saml2/mdstore.py", line 1236, in single_logout_service
    return self.service(entity_id, "%s_descriptor" % typ,
  File "~/DEV/IdentityPython/env/lib/python3.8/site-packages/saml2/mdstore.py", line 1147, in service
    raise UnsupportedBinding(binding)
saml2.s_utils.UnsupportedBinding: None
````
@peppelinux peppelinux changed the title Dev v1.3.2 Jun 20, 2021
@peppelinux peppelinux merged commit ae57a8c into master Jun 20, 2021
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.

1 participant