Skip to content

filter files containing a hash in the path for ftp storages#21789

Merged
MorrisJobke merged 1 commit intomasterfrom
ftp-filter-hash
Jul 13, 2020
Merged

filter files containing a hash in the path for ftp storages#21789
MorrisJobke merged 1 commit intomasterfrom
ftp-filter-hash

Conversation

@icewind1991
Copy link
Member

the php ftp streamwrapper doesn't handle hashes correctly and will break when it tries to enter a path containing a hash.

By filtering out paths containing a hash we can at least stop the external storage from breaking completely

Signed-off-by: Robin Appelman robin@icewind.nl

the php ftp streamwrapper doesn't handle hashes correctly and will break when it tries to enter a path containing a hash.

By filtering out paths containing a hash we can at least stop the external storage from breaking completely

Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Jul 10, 2020
@icewind1991 icewind1991 added this to the Nextcloud 20 milestone Jul 10, 2020
Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fair enough....

@MorrisJobke MorrisJobke merged commit 7569bc1 into master Jul 13, 2020
@MorrisJobke MorrisJobke deleted the ftp-filter-hash branch July 13, 2020 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants