fix: Remove the extra " on the uuid for the membership endpoint CY-7670#149
Conversation
Before it was needed to be wrapped in quotes, as the UUID was sent on the body (the code was wrapping the UUID in quotes). As it is now part of the URL we just need to encode it, not to add an extra pair of doulbe quotes
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferences |
Codacy's Analysis Summary0 new issue (≤ 0 minor issue) Review Pull Request in Codacy →
|
There was a problem hiding this comment.
Pull request overview
This PR fixes URL encoding for the Bitbucket v2 “workspace membership/permission” user endpoint by removing an unintended extra pair of double quotes around the workspace UUID before encoding, aligning with the UUID now being passed as part of the URL path.
Changes:
- Stop wrapping
workspaceUUIDin double quotes beforeURLEncoder.encode(...)ingetWorkspaceMembership.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Before it was needed to be wrapped in quotes, as the UUID was sent on the body (the code was wrapping the UUID in quotes). As it is now part of the URL we just need to encode it, not to add an extra pair of doulbe quotes