aad-sso-wordpress icon indicating copy to clipboard operation
aad-sso-wordpress copied to clipboard

Added functionality to retain URL parameters for login redirects

Open brm252 opened this issue 7 years ago • 1 comments

brm252 avatar May 29 '18 14:05 brm252

In order to ensure proper parsing/encoding, I would suspect that another nested http_build_query should be implemented to "carry over" any params in this fashion...

'redirect_uri'  => $settings->redirect_uri
     . $_SERVER['QUERY_STRING']
          ? "?" . http_build_query( $_GET )
          : "",

Overall, I am concerned that adding data to the authorization loop might break something. @psignoret would know more about this.

bradkovach avatar May 29 '18 19:05 bradkovach