fix: Incorrect dependency chain of Parse uses browser build instead of Node build#10067
Conversation
|
🚀 Thanks for opening this pull request! |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
📝 WalkthroughWalkthroughSwaps the Parse SDK import in Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Parse import in FilesControllerParse import in FilesController uses browser build instead of Node build
Parse import in FilesController uses browser build instead of Node buildParse uses browser build instead of Node build
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## alpha #10067 +/- ##
==========================================
+ Coverage 92.16% 92.57% +0.41%
==========================================
Files 191 191
Lines 15603 15603
Branches 176 176
==========================================
+ Hits 14380 14445 +65
+ Misses 1207 1146 -61
+ Partials 16 12 -4 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
# [9.3.0-alpha.8](9.3.0-alpha.7...9.3.0-alpha.8) (2026-02-21) ### Bug Fixes * Incorrect dependency chain of `Parse` uses browser build instead of Node build ([#10067](#10067)) ([1a2521d](1a2521d))
|
🎉 This change has been released in version 9.3.0-alpha.8 |
# [9.3.0](9.2.0...9.3.0) (2026-02-21) ### Bug Fixes * `Parse.Query.select('authData')` for `_User` class doesn't return auth data ([#10055](#10055)) ([44a5bb1](44a5bb1)) * AuthData validation incorrectly triggered on unchanged providers ([#10025](#10025)) ([d3d6e9e](d3d6e9e)) * Default ACL overwrites custom ACL on `Parse.Object` update ([#10061](#10061)) ([4ef89d9](4ef89d9)) * Default HTML pages for password reset, email verification not found ([#10034](#10034)) ([e299107](e299107)) * Default HTML pages for password reset, email verification not found ([#10041](#10041)) ([a4265bb](a4265bb)) * Incorrect dependency chain of `Parse` uses browser build instead of Node build ([#10067](#10067)) ([1a2521d](1a2521d)) * Unlinking auth provider triggers auth data validation ([#10045](#10045)) ([b6b6327](b6b6327)) ### Features * Add `Parse.File.url` validation with config `fileUpload.allowedFileUrlDomains` against SSRF attacks ([#10044](#10044)) ([4c9c948](4c9c948)) * Add event information to `verifyUserEmails`, `preventLoginWithUnverifiedEmail` to identify invoking signup / login action and auth provider ([#9963](#9963)) ([ed98c15](ed98c15)) * Add support for streaming file upload via `Buffer`, `Readable`, `ReadableStream` ([#10065](#10065)) ([f0feb48](f0feb48)) * Upgrade to parse 8.2.0, @parse/push-adapter 8.3.0 ([#10066](#10066)) ([8b5a14e](8b5a14e))
|
🎉 This change has been released in version 9.3.0 |
Pull Request
Issue
Incorrect dependency chain of
Parseuses browser build instead of Node build. This causes all kinds of difficult to debug bugs becauseParsebuilds for browser and Node are imported, may or not be initialized and overwrite each other due to a globally pinnedParseobject. The fundamental fix will be #8787 in the future. This PR fixed an immediate issue that causes CI fails and can cause production code issues. Part of the fix is the update to@parse/push-adapter8.3.1 which also imports the browser build ofParseinstead of the Node build. This and the import in Parse Server's FilesController produce this strange interaction.Summary by CodeRabbit