Skip to content

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

Open
etiennelatendresse-optable wants to merge 2 commits into
masterfrom
tweak-and-rename-geotargeting
Open

Rename geotargeting to georouting and simplify to per-region edge hosts#308
etiennelatendresse-optable wants to merge 2 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
AU: "au.edge.optable.co",
CA: "ca.edge.optable.co",
EU: "eu.edge.optable.co",
NA: "na.edge.optable.co",

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.

Should be US: "na.edge.optable.co" instead of NA: "na.edge.optable.co". We could support both though.

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