Skip to content

fix(deps): update uppy monorepo #12447

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Apr 8, 2025

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@uppy/core (source) 4.4.3 -> 4.4.4 age adoption passing confidence
@uppy/utils (source) 6.1.2 -> 6.1.3 age adoption passing confidence

Release Notes

transloadit/uppy (@​uppy/core)

v4.4.4

Compare Source


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

Copy link

update-docs bot commented Apr 8, 2025

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@renovate renovate bot force-pushed the renovate/uppy-monorepo branch 3 times, most recently from d65e6ba to c453532 Compare April 17, 2025 07:57
@LukasHirt
Copy link
Collaborator

LukasHirt commented Apr 22, 2025

I tried this locally and there seems to be the following error: Scratch that... was due to my setup. Upload works fine for me locally. I'll try to rebase this PR.

Refused to get unsafe header "Upload-Offset"
getHeader @ httpStack.js:110Understand this error
loggers.js:14 [Uppy] [10:40:22] tus: invalid or missing offset value, originated from request (method: HEAD, url: https://localhost:9200/data/eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJhdWQiOlsicmV2YSJdLCJleHAiOjE3NDUzOTc2MjAsImlhdCI6MTc0NTMxMTIyMCwidGFyZ2V0IjoiaHR0cDovL2xvY2FsaG9zdDo5MTU4L2RhdGEvdHVzL2I5NTE5NTNhLThlZjUtNGZiYS1iYzUzLTNlM2ZlMjcwMmEwNSJ9.M5HTeMsprjzxY5b3gwvhA2e4N1KuFb_gKQHKnLf0Vzg, response code: 200, response text: , request id: a9d23f3a-c9ac-43e4-a3ea-e3b36954a986)

@LukasHirt LukasHirt force-pushed the renovate/uppy-monorepo branch from c453532 to 264f4ff Compare April 22, 2025 08:47
@LukasHirt
Copy link
Collaborator

It seems that the complete event is now called when the upload is cancelled which was not the case with previous version.

@LukasHirt
Copy link
Collaborator

The complete event is emitted with empty payload when cancelling uploads. That breaks our code. I opened an issue in the uppy repo as I think this is a bug.

@LukasHirt
Copy link
Collaborator

The upstream issue got resolved. Let's wait for the release and then try to update it to the newer version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant