make admin accounts optional

devict · Feb 23, 2025 · 8ed57a0 · 8ed57a0
1 parent 4134ba3
commit 8ed57a0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/pkg/config/config.go b/pkg/config/config.go
@@ -16,8 +16,8 @@ type Config struct {
 	Email         *EmailConfig
 	Twitter       *TwitterConfig
 	SlackHook     string `envconfig:"SLACK_HOOK"`
-	AdminUser     string `envconfig:"ADMIN_USER" required:"true"`
-	AdminPassword string `envconfig:"ADMIN_PASSWORD" require:"true"`
+	AdminUser     string `envconfig:"ADMIN_USER"`
+	AdminPassword string `envconfig:"ADMIN_PASSWORD"`
 }
 
 type EmailConfig struct {

diff --git a/pkg/server/server.go b/pkg/server/server.go
@@ -82,9 +82,13 @@ func NewServer(c *ServerConfig) (http.Server, error) {
 
 	// Admin routes
 	admin := router.Group("/admin")
-	admin.Use(gin.BasicAuth(gin.Accounts{
-		c.Config.AdminUser: c.Config.AdminPassword,
-	}))
+
+	accounts := gin.Accounts{}
+	if c.Config.AdminUser != "" {
+		accounts[c.Config.AdminUser] = c.Config.AdminPassword
+	}
+
+	admin.Use(gin.BasicAuth(accounts))
 	{
 		admin.GET("", ctrl.AdminIndex)
 		admin.GET("/jobs/:id/edit", ctrl.EditJob)