Skip to content

Comments

fix(Uploader): Pass on customHeaders when creating chunked upload workspace#1901

Merged
skjnldsv merged 1 commit intomainfrom
fix/uploader/chunked-upload-custom-headers
Oct 15, 2025
Merged

fix(Uploader): Pass on customHeaders when creating chunked upload workspace#1901
skjnldsv merged 1 commit intomainfrom
fix/uploader/chunked-upload-custom-headers

Conversation

@provokateurin
Copy link
Contributor

Frontend part for nextcloud/server#55366

Without this, https://github.com/nextcloud/server/blob/50f287402a3bd1374f890f79c9504cb816751c7e/apps/files_sharing/src/public-nickname-handler.ts#L26 will not have an effect on this request and gets blocked by the backend if it is a file drop.

@provokateurin provokateurin added bug Something isn't working 3. to review Waiting for reviews labels Oct 15, 2025
…kspace

Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin force-pushed the fix/uploader/chunked-upload-custom-headers branch from 3c06eca to dda9228 Compare October 15, 2025 19:06
@provokateurin
Copy link
Contributor Author

Ok I realized this is not necessary as the backend is the issue in this case, however it might not be so bad to merge this to avoid issues in the future if some other special header is required.

@skjnldsv skjnldsv merged commit 8f52c44 into main Oct 15, 2025
21 checks passed
@skjnldsv skjnldsv deleted the fix/uploader/chunked-upload-custom-headers branch October 15, 2025 20:45
@ShGKme
Copy link
Contributor

ShGKme commented Nov 17, 2025

/backport to next

@backportbot backportbot bot added the backport-request Pending backport by the backport-bot label Nov 17, 2025
@backportbot backportbot bot removed the backport-request Pending backport by the backport-bot label Nov 17, 2025
@ShGKme ShGKme mentioned this pull request Jan 13, 2026
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 bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants