From c55d84b5d2f926eb87208fceff3867b8713b815b Mon Sep 17 00:00:00 2001 From: Mark Sagi-Kazar Date: Thu, 14 Jan 2021 15:50:19 +0100 Subject: [PATCH] feat: add flags for bind address config options Signed-off-by: Mark Sagi-Kazar --- cmd/dex/serve.go | 38 +++++++++++++++++++++++++++++++++++++- 1 file changed, 37 insertions(+), 1 deletion(-) diff --git a/cmd/dex/serve.go b/cmd/dex/serve.go index 858281af..634e2606 100644 --- a/cmd/dex/serve.go +++ b/cmd/dex/serve.go @@ -30,13 +30,20 @@ import ( ) type serveOptions struct { + // Config file path config string + + // Flags + webHTTPAddr string + webHTTPSAddr string + telemetryAddr string + grpcAddr string } func commandServe() *cobra.Command { options := serveOptions{} - return &cobra.Command{ + cmd := &cobra.Command{ Use: "serve [flags] [config file]", Short: "Launch Dex", Example: "dex serve config.yaml", @@ -50,6 +57,15 @@ func commandServe() *cobra.Command { return runServe(options) }, } + + flags := cmd.Flags() + + flags.StringVar(&options.webHTTPAddr, "web-http-addr", "", "Web HTTP address") + flags.StringVar(&options.webHTTPSAddr, "web-https-addr", "", "Web HTTPS address") + flags.StringVar(&options.telemetryAddr, "telemetry-addr", "", "Telemetry address") + flags.StringVar(&options.grpcAddr, "grpc-addr", "", "gRPC API address") + + return cmd } func runServe(options serveOptions) error { @@ -64,6 +80,8 @@ func runServe(options serveOptions) error { return fmt.Errorf("error parse config file %s: %v", configFile, err) } + applyConfigOverrides(options, &c) + logger, err := newLogger(c.Logger.Level, c.Logger.Format) if err != nil { return fmt.Errorf("invalid config: %v", err) @@ -383,3 +401,21 @@ func newLogger(level string, format string) (log.Logger, error) { Level: logLevel, }, nil } + +func applyConfigOverrides(options serveOptions, config *Config) { + if options.webHTTPAddr != "" { + config.Web.HTTP = options.webHTTPAddr + } + + if options.webHTTPSAddr != "" { + config.Web.HTTPS = options.webHTTPSAddr + } + + if options.telemetryAddr != "" { + config.Telemetry.HTTP = options.telemetryAddr + } + + if options.grpcAddr != "" { + config.GRPC.Addr = options.grpcAddr + } +}