Skip to content

fix: Remove the extra " on the uuid for the membership endpoint CY-7670#149

Merged
machadoit merged 1 commit into
masterfrom
fix-apis-CY-7670
Feb 9, 2026
Merged

fix: Remove the extra " on the uuid for the membership endpoint CY-7670#149
machadoit merged 1 commit into
masterfrom
fix-apis-CY-7670

Conversation

@machadoit

Copy link
Copy Markdown
Contributor

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

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
@machadoit
machadoit requested a review from zhamborova February 9, 2026 13:28
@machadoit
machadoit requested a review from lolgab as a code owner February 9, 2026 13:28
Copilot AI review requested due to automatic review settings February 9, 2026 13:28
@codacy-production

Copy link
Copy Markdown

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.00% 0.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (615538c) 9 0 0.00%
Head commit (722233c) 9 (+0) 0 (+0) 0.00% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#149) 1 0 0.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@codacy-production

Copy link
Copy Markdown

Codacy's Analysis Summary

0 new issue (≤ 0 minor issue)
0 new security issue (≤ 0 issue)
0 complexity
0 duplications

Review Pull Request in Codacy →

AI Reviewer available: add the codacy-review label to get contextual insights without leaving GitHub.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 workspaceUUID in double quotes before URLEncoder.encode(...) in getWorkspaceMembership.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@machadoit
machadoit merged commit e2238a5 into master Feb 9, 2026
11 checks passed
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.

3 participants