Skip to content

fix(auth): do not use SSO JSON region as Kiro API host - #276

Open
BetterAndBetterII wants to merge 1 commit into
jwadow:mainfrom
BetterAndBetterII:fix/credentials-region-api-host
Open

fix(auth): do not use SSO JSON region as Kiro API host#276
BetterAndBetterII wants to merge 1 commit into
jwadow:mainfrom
BetterAndBetterII:fix/credentials-region-api-host

Conversation

@BetterAndBetterII

Copy link
Copy Markdown

Summary

Test plan

  • uv run pytest tests/unit/test_auth_manager.py -q (129 passed)
  • New regression coverage for JSON region=eu-west-1 leaving API host at default
  • Existing SSO/ARN/priority hierarchy tests updated for the corrected contract

Credentials-file `region` is for OIDC refresh only. Driving api_host from
it sent traffic to unreachable regional endpoints (issue jwadow#81). Prefer a
verified profile ARN, KIRO_API_REGION / api_region, or the default.
@cla-bot

cla-bot Bot commented Aug 31, 2026

Copy link
Copy Markdown

Thanks for the PR! 馃帀

Before merge, we need a one-time CLA confirmation.
It confirms that you have the right to contribute this code and allow the project to use it.

Full CLA text:
https://github.com/jwadow/kiro-gateway/blob/main/CLA.md

Please reply once with:

I have read the CLA and I accept its terms

You need to write once, all further messages from me can be ignored.

@BetterAndBetterII

Copy link
Copy Markdown
Author

I have read the CLA and I accept its terms

@ankitcharolia

Copy link
Copy Markdown

@BetterAndBetterII could you give a try to this gateway: https://github.com/ankitcharolia/kiro-gateway

It works quite well with All AI harness and actively being developed. The most important thing is that it is ACP compliant

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.

BUG: Region override from credentials file causes connection failure when non-us-east-1 region is unreachable

2 participants