-
-
Notifications
You must be signed in to change notification settings - Fork 593
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
fix: Parse.Hooks
requests have double forward slash in URL
#2441
Conversation
Thanks for opening this pull request! |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## alpha #2441 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 64 64
Lines 6256 6256
Branches 1476 1476
=========================================
Hits 6256 6256 ☔ View full report in Codecov by Sentry. |
Could this possibly be a breaking change, thinking about custom parsing of the URI on the server or firewall? |
We might want to merge this as a breaking change, make a stable release and then add that to the Parse Sever 8 release as a breaking change. Even though these are internal requests and therefore may be less likely to be restricted by a firewall, it's still possible. |
BREAKING CHANGE: Internal REST requests for @dplewis Is this correct? |
LGTM! |
# [6.0.0-alpha.2](6.0.0-alpha.1...6.0.0-alpha.2) (2025-02-16) ### Bug Fixes * `Parse.Hooks` requests have double forward slash in URL ([#2441](#2441)) ([1fc520c](1fc520c)) ### BREAKING CHANGES * Internal REST requests for `Parse.Hooks` use a URL that contains a double forward slash, for example `http://localhost/parse//hooks/functions`. This release changes the double forward slash to a single forward slash. ([1fc520c](1fc520c))
🎉 This change has been released in version 6.0.0-alpha.2 |
Pull Request
Issue
Closes: #2406
Approach
Remove extra slash
Tasks