Uh oh!
There was an error while loading. Please reload this page.
[app:files] Use current directory for hashing. - #14674
Conversation
This fixes collisions that were causing uploads to break in a very terrible way. Kudos to @kesselb for finding the problematic place and to @hottwister for the proposed solution. Fixesnextcloud#10527.
kesselb
commented
Mar 14, 2019
Please keep in mind that these changes are only a part of the fix from owncloud. I'm still unsure if we need the other changes for something. |
kesselb
commented
Mar 14, 2019
@evilham could you please sign off your commits (check https://github.com/nextcloud/server/pull/14674/checks?check_run_id=77759184 for help)? |
MorrisJobke
commented
Mar 20, 2019
@evilham Any chance that you can add this? |
MorrisJobke
commented
Mar 20, 2019
Should also be fine without, because it's very little code change itself. |
MorrisJobke
commented
Mar 20, 2019
/backport to stable15 |
Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22 |
backport to stable15 in #14762 |
evilham
commented
Mar 22, 2019
Sorry for the delay and thank you for merging. Just as a comment: although I understand the motivation, I strongly disagree with the legal-name requirement; and indeed those bits of code are already unmistakeably under a compatible license. |
This fixes collisions that were causing uploads to break in a very
terrible way.
Kudos to @kesselb for finding the problematic place and to
@hottwister for the proposed solution.
Fixes#10527.
It would be great if this were backported to 15.0.