-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Next companion major version (v4) #3180
Comments
mifi
added a commit
that referenced
this issue
Aug 10, 2022
mifi
added a commit
that referenced
this issue
Aug 10, 2022
Companion v4 has been released |
HeavenFox
pushed a commit
to docsend/uppy
that referenced
this issue
Jun 27, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I created this issue to collect any breaking changes that we want to do for the next major:
streamingUpload
true by default (and remove from all docs/examples where it is set totrue
)uppy/packages/@uppy/companion/src/companion.js
Line 42 in 50cd8c3
wrapLegacyProvider
backward compatibility code and default to the new async Provider APIuppy/packages/@uppy/companion/src/server/provider/index.js
Line 87 in 313d75b
uppy/packages/@uppy/companion/src/standalone/index.js
Line 85 in d0d117c
uploadUrls
is not set and if running in production, in order to prevent SSRF enforce usage of uploadUrls #3965 - see also Make uploadUrls recommended #3182providerOptions
) instead of their ownsearchProviders
to avoid confusion.The text was updated successfully, but these errors were encountered: