Skip to content

feat: disable admin server - #63

Merged
peterldowns merged 1 commit into
mainfrom
pd-disable-admin
Jul 29, 2025
Merged

feat: disable admin server#63
peterldowns merged 1 commit into
mainfrom
pd-disable-admin

Conversation

@peterldowns

@peterldowns peterldowns commented Jul 29, 2025

Copy link
Copy Markdown
Owner

Following up @hslatman's comment on #62, this PR completely disables the Caddy admin server. The previous PR stopped using it as a certificate issuer's ask endpoint, but this PR actually disables the server entirely.

Testing

  • manually confirmed that localias run does NOT bind to port 2019
$ localias start
...
$ localias status
daemon running with pid 29450
$ lsof -Pn | grep -E ':2019'
  • confirmed that proxying requests still seems to work correctly
  • confirmed that cert issuance for new aliases works correctly

@peterldowns
peterldowns marked this pull request as ready for review July 29, 2025 20:15
@peterldowns
peterldowns merged commit 30eb016 into main Jul 29, 2025
3 checks passed
@hslatman

Copy link
Copy Markdown

Nice 😄

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.

2 participants