Skip to content

fix(loc): authenticate google translate requests with IMS token - #628

Merged
kptdobe merged 2 commits into
mainfrom
translate-ims-auth
Aug 4, 2026
Merged

fix(loc): authenticate google translate requests with IMS token#628
kptdobe merged 2 commits into
mainfrom
translate-ims-auth

Conversation

@kptdobe

@kptdobe kptdobe commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • translate.da.live is moving to /translate/<service>/<org>/<site> routes that require a valid IMS bearer token (adobe/da-translate#5). Legacy /google and /smartling remain for now but are deprecated server-side.
  • Moves the google loc connector off the anonymous POST /google call onto ${DA_TRANSLATE}/translate/google/<org>/<site> via daFetch, which attaches the IMS bearer token.
  • Adds translate.da.live to ALLOWED_TOKEN in nx2/utils/utils.js so daFetch knows to attach auth for that origin.
  • Smartling connector is unaffected — it talks directly to Smartling's own API, never through translate.da.live.

Test plan

  • npm run lint
  • npm test (1115 passing, includes new test/loc/connectors/google/index.test.js)

translate.da.live now requires /translate/<service>/<org>/<site>
routes with a valid IMS bearer token (adobe/da-translate#5). Move the
google connector off the anonymous /google endpoint onto the new
authenticated route via daFetch.
@aem-code-sync

aem-code-sync Bot commented Jul 30, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

@kptdobe
kptdobe merged commit f67db92 into main Aug 4, 2026
2 checks passed
@kptdobe
kptdobe deleted the translate-ims-auth branch August 4, 2026 09:19
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