Skip to content

Delete redundant urldecode - #10681

Closed
soudis wants to merge 1 commit into
nextcloud:masterfrom
soudis:patch-1
Closed

Delete redundant urldecode#10681
soudis wants to merge 1 commit into
nextcloud:masterfrom
soudis:patch-1

Conversation

@soudis

Copy link
Copy Markdown

As discussed in #6822 the urdecode is redundant and creates problems (especially with the discoursesso app)

As discussed in #6822 the urdecode is redundant and creates problems (especially with the discoursesso app)
@blizzz

Copy link
Copy Markdown
Member

I cannot reproduce it on current master, the link as described ("https://nextcloud13.local/index.php/apps/activity/?filter=b%0Ay") would bring me to activities, no error page. I guess we can close that, @soudis?

@soudis

soudis commented Nov 1, 2018

Copy link
Copy Markdown
Author

I cannot reproduce it on current master, the link as described ("https://nextcloud13.local/index.php/apps/activity/?filter=b%0Ay") would bring me to activities, no error page. I guess we can close that, @soudis?

Sorry, I was not precise enough, you need to be logged out, because this redirects you to:

https://.../index.php/login?redirect_url=/index.php/apps/activity/%3Ffilter%3Db%250Ay

which then leads to the error / blank page.

@ChristophWurst

Copy link
Copy Markdown
Member

-> #13137

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@soudis@blizzz@ChristophWurst