Closed (fixed)
Project:
SAML Authentication
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
23 Mar 2017 at 14:15 UTC
Updated:
25 Aug 2017 at 12:25 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
roderikBaseline: status as of #2670118-24: Make getPostLoginDestination / getPostLogoutDestination configurable with
* no_cache: TRUE added to the routing.yml (for links both before and after the redirect to the IDP);
* a large comment inserted somewhwere.
Comment #3
roderik@seanB: can you try out this patch? You'll probably need to patch the -dev version with this issue, in order to get the redirect to your external site to work. (Because I removed the following from the -dev code: #2670118-9: Make getPostLoginDestination / getPostLogoutDestination configurable. So you are the dedicated guinea pig now.)
Done:
Comment #4
roderikRenamed redirectResponseFromUrl() to createRedirectResponse(). ('redirect' is also a verb.)
Interdiff is from 1 to 3.
Comment #5
seanb#3 seems to work for me. Nice work!
Comment #6
pcambra+1 to the RTBC, looks great
Comment #7
roderikThanks all! Finally committed.