Skip to content

Rename geotargeting to georouting and simplify to per-region edge hosts#308

Merged
etiennelatendresse-optable merged 3 commits into
masterfrom
tweak-and-rename-geotargeting
Jul 24, 2026
Merged

Rename geotargeting to georouting and simplify to per-region edge hosts#308
etiennelatendresse-optable merged 3 commits into
masterfrom
tweak-and-rename-geotargeting

Conversation

@etiennelatendresse-optable

@etiennelatendresse-optable etiennelatendresse-optable commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Follow-up to the merged geotargeting addon

What changed

  • Rename geotargetingto georouting
  • All regions use their edge host: US -> na.edge, CA -> ca.edge, EU -> eu.edge, AU -> au.edge
  • Dropped the -auth handling and the tuple GeoMapEntry moving toward one tenant/DCN per customer per region. GeoMap is now a plain Record<string, string> (geo -> host).
  • getGeoRouting(geo, geoMap?) returns the host string or null, instead of getGeoConfig(nodeName, geo) returning { host, node }. The caller supplies node/site, this addon only resolves the host.

Open decision regarding US/CA on the same edge

The simplification now only returns the host. That cleanly handles the common case (each region has its own edge host), but if a customer runs separate US and CA DCNs on the same edge (e.g. both on na.edge), the host alone can't differentiate them, the caller must pick the DCN via node/site, or pass a custom GeoMap. This is documented in the README and the module header.

How to test

  • Tested
  • Docs / README updated

@etiennelatendresse-optable etiennelatendresse-optable changed the title Rename geotargeting → georouting and simplify to per-region edge hosts Rename geotargeting to georouting and simplify to per-region edge hosts Jul 21, 2026
@mosherBT

Copy link
Copy Markdown
Contributor

Don't merge until @jplaroche reviews

Comment thread lib/addons/geo-routing.ts

@jplaroche jplaroche left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@etiennelatendresse-optable
etiennelatendresse-optable merged commit 50b77b7 into master Jul 24, 2026
7 checks passed
@etiennelatendresse-optable
etiennelatendresse-optable deleted the tweak-and-rename-geotargeting branch July 24, 2026 18:39
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