Closed anglov closed 3 years ago
I have looked in the history and I do not know why the returnTo
url is wrapped in encodeURI
, I will do some investigation and remove if necessary.
Seems returnTo
is used as-is throughout NodeBB, so it should be safe to remove it.
nodebb version: 1.15.5 plugin version: 4.6.10 Revalidate option is enabled in plugin config.
When url having utf-8 characters – for example category name is „Zażółć gęslą jaźń” and just reload page, redirection loop will be encoutered.
In this example:
and so on.
I found that https://github.com/julianlam/nodebb-plugin-session-sharing/blob/master/library.js#L487 removing encodeURI call fixes this problem, but not sure if it's a valid solution.