Skip to content

Conversation

@mdavino
Copy link

@mdavino mdavino commented Jul 15, 2021

No description provided.

peppelinux and others added 3 commits June 27, 2021 22:21
* Update views.py (#297)

changed from error to debug

* Unsupported Binding exception in SLO

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
````

* v1.3.2

* chore: Documentation improvements that fixed #231 and #299 (@tkloda)

Co-authored-by: sur.la.route <christopher.pickering@keemail.me>
@peppelinux peppelinux merged commit bf54cc7 into IdentityPython:dev Jul 15, 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.

3 participants