Skip to content

fix(config): replace external-url into studio api#4803

Open
9bany wants to merge 1 commit intosupabase:developfrom
9bany:replace-external-url-into-studio-api
Open

fix(config): replace external-url into studio api#4803
9bany wants to merge 1 commit intosupabase:developfrom
9bany:replace-external-url-into-studio-api

Conversation

@9bany
Copy link

@9bany 9bany commented Feb 4, 2026

What kind of change does this PR introduce?

Fix TODO noted in code based

What is the current behavior?

This PR remove external-url in config, remove depended of studio.api on the external-url

What is the new behavior?

The studio.api variable is set directly in the Load configuration.

Additional context

None

@9bany 9bany requested a review from a team as a code owner February 4, 2026 09:24
@9bany
Copy link
Author

9bany commented Feb 4, 2026

Could you please check this PR @sweatybridge ?

Copy link
Contributor

@sweatybridge sweatybridge left a comment

Choose a reason for hiding this comment

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

The TODO is actually meant to keep Api.ExternalUrl and delete Studio.ApiUrl

@9bany 9bany force-pushed the replace-external-url-into-studio-api branch from d6a1803 to 7852108 Compare February 24, 2026 03:42
@coderabbitai
Copy link

coderabbitai bot commented Feb 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Cache: Disabled due to Reviews > Disable Cache setting

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 1d5b8ed and 7852108.

📒 Files selected for processing (5)
  • internal/start/start.go
  • pkg/config/api.go
  • pkg/config/config.go
  • pkg/config/templates/config.toml
  • pkg/config/testdata/config.toml
💤 Files with no reviewable changes (2)
  • pkg/config/testdata/config.toml
  • pkg/config/config.go

📝 Walkthrough

Summary by CodeRabbit

  • Refactor
    • External URL configuration has been moved from studio-specific to a global setting, simplifying configuration management and allowing centralized control.

Walkthrough

The changes refactor the API external URL configuration from a studio-specific field to a global API configuration field. The Studio.ApiUrl field is removed from the config struct, and its validation logic is eliminated. The Api.ExternalUrl field is newly exposed in TOML and JSON serialization with the tag toml:"external_url". Configuration templates are updated to include a global external_url option in place of the studio-specific api_url. The startup code is updated to reference utils.Config.Api.ExternalUrl instead of utils.Config.Studio.ApiUrl for the Studio container's public URL.


Comment @coderabbitai help to get the list of available commands and usage tips.

@9bany
Copy link
Author

9bany commented Feb 24, 2026

Sorry for the late response,
Please check again, thanks a lot @sweatybridge

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