Alex-AK / Alex-AK/discord-voice-notifications

Add security to HTTP server (e.g. health endpoint)

Aperta
#2 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
enhancement good first issue help wanted
Lingua principale
TypeScript
Stelle
2
Fork
0
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

The project currently exposes an HTTP server with a health check endpoint. While this works fine for local or private deployments (e.g. running on a home server with no public ports), it would be insecure if deployed on a publicly accessible network.

### Problem

* The HTTP server does **not** implement any form of authentication, IP restriction, or rate limiting.
* If the server is ever run with an open port (e.g. on a VPS, cloud instance, or container), it would expose potentially sensitive endpoints or metadata to the internet.

### Goals

* Protect the health endpoint and any other exposed HTTP routes
* Ensure the server fails safe in public deployments
* Keep the local dev experience simple

### Possible Solutions

* Require a token or secret header to access the health endpoint
* Allow configuration of allowed IPs (e.g. `127.0.0.1` by default)
* Consider rate limiting or request throttling

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.