Skip to content
This repository has been archived by the owner on Oct 24, 2020. It is now read-only.

CAS backward compatibility patch #43

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

auxepaul
Copy link

@auxepaul auxepaul commented Apr 2, 2020

No re-encoding of ':' and '/' of entityId parameters when ShibCAS constructs the CAS service url ; backward compatibility with old CAS servers because they can't validate ST tickets when the encoding changes.

For example, [https://idp.example/idp/Authn/External?conversation=e1s1&entityId=https://sp.example] does not match supplied service [https://idp.example/idp/Authn/External?conversation=e1s1&entityId=https%3A%2F%2Fsp.example]

…backward compatibility with old java CAS clients)
…structs the CAS service url ; backward compatibility with old CAS servers because they can't validate ST tickets when the encoding changes.

For example, [https://idp.example/idp/Authn/External?conversation=e1s1&entityId=https://sp.example] does not match supplied service [https://idp.example/idp/Authn/External?conversation=e1s1&entityId=https%3A%2F%2Fsp.example]
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant