Skip to content

internal: make domain-registration available for test env#5245

Merged
battermann merged 3 commits into
developfrom
battermann/enable-basic-auth-for-domain-registration-on-test-envs
May 29, 2026
Merged

internal: make domain-registration available for test env#5245
battermann merged 3 commits into
developfrom
battermann/enable-basic-auth-for-domain-registration-on-test-envs

Conversation

@battermann
Copy link
Copy Markdown
Contributor

@battermann battermann commented May 28, 2026

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@battermann battermann marked this pull request as ready for review May 28, 2026 08:42
@battermann battermann requested review from a team as code owners May 28, 2026 08:42
@battermann battermann requested a review from Copilot May 28, 2026 08:42
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label May 28, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR makes internal domain-registration endpoints reachable through nginz for testing, protected by basic auth, and adds an internal changelog entry.

Changes:

  • Adds nginz routing entries for /i/domain-registration/... lock, unlock, preauthorize, unauthorize, and base GET/PUT/DELETE routes.
  • Marks the routes as unversioned, zauth-disabled, and basic-auth protected.
  • Adds a changelog entry for the internal testing exposure.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
charts/nginz/values.yaml Adds nginz path configuration for internal domain-registration routes.
changelog.d/5-internal/basic-auth-for-test-env Documents the internal API availability for testing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread charts/nginz/values.yaml Outdated
Comment thread charts/nginz/values.yaml Outdated
Comment thread charts/nginz/values.yaml Outdated
Comment thread charts/nginz/values.yaml Outdated
Comment thread charts/nginz/values.yaml Outdated
@battermann battermann force-pushed the battermann/enable-basic-auth-for-domain-registration-on-test-envs branch from 11e021b to 81f3556 Compare May 28, 2026 08:48
@battermann battermann requested a review from Copilot May 28, 2026 08:48
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@battermann battermann merged commit 42e6eb0 into develop May 29, 2026
11 checks passed
@battermann battermann deleted the battermann/enable-basic-auth-for-domain-registration-on-test-envs branch May 29, 2026 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants