Skip to content
This repository was archived by the owner on Jul 28, 2026. It is now read-only.

server: Change default HTTP/gRPC listen address - #1481

Merged
rfratto merged 4 commits into
grafana-cold-storage:mainfrom
rfratto:new-http-port-default
Mar 14, 2022
Merged

server: Change default HTTP/gRPC listen address#1481
rfratto merged 4 commits into
grafana-cold-storage:mainfrom
rfratto:new-http-port-default

Conversation

@rfratto

@rfratto rfratto commented Mar 10, 2022

Copy link
Copy Markdown
Contributor

This commit changes the default listen addresses to be 127.0.0.1:12345
(for HTTP) and 127.0.0.1:12346 (for gRPC). This makes listening on all
interfaces opt-in rather than opt-out, avoiding accidental overexposure
of access.

Closes #852.

Additionally, the -reload-addr and -reload-port flags have been
removed as a follow up to #1476. Now that the HTTP and gRPC server are
static for the lifetime of the application, it is impossible for the
user to change their configuration file in such a way to cause it to
shut down while performing a reload. This means that the -reload-addr
and -reload-port no longer have a use and can be removed safely.

…1:12346

This commit changes the default listen addresses to be 127.0.0.1:12345
(for HTTP) and 127.0.0.1:12346 (for gRPC). This makes listening on all
interfaces opt-in rather than opt-out, avoiding accidental overexposure
of access.

Closes grafana-cold-storage#852.

Additionally, the `-reload-addr` and `-reload-port` flags have been
removed as a follow up to grafana-cold-storage#1476. Now that the HTTP and gRPC server are
static for the lifetime of the application, it is impossible for the
user to change their configuration file in such a way to cause it to
shut down while performing a reload. This means that the `-reload-addr`
and `-reload-port` no longer have a use and can be removed safely.
@rfratto
rfratto requested a review from mattdurham March 10, 2022 19:18
@rfratto

rfratto commented Mar 11, 2022

Copy link
Copy Markdown
Contributor Author

@mattdurham this is ready for review now :)

@mattdurham mattdurham left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@rfratto
rfratto merged commit 1a39df4 into grafana-cold-storage:main Mar 14, 2022
@rfratto
rfratto deleted the new-http-port-default branch March 14, 2022 15:12
@github-actions github-actions Bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Apr 2, 2024
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Default config exposes /debug/pprof publicly

2 participants