Skip to content

chore(docs): introduce stac-auth-proxy ROOT_PATH_SKIP_PREFIXES.#587

Merged
pantierra merged 1 commit into
mainfrom
fix/ROOT_PATH_SKIP_PREFIXES
Jul 24, 2026
Merged

chore(docs): introduce stac-auth-proxy ROOT_PATH_SKIP_PREFIXES.#587
pantierra merged 1 commit into
mainfrom
fix/ROOT_PATH_SKIP_PREFIXES

Conversation

@pantierra

@pantierra pantierra commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Related #568

Let's use and document the upstream changes: developmentseed/stac-auth-proxy#189

@claude

claude Bot commented Jul 24, 2026

Copy link
Copy Markdown

Claude finished @pantierra's task in 1m 11s —— View job


Review

✅ No blocking issues — safe to merge.

  • charts/eoapi/values.yaml:488 bumps image.tag from v1.1.0v1.2.0, which now matches the stac-auth-proxy dependency version already pinned in charts/eoapi/Chart.yaml:60 (previously out of sync — good fix).
  • charts/eoapi/values.yaml:520 adds ROOT_PATH_SKIP_PREFIXES so /raster, /vector, /browser, /multidim links aren't incorrectly rewritten with the /stac prefix.
  • charts/eoapi/profiles/experimental.yaml drops the per-profile image.tag pin (now inherits the corrected default) and removes the startupProbe/livenessProbe/readinessProbe overrides that pointed at /stac/healthz. Per docs/stac-auth-proxy.md:96, the proxy's actual health endpoint is /healthz at root (ROOT_PATH only affects link rewriting), so these overrides looked like a workaround for older-version behavior that's no longer needed with v1.2.0 — removing them lets the subchart's own correct default probes apply.
  • Docs update in docs/stac-auth-proxy.md clearly explains the new setting and its default.

No correctness, security, or best-practice issues found in this diff.


💰 Estimated review cost: $0.47 · 1m10s · 17 turns

@pantierra
pantierra requested a review from lhoupert July 24, 2026 00:08
@pantierra
pantierra merged commit fe63966 into main Jul 24, 2026
5 checks passed
@pantierra
pantierra deleted the fix/ROOT_PATH_SKIP_PREFIXES branch July 24, 2026 00:35
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.

1 participant