Parameter "logout_requests_signed" in config not signing logout requests

Code Version

6.5.1

Expected Behavior

I’m using pysaml2 through djangosaml2 as an SP. The ADFS IdP requires logout requests to be signed.

With this configuration:

SAML_CONFIG = {
    ...
    "service": {
        "sp": {
            "logout_requests_signed": True,
        }
    }
}

The logout requests (initiated by the SP) should be signed.

Current Behavior

The request is not signed and the IdP doesn’t accepts it.

Possible Solution

The djangosaml2 library calls the global_logout() method to initiate the logout, and doesn’t sets the sign argument. So either in global_logout() or do_logout() methods, the following could be added:

if sign is None:
    sign = self.logout_requests_signed

[joachimBurket]

Hello,

I tested with the latest HEAD on my project and it is working as intended. Thanks for the fix!

[c00kiemon5ter]

Hello, I will create a release soon. Sorry this did not happen faster.