Skip to content

Commit

Permalink
cmd/serve: make http port a flag
Browse files Browse the repository at this point in the history
It has been made a string instead of an integer to allow for specifying
a name from /etc/services etc.
  • Loading branch information
dotwaffle committed Apr 7, 2020
1 parent 4e0d262 commit 8835cfb
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion cmd/serve.go
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@ var serveCmd = &cobra.Command{
fmt.Fprintf(w, export)
}
http.HandleFunc("/export.json", httpExport)
if err := http.ListenAndServe(":8080", nil); err != nil {
if err := http.ListenAndServe(":"+viper.GetString("port"), nil); err != nil {
log.Fatal().Err(err).Msg("http.ListenAndServe()")
}
},
Expand All @@ -189,6 +189,12 @@ func init() {
log.Fatal().Err(err).Msg("viper.BindPFlag(): rpki")
}

// http port
serveCmd.Flags().StringP("port", "p", "8080", "tcp port for http server")
if err := viper.BindPFlag("port", serveCmd.Flags().Lookup("port")); err != nil {
log.Fatal().Err(err).Msg("viper.BindPFlag(): port")
}

// refresh interval
serveCmd.Flags().DurationP("refresh", "R", time.Hour, "interval between refreshing external data")
if err := viper.BindPFlag("refresh", serveCmd.Flags().Lookup("refresh")); err != nil {
Expand Down

0 comments on commit 8835cfb

Please sign in to comment.