Skip to content
This repository was archived by the owner on Sep 1, 2020. It is now read-only.
This repository was archived by the owner on Sep 1, 2020. It is now read-only.

OP-BackChannel-Discovery and OP-FrontChannel-Discovery don't assert that OP supports back- and front-channel logout #223

Description

@travisspencer

I have an OP that doesn't support back- or front-channel logout, yet OP-BackChannel-Discovery and OP-FrontChannel-Discovery pass. Here's the salient parts of the metadata of the OP under test (as reported on the ProviderConfigurationResponse step of that tests):

{
    "backchannel_logout_session_supported": false,
    "backchannel_logout_supported": false,
    "frontchannel_logout_session_supported": false,
    "frontchannel_logout_supported": false,
...
}

Do these test pass simply because those fields are in the JSON object? Shouldn't they also assert that the values are true?

Also, there's no check for end_session_endpoint in the metadata. Shouldn't there be one?

As a point of comparison, the test OP-Session-Discovery also fails on this OP with this error:

These following required claims are missing: ['check_session_iframe', 'end_session_endpoint']

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions