Skip to content

Commit

Permalink
Merge pull request lightninglabs#120 from Roasbeef/profile-port-fix
Browse files Browse the repository at this point in the history
config: fix profile port arg parsing
  • Loading branch information
Roasbeef authored Nov 20, 2023
2 parents 8088b61 + c715f72 commit a9b80ca
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 7 deletions.
8 changes: 3 additions & 5 deletions aperture.go
Original file line number Diff line number Diff line change
Expand Up @@ -201,8 +201,8 @@ func (a *Aperture) Start(errChan chan error) error {
}

// Enable http profiling and validate profile port number if requested.
if a.cfg.ProfilePort != 0 {
if a.cfg.ProfilePort < 1024 || a.cfg.ProfilePort > 65535 {
if a.cfg.Profile != 0 {
if a.cfg.Profile < 1024 || a.cfg.Profile > 65535 {
return fmt.Errorf("the profile port must be between " +
"1024 and 65535")
}
Expand All @@ -212,9 +212,7 @@ func (a *Aperture) Start(errChan chan error) error {
"/debug/pprof", http.StatusSeeOther,
))

listenAddr := fmt.Sprintf(
"localhost:%d", a.cfg.ProfilePort,
)
listenAddr := fmt.Sprintf("localhost:%d", a.cfg.Profile)

log.Infof("Starting profile server at %s", listenAddr)
fmt.Println(http.ListenAndServe(listenAddr, nil))
Expand Down
4 changes: 2 additions & 2 deletions config.go
Original file line number Diff line number Diff line change
Expand Up @@ -200,8 +200,8 @@ type Config struct {
// BaseDir is a custom directory to store all aperture flies.
BaseDir string `long:"basedir" description:"Directory to place all of aperture's files in."`

// ProfilePort is the port on which the pprof profile will be served.
ProfilePort uint16 `long:"profile" description:"Enable HTTP profiling on given port -- NOTE port must be between 1024 and 65535"`
// Profile is the port on which the pprof profile will be served.
Profile uint16 `long:"profile" description:"Enable HTTP profiling on given port -- NOTE port must be between 1024 and 65535"`
}

func (c *Config) validate() error {
Expand Down

0 comments on commit a9b80ca

Please sign in to comment.