Skip to content

Fix URL escaping of shared files#509

Merged
skjnldsv merged 1 commit intonextcloud:masterfrom
Talv:fix-shares-preview
Nov 7, 2020
Merged

Fix URL escaping of shared files#509
skjnldsv merged 1 commit intonextcloud:masterfrom
Talv:fix-shares-preview

Conversation

@Talv
Copy link
Contributor

@Talv Talv commented Jun 5, 2020

This fixes nextcloud/server#20671 which was incorrectly flagged (at least I believe so) as duplicate of #415 .

I'm unable to reproduce #415 on JPG files, and I doubt these two are the same issues.

@Talv Talv force-pushed the fix-shares-preview branch from f7e8b34 to bef28f1 Compare June 5, 2020 11:38
@skjnldsv
Copy link
Member

skjnldsv commented Jun 16, 2020

Yeah, that whole thing was confusing.

There was some attempts that broke other stuff
#393 (comment)
#395

So all of this works on various browsers and files with ? or # in them? :)

@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish bug Something isn't working labels Nov 7, 2020
@skjnldsv
Copy link
Member

skjnldsv commented Nov 7, 2020

/compile amend /

@skjnldsv skjnldsv added the high High priority label Nov 7, 2020
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
@skjnldsv skjnldsv merged commit 71e7a68 into nextcloud:master Nov 7, 2020
@skjnldsv
Copy link
Member

skjnldsv commented Nov 7, 2020

/backport to stable20

@backportbot-nextcloud backportbot-nextcloud bot added the backport-request Pending backport by the backport-bot label Nov 7, 2020
@skjnldsv
Copy link
Member

skjnldsv commented Nov 7, 2020

/backport to stable19

@skjnldsv
Copy link
Member

skjnldsv commented Nov 7, 2020

/backport to stable18

@backportbot-nextcloud
Copy link

The backport to stable18 failed. Please do this backport manually.

@skjnldsv skjnldsv removed the backport-request Pending backport by the backport-bot label Nov 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug Something isn't working high High priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issues with public shares with a plus in sub folders

2 participants