Skip to content

fix(browser): add missing /v1 prefix to instance API endpoints#170

Merged
amstuta merged 1 commit intomainfrom
me
Apr 3, 2026
Merged

fix(browser): add missing /v1 prefix to instance API endpoints#170
amstuta merged 1 commit intomainfrom
me

Conversation

@rbeuque74
Copy link
Copy Markdown
Member

Added the /v1 version prefix to two instance API endpoint paths in internal/services/browser/api.go that were missing it:

  • deleteInstance function (line 1592)
  • cleanupCreatedResources function (line 2131)

This ensures consistency with the rest of the codebase where actual API endpoint paths include the /v1 or /v2 version prefix.

This fix the compatibility for the Cloud Shell

Added the /v1 version prefix to two instance API endpoint paths in
internal/services/browser/api.go that were missing it:

- deleteInstance function (line 1592)
- cleanupCreatedResources function (line 2131)

This ensures consistency with the rest of the codebase where actual
API endpoint paths include the /v1 or /v2 version prefix.

Signed-off-by: Romain Beuque <556072+rbeuque74@users.noreply.github.com>
@rbeuque74 rbeuque74 requested a review from a team as a code owner April 2, 2026 20:17
@amstuta amstuta merged commit 077d44d into main Apr 3, 2026
7 checks passed
@rbeuque74 rbeuque74 deleted the me branch April 3, 2026 11:52
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.

2 participants