Enabling Discourse as SSO provider (in wordpress but that is unrelated) doesn’t work when setting the redirect paramenter to non-latin chars. for example this works: https://www.my_wordpress_website.com/?discourse_sso=1&redirect_to=https://www.my_wordpress_website.com/AAAA/ but this doesn’t work: https://www.my_wordpress_website.com/? discourse_sso=1&redirect_to=https://www.my_wordpress_website.com/אאאא/ by “doesn’t work” i mean that the bad url just redirects to an “Oops error page” in Discourse

@gerald is right, the plugin does not deal properly with URL and Unicode characters. I’m helping him on this matter. After investigating, it seems you need to encode properly the URL before sending the request: wp-discourse/lib/sso-client/query-redirect.php at main · discourse/wp-discourse · GitHub Possible and working suggestion: utf8_uri_encode($redirect_to) If you don’t encode, Discourse will not be happy and will error that URI must be ASCII only. Another place to fix is when the plugin receives the response: wp-discourse/lib/sso-client/client.php at main · discourse/wp-discourse · GitHub I’m not totally sure why urldecode and sanitize_text_field is used before the string is decoded, but once decoded, the sanitize_text_field call on the URL will strip all the encoded characters. Possible suggestion: urldecode needs to be called before. So far, those fixes seem to fix the issue gerald had.

Thanks, but Wordpress creates that url via this discourse plugin shortcode [discourse_sso_client login=‘Login here’ ] The redirect URL is automatically set to the page in which the shortcode was embedded.

You can add a redirect parameter to the shortcode: [discourse_sso_client login='Login Through the Forum' redirect=https://wordpress.example.com/%D7%90%D7%90%D7%90%D7%90/] That will create a link like this: Login Through the Forum You can also manually create a login link instead of using the shortcode. It needs to have the following query parameters: ?discourse_sso=1&redirect_to=https://example.com/your-wordpress-page

I am aware that I can pass the redirect string into the shortcode, but that is not always practical. For example in my case (which I think is common), the shortcode is embedded in a sidebar shared in all pages of the website. So overriding the redirect means that users will never be redirected to the right page. I guess i could write my own shortcode for that somehow… anyway, thanks for the answer and please consider this a feature request to support wordpress with unicode urls :slight_smile:

SSO redirect fails for non latin chars

Product Support WordPress

codinghorror (Jeff Atwood) November 8, 2018, 10:08pm 2

Use the transliterated URL characters instead. I wouldn’t expect unicode to work in that field.

https://www.my_wordpress_website.com/%D7%90%D7%90%D7%90%D7%90/

Topic		Replies	Views
SSO redirect link WordPress	8	4815	February 8, 2021
Wordpress SSO Provider - Blank Page WordPress	6	1074	July 10, 2019
WP Discourse SSO redirect loop WordPress	2	831	May 29, 2020
PAID Wordpress-Discourse SSO Marketplace	7	2159	August 12, 2015
2 questions about WP Discourse WordPress	5	1137	December 13, 2017

SSO redirect fails for non latin chars

Related topics