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

Compatiblity configuration #1106

Open
UweKeim opened this issue Jul 23, 2019 · 1 comment
Open

Compatiblity configuration #1106

UweKeim opened this issue Jul 23, 2019 · 1 comment

Comments

@UweKeim
Copy link

UweKeim commented Jul 23, 2019

When looking at the documentation for the <compatibility> element, it lists these attributes:

  • unpackEntitiesDescriptorInIdentityProviderMetadata
  • IgnoreAuthenticationContextInResponse

In contrast, when looking at the "Compatibility.cs" source file, I see that there are these properties:

  • UnpackEntitiesDescriptorInIdentityProviderMetadata
  • DisableLogoutStateCookie
  • StrictOwinAuthenticationMode
  • IgnoreAuthenticationContextInResponse
  • IgnoreMissingInResponseTo

From the c'tor in that very source code file, it seems that only these are actually filled from the XML configuration file:

  • UnpackEntitiesDescriptorInIdentityProviderMetadata
  • DisableLogoutStateCookie
  • IgnoreMissingInResponseTo

and these ones are assignable by code only, not by XML configuration file:

  • StrictOwinAuthenticationMode
  • IgnoreAuthenticationContextInResponse

What bothers me here is:

  • The documentation seems to be wrong, citing properties that acutally are not assigned from XML configuration file (IgnoreAuthenticationContextInResponse) while omitting properties that are acutally filled from XML configuration file (IgnoreMissingInResponseTo).

My questions:

  • Are my assumptions right, or did I completely confuse something?
  • Any chance to make all these properties configurably via the <compatibility> XML configuration file?
@afandre
Copy link

afandre commented Oct 9, 2019

ignoreMissingInResponseTo would help me because I need Sp initiated login.

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

No branches or pull requests

3 participants