Rename healthcheck route

Author: joecorall

README.md

@@ -4,7 +4,7 @@ A lightweight Go service that combines ping health checks with power management
 
 ## Features
 
-- **HTTP Health Endpoints**: `/ping` (monitored) and `/health` (healthcheck)
+- **HTTP Health Endpoints**: `/ping` (monitored) and `/healthcheck`
 - **Activity Monitoring**: Tracks requests to `/ping` endpoint
 - **Configurable Timeouts**: Environment-controlled inactivity periods
 - **GCP Integration**: Automatic instance suspension via GCP API service
@@ -24,14 +24,13 @@ docker run -p 8808:8808 -e INACTIVITY_TIMEOUT=90 lightswitch:latest
 | `PORT`               | `8808`  | HTTP server port                         |
 | `INACTIVITY_TIMEOUT` | `90`    | Seconds of inactivity before shutdown    |
 | `CHECK_INTERVAL`     | `30`    | Seconds between activity checks          |
-| `LIBOPS_PROXY_URL`   | -       | GCP proxy URL for suspension             |
 | `LIBOPS_KEEP_ONLINE` | -       | Set to "yes" to disable auto-shutdown    |
 | `LOG_LEVEL`          | `INFO`  | Logging level (DEBUG, INFO, WARN, ERROR) |
 
 ### Endpoints
 
 - `GET /ping` - Returns "pong", activity is logged and monitored
-- `GET /health` - Returns "healthy", used for container healthchecks
+- `GET /healthcheck` - used for container healthchecks
 
 ## Integration
 
@@ -43,7 +42,7 @@ This service is designed to work in tandem with [ppb (Proxy Power Button)](https
    - Receives incoming requests for dormant instances
    - Starts the GCE instance via GCP API
    - Proxies requests to the now-running backend
-   
+
 2. **lightsout (Shutdown)**: Monitors activity and automatically shuts down instances during idle periods
    - Tracks activity via `/ping` endpoint calls
    - Monitors for configurable inactivity timeouts

main.go

@@ -363,11 +363,6 @@ func pingHandler(w http.ResponseWriter, r *http.Request) {
 func healthHandler(w http.ResponseWriter, r *http.Request) {
 	w.Header().Set("Content-Type", "text/plain")
 	w.WriteHeader(http.StatusOK)
-	if _, err := w.Write([]byte("healthy")); err != nil {
-		slog.Error("Failed to write health response", "error", err)
-		http.Error(w, "Failed to write response", http.StatusInternalServerError)
-		return
-	}
 }
 
 func main() {
@@ -384,7 +379,7 @@ func main() {
 
 	// Setup HTTP handlers
 	http.HandleFunc("/ping", pingHandler)
-	http.HandleFunc("/health", healthHandler)
+	http.HandleFunc("/healthcheck", healthHandler)
 
 	// Setup HTTP server
 	server := &http.Server{

main_test.go

@@ -221,7 +221,7 @@ func TestHealthEndpoint(t *testing.T) {
 	cleanup := setupTestEnvironment()
 	defer cleanup()
 
-	req := httptest.NewRequest("GET", "/health", nil)
+	req := httptest.NewRequest("GET", "/healthcheck", nil)
 	w := httptest.NewRecorder()
 
 	healthHandler(w, req)
@@ -230,10 +230,6 @@ func TestHealthEndpoint(t *testing.T) {
 		t.Fatalf("Expected status 200, got %d", w.Code)
 	}
 
-	if w.Body.String() != "healthy" {
-		t.Fatalf("Expected 'healthy', got '%s'", w.Body.String())
-	}
-
 	if w.Header().Get("Content-Type") != "text/plain" {
 		t.Fatalf("Expected Content-Type 'text/plain', got '%s'", w.Header().Get("Content-Type"))
 	}