Skip to content

chore(ingress): remove unreachable code - #160

Merged
kvinwang merged 1 commit into
mainfrom
chore/ingress-dead-code
Oct 16, 2026
Merged

kvinwang merged 1 commit into
mainfrom
chore/ingress-dead-code

Conversation

@kvinwang

Copy link
Copy Markdown
Collaborator
  • sanitize_client_max_body_size, sanitize_proxy_timeout, sanitize_proxy_buffer_size, sanitize_proxy_buffers: nginx leftovers only their tests called; entrypoint.sh just warns those settings are ignored.
  • The *) branch of the final CHALLENGE_TYPE dispatch: the value is validated at the top of entrypoint.sh.

Stacked on #159 only to avoid a conflict in functions.sh with #156.

@kvinwang
kvinwang force-pushed the fix/ingress-issue-retry branch from cd0d500 to 140173f Compare October 16, 2026 01:09
@kvinwang
kvinwang force-pushed the chore/ingress-dead-code branch 2 times, most recently from 8912503 to 30a2624 Compare October 16, 2026 01:20
@kvinwang
kvinwang force-pushed the fix/ingress-issue-retry branch from 140173f to 829d9fa Compare October 16, 2026 01:20
@kvinwang
kvinwang force-pushed the chore/ingress-dead-code branch from 30a2624 to 1c7c6ae Compare October 16, 2026 01:20
@kvinwang
kvinwang force-pushed the fix/ingress-issue-retry branch from 829d9fa to ec5a04b Compare October 16, 2026 01:20
@kvinwang
kvinwang force-pushed the chore/ingress-dead-code branch from 1c7c6ae to f46c7da Compare October 16, 2026 01:23
@kvinwang
kvinwang force-pushed the fix/ingress-issue-retry branch from ec5a04b to 4f30b0a Compare October 16, 2026 01:23
@kvinwang
kvinwang force-pushed the chore/ingress-dead-code branch from f46c7da to 0694bf5 Compare October 16, 2026 01:25
@kvinwang
kvinwang force-pushed the fix/ingress-issue-retry branch from 4f30b0a to 88b936b Compare October 16, 2026 01:25
- sanitize_client_max_body_size, sanitize_proxy_timeout,
  sanitize_proxy_buffer_size and sanitize_proxy_buffers are left over
  from the nginx days; only their tests called them. entrypoint.sh just
  warns that those settings are ignored.
- The `*)` branch of the final CHALLENGE_TYPE dispatch in entrypoint.sh
  cannot run: the value is validated at the top of the script.

Signed-off-by: Kevin Wang <wy721@qq.com>
@kvinwang
kvinwang force-pushed the chore/ingress-dead-code branch from 0694bf5 to f16106b Compare October 16, 2026 01:27
@kvinwang
kvinwang force-pushed the fix/ingress-issue-retry branch from 88b936b to d651b61 Compare October 16, 2026 01:27
@kvinwang
kvinwang changed the base branch from fix/ingress-issue-retry to main October 16, 2026 01:31
@kvinwang kvinwang closed this Oct 16, 2026
@kvinwang kvinwang reopened this Oct 16, 2026
@kvinwang
kvinwang merged commit 5c949e7 into main Oct 16, 2026
6 checks passed
@kvinwang
kvinwang deleted the chore/ingress-dead-code branch October 16, 2026 01:33
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