Skip to content

fix(chart): keep Basic Auth attached to renamed web route - #4

Open
plsgiveup wants to merge 1 commit into
zenith-network:mainfrom
plsgiveup:fix/web-auth-policy-target
Open

plsgiveup wants to merge 1 commit into
zenith-network:mainfrom
plsgiveup:fix/web-auth-policy-target

Conversation

@plsgiveup

Copy link
Copy Markdown

Summary

  • Use the web.fullname helper for the SecurityPolicy name and HTTPRoute target.
  • Place the policy in the Helm release namespace.

Why

A custom nameOverride renames the HTTPRoute while the policy still targets ipfs-operator-web, so Basic Auth does not attach to the renamed route, which includes /swarm_key.

Validation

  • Checked matching names for the default chart name and nameOverride=deneme.
  • git diff --check passed.
  • Helm and live-cluster validation were unavailable locally.

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