Merge pull request #9 from ProjectLighthouseCAU/api-token-changes

Remove trusted proxy setting as it does not work with docker name resolution and we don't need it anyways

Author: NicoBiernat

config/config.go

@@ -31,8 +31,8 @@ var (
 	ApiHost     string = getString("API_HOST", "https://lighthouse.uni-kiel.de") // for CORS and Swagger UI API documentation
 	ApiBasePath string = getString("API_BASE_PATH", "/api")                      // used only for Swagger UI
 
-	ProxyHeader  string = getString("PROXY_HEADER", "X-Real-Ip") // "X-Real-Ip" behind a reverse proxy and "" for hosting without a proxy
-	TrustedProxy string = getString("TRUSTED_PROXY", "traefik")
+	ProxyHeader string = getString("PROXY_HEADER", "X-Real-Ip") // "X-Real-Ip" behind a reverse proxy and "" for hosting without a proxy
+
 	// Cross-Origin-Resource-Sharing
 	CorsAllowOrigins     string = getString("CORS_ALLOW_ORIGINS", ApiHost) // by default only allow the API host, add allowed origins by appending them separated with commas
 	CorsAllowCredentials bool   = getBool("CORS_ALLOW_CREDENTIALS", false)

setup/setup.go

@@ -28,12 +28,10 @@ func Setup() *fiber.App {
 
 	log.Println("Starting Heimdall")
 	app := fiber.New(fiber.Config{
-		AppName:                 "Heimdall",
-		CaseSensitive:           true,
-		StrictRouting:           true,
-		ProxyHeader:             config.ProxyHeader,
-		EnableTrustedProxyCheck: true,
-		TrustedProxies:          []string{config.TrustedProxy},
+		AppName:       "Heimdall",
+		CaseSensitive: true,
+		StrictRouting: true,
+		ProxyHeader:   config.ProxyHeader,
 	})
 
 	// Dependency Injection