Skip to content

feat(port-ocean): derive ingestUrl from baseUrl when unset - #285

Open
haimn-support wants to merge 1 commit into
mainfrom
feat/port-ocean-derive-ingest-url
Open

feat(port-ocean): derive ingestUrl from baseUrl when unset#285
haimn-support wants to merge 1 commit into
mainfrom
feat/port-ocean-derive-ingest-url

Conversation

@haimn-support

Copy link
Copy Markdown

Previously port.ingestUrl defaulted to https://ingest.getport.io and had to be set manually for non-default regions (e.g. US). It now defaults to empty and is derived from port.baseUrl by replacing the api. host prefix with ingest. (https://api.us.getport.io -> https://ingest.us.getport.io), so a single baseUrl change is enough to point a deployment at the right region. An explicit port.ingestUrl still overrides the derivation. Bumps chart to 0.18.0.

Description

What - Add a mechanism to calculate ingestUrl by using the baseUrl
Why - To eliminate the need to add another mandatory param and avoid issues where the baseUrl and ingestUrl mismatch
How - Previously port.ingestUrl defaulted to https://ingest.getport.io and had to be set manually for non-default regions (e.g. US). It now defaults to empty and is derived from port.baseUrl by replacing the api. host prefix with ingest. (https://api.us.getport.io -> https://ingest.us.getport.io), so a single baseUrl change is enough to point a deployment at the right region. An explicit port.ingestUrl still overrides the derivation.

Type of change

Please leave one option from the following and delete the rest:

  • Bug fix (non-breaking change which fixes an issue)
  • [ X] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Non-breaking change (fix of existing functionality that will not change current behavior)
  • Documentation (added/updated documentation)

Previously port.ingestUrl defaulted to https://ingest.getport.io and had to
be set manually for non-default regions (e.g. US). It now defaults to empty
and is derived from port.baseUrl by replacing the `api.` host prefix with
`ingest.` (https://api.us.getport.io -> https://ingest.us.getport.io), so a
single baseUrl change is enough to point a deployment at the right region.
An explicit port.ingestUrl still overrides the derivation. Bumps chart to
0.18.0.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

1 participant