Skip to content

Don't allow anchors and queries in remote urls - #19907

Merged
rullzer merged 2 commits into
masterfrom
bugfix/noid/dont-allow-anchor-and-queries-in-remote-url
Mar 13, 2020
Merged

Don't allow anchors and queries in remote urls#19907
rullzer merged 2 commits into
masterfrom
bugfix/noid/dont-allow-anchor-and-queries-in-remote-url

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Signed-off-by: Joas Schilling coding@schilljs.com

Signed-off-by: Joas Schilling <coding@schilljs.com>
@rullzer

Copy link
Copy Markdown
Member

CI says no

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

Ah, willReturnOnConsecutiveCalls doesn't use arrays as wiithConsecutive

@nickvergessennickvergessen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 13, 2020
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable18

@rullzer
rullzer merged commit dc0ee35 into masterMar 13, 2020
@rullzer
rullzer deleted the bugfix/noid/dont-allow-anchor-and-queries-in-remote-url branch March 13, 2020 14:51
@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable18 in #19933

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

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: sharing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@nickvergessen@rullzer@juliusknorr