Skip to content

Build problem type/instance URLs from forwarded host, proto and prefix - #129

Merged
juztas merged 1 commit into
mainfrom
fix_ext_url
Oct 6, 2026
Merged

juztas merged 1 commit into
mainfrom
fix_ext_url

Conversation

@juztas

@juztas juztas commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@juztas

juztas commented Sep 30, 2026

Copy link
Copy Markdown
Contributor Author

@jmacauley @pmrich @gabor-lbl Can we get this merged?

This allows to set the correct url in returned output/problem, something like https://rig.staging.american....

@juztas

juztas commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

Without fix, return via AmSC RIG is:

{
  "type": "https://api.alcf.anl.gov/problems/generic-error",
  "status": 500,
  "title": "Error",
  "detail": "Error retrieving database objects: [Errno 111] Connection refused",
  "instance": "https://facility-api-vmw-01.cels.anl.gov/api/v1/facility?modified_since=2026-02-21T12%253A00%253A00Z"
}

With a fix:

{
  "type": "https://rig.infra.dev.american-science-cloud.org/rig/external/esnet-west/problems/unauthorized",
  "status": 401,
  "title": "Unauthorized",
  "detail": "AmSC authentication failed: No local mapping for AmSC project 'amsc-admins-test-only'. || Facility Specific authentication failed: 403: Token not accepted by any provider",
  "instance": "https://rig.infra.dev.american-science-cloud.org/rig/external/esnet-west/api/v2/account/projects"
}

@bcote-anl

Copy link
Copy Markdown
Contributor

This looks nice. I'm ok if you want to merge. Thanks!

@gabor-lbl

Copy link
Copy Markdown
Contributor

Fine with me, thanks!

@juztas
juztas merged commit 7867e74 into main Oct 6, 2026
1 check passed
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