Skip to content

Commit

Permalink
feat: config supports toggling HTTP and Stats servers on/off (#594)
Browse files Browse the repository at this point in the history
  • Loading branch information
Jonatan Wallmander committed Nov 14, 2024
1 parent 682dccb commit 45494f5
Show file tree
Hide file tree
Showing 2 changed files with 65 additions and 28 deletions.
73 changes: 45 additions & 28 deletions cmd/soft/serve/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -93,34 +93,51 @@ func NewServer(ctx context.Context) (*Server, error) {
// Start starts the SSH server.
func (s *Server) Start() error {
errg, _ := errgroup.WithContext(s.ctx)
errg.Go(func() error {
s.logger.Print("Starting Git daemon", "addr", s.Config.Git.ListenAddr)
if err := s.GitDaemon.Start(); !errors.Is(err, daemon.ErrServerClosed) {
return err
}
return nil
})
errg.Go(func() error {
s.logger.Print("Starting HTTP server", "addr", s.Config.HTTP.ListenAddr)
if err := s.HTTPServer.ListenAndServe(); !errors.Is(err, http.ErrServerClosed) {
return err
}
return nil
})
errg.Go(func() error {
s.logger.Print("Starting SSH server", "addr", s.Config.SSH.ListenAddr)
if err := s.SSHServer.ListenAndServe(); !errors.Is(err, ssh.ErrServerClosed) {
return err
}
return nil
})
errg.Go(func() error {
s.logger.Print("Starting Stats server", "addr", s.Config.Stats.ListenAddr)
if err := s.StatsServer.ListenAndServe(); !errors.Is(err, http.ErrServerClosed) {
return err
}
return nil
})

// optionally start the SSH server
if s.Config.SSH.Enabled {
errg.Go(func() error {
s.logger.Print("Starting SSH server", "addr", s.Config.SSH.ListenAddr)
if err := s.SSHServer.ListenAndServe(); !errors.Is(err, ssh.ErrServerClosed) {
return err

Check failure on line 102 in cmd/soft/serve/server.go

View workflow job for this annotation

GitHub Actions / lint-soft

error returned from external package is unwrapped: sig: func (*github.com/charmbracelet/soft-serve/pkg/ssh.SSHServer).ListenAndServe() error (wrapcheck)
}
return nil
})
}

// optionally start the git daemon
if s.Config.Git.Enabled {
errg.Go(func() error {
s.logger.Print("Starting Git daemon", "addr", s.Config.Git.ListenAddr)
if err := s.GitDaemon.Start(); !errors.Is(err, daemon.ErrServerClosed) {
return err

Check failure on line 113 in cmd/soft/serve/server.go

View workflow job for this annotation

GitHub Actions / lint-soft

error returned from external package is unwrapped: sig: func (*github.com/charmbracelet/soft-serve/pkg/daemon.GitDaemon).Start() error (wrapcheck)
}
return nil
})
}

// optionally start the HTTP server
if s.Config.HTTP.Enabled {
errg.Go(func() error {
s.logger.Print("Starting HTTP server", "addr", s.Config.HTTP.ListenAddr)
if err := s.HTTPServer.ListenAndServe(); !errors.Is(err, http.ErrServerClosed) {
return err

Check failure on line 124 in cmd/soft/serve/server.go

View workflow job for this annotation

GitHub Actions / lint-soft

error returned from external package is unwrapped: sig: func (*github.com/charmbracelet/soft-serve/pkg/web.HTTPServer).ListenAndServe() error (wrapcheck)
}
return nil
})
}

// optionally start the Stats server
if s.Config.Stats.Enabled {
errg.Go(func() error {
s.logger.Print("Starting Stats server", "addr", s.Config.Stats.ListenAddr)
if err := s.StatsServer.ListenAndServe(); !errors.Is(err, http.ErrServerClosed) {
return err

Check failure on line 135 in cmd/soft/serve/server.go

View workflow job for this annotation

GitHub Actions / lint-soft

error returned from external package is unwrapped: sig: func (*github.com/charmbracelet/soft-serve/pkg/stats.StatsServer).ListenAndServe() error (wrapcheck)
}
return nil
})
}

errg.Go(func() error {
s.Cron.Start()
return nil
Expand Down
20 changes: 20 additions & 0 deletions pkg/config/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@ var binPath = "soft"

// SSHConfig is the configuration for the SSH server.
type SSHConfig struct {
// Enabled toggles the SSH server on/off
Enabled bool `env:"SSH_ENABLED" yaml:"ssh_enabled"`

// ListenAddr is the address on which the SSH server will listen.
ListenAddr string `env:"LISTEN_ADDR" yaml:"listen_addr"`

Expand All @@ -39,6 +42,9 @@ type SSHConfig struct {

// GitConfig is the Git daemon configuration for the server.
type GitConfig struct {
// Enabled toggles the Git daemon on/off
Enabled bool `env:"GIT_ENABLED" yaml:"git_enabled"`

// ListenAddr is the address on which the Git daemon will listen.
ListenAddr string `env:"LISTEN_ADDR" yaml:"listen_addr"`

Expand All @@ -57,6 +63,9 @@ type GitConfig struct {

// HTTPConfig is the HTTP configuration for the server.
type HTTPConfig struct {
// Enabled toggles the HTTP server on/off
Enabled bool `env:"HTTP_ENABLED" yaml:"http_enabled"`

// ListenAddr is the address on which the HTTP server will listen.
ListenAddr string `env:"LISTEN_ADDR" yaml:"listen_addr"`

Expand All @@ -72,6 +81,9 @@ type HTTPConfig struct {

// StatsConfig is the configuration for the stats server.
type StatsConfig struct {
// Enabled toggles the Stats server on/off
Enabled bool `env:"STATS_ENABLED" yaml:"stats_enabled"`

// ListenAddr is the address on which the stats server will listen.
ListenAddr string `env:"LISTEN_ADDR" yaml:"listen_addr"`
}
Expand Down Expand Up @@ -165,21 +177,25 @@ func (c *Config) Environ() []string {
fmt.Sprintf("SOFT_SERVE_DATA_PATH=%s", c.DataPath),
fmt.Sprintf("SOFT_SERVE_NAME=%s", c.Name),
fmt.Sprintf("SOFT_SERVE_INITIAL_ADMIN_KEYS=%s", strings.Join(c.InitialAdminKeys, "\n")),
fmt.Sprintf("SOFT_SERVE_SSH_ENABLED=%s", c.SSH.Enabled),

Check failure on line 180 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / lint

printf: fmt.Sprintf format %s has arg c.SSH.Enabled of wrong type bool (govet)

Check failure on line 180 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / test_postgres

fmt.Sprintf format %s has arg c.SSH.Enabled of wrong type bool

Check failure on line 180 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / build / build (ubuntu-latest)

fmt.Sprintf format %s has arg c.SSH.Enabled of wrong type bool

Check failure on line 180 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / build / build (macos-latest)

fmt.Sprintf format %s has arg c.SSH.Enabled of wrong type bool
fmt.Sprintf("SOFT_SERVE_SSH_LISTEN_ADDR=%s", c.SSH.ListenAddr),
fmt.Sprintf("SOFT_SERVE_SSH_PUBLIC_URL=%s", c.SSH.PublicURL),
fmt.Sprintf("SOFT_SERVE_SSH_KEY_PATH=%s", c.SSH.KeyPath),
fmt.Sprintf("SOFT_SERVE_SSH_CLIENT_KEY_PATH=%s", c.SSH.ClientKeyPath),
fmt.Sprintf("SOFT_SERVE_SSH_MAX_TIMEOUT=%d", c.SSH.MaxTimeout),
fmt.Sprintf("SOFT_SERVE_SSH_IDLE_TIMEOUT=%d", c.SSH.IdleTimeout),
fmt.Sprintf("SOFT_SERVE_GIT_ENABLED=%s", c.Git.Enabled),

Check failure on line 187 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / lint

printf: fmt.Sprintf format %s has arg c.Git.Enabled of wrong type bool (govet)

Check failure on line 187 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / test_postgres

fmt.Sprintf format %s has arg c.Git.Enabled of wrong type bool

Check failure on line 187 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / build / build (ubuntu-latest)

fmt.Sprintf format %s has arg c.Git.Enabled of wrong type bool

Check failure on line 187 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / build / build (macos-latest)

fmt.Sprintf format %s has arg c.Git.Enabled of wrong type bool
fmt.Sprintf("SOFT_SERVE_GIT_LISTEN_ADDR=%s", c.Git.ListenAddr),
fmt.Sprintf("SOFT_SERVE_GIT_PUBLIC_URL=%s", c.Git.PublicURL),
fmt.Sprintf("SOFT_SERVE_GIT_MAX_TIMEOUT=%d", c.Git.MaxTimeout),
fmt.Sprintf("SOFT_SERVE_GIT_IDLE_TIMEOUT=%d", c.Git.IdleTimeout),
fmt.Sprintf("SOFT_SERVE_GIT_MAX_CONNECTIONS=%d", c.Git.MaxConnections),
fmt.Sprintf("SOFT_SERVE_HTTP_ENABLED=%s", c.HTTP.Enabled),

Check failure on line 193 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / lint

printf: fmt.Sprintf format %s has arg c.HTTP.Enabled of wrong type bool (govet)

Check failure on line 193 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / test_postgres

fmt.Sprintf format %s has arg c.HTTP.Enabled of wrong type bool

Check failure on line 193 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / build / build (ubuntu-latest)

fmt.Sprintf format %s has arg c.HTTP.Enabled of wrong type bool

Check failure on line 193 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / build / build (macos-latest)

fmt.Sprintf format %s has arg c.HTTP.Enabled of wrong type bool
fmt.Sprintf("SOFT_SERVE_HTTP_LISTEN_ADDR=%s", c.HTTP.ListenAddr),
fmt.Sprintf("SOFT_SERVE_HTTP_TLS_KEY_PATH=%s", c.HTTP.TLSKeyPath),
fmt.Sprintf("SOFT_SERVE_HTTP_TLS_CERT_PATH=%s", c.HTTP.TLSCertPath),
fmt.Sprintf("SOFT_SERVE_HTTP_PUBLIC_URL=%s", c.HTTP.PublicURL),
fmt.Sprintf("SOFT_SERVE_STATS_ENABLED=%s", c.Stats.Enabled),

Check failure on line 198 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / lint

printf: fmt.Sprintf format %s has arg c.Stats.Enabled of wrong type bool (govet)

Check failure on line 198 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / test_postgres

fmt.Sprintf format %s has arg c.Stats.Enabled of wrong type bool

Check failure on line 198 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / build / build (ubuntu-latest)

fmt.Sprintf format %s has arg c.Stats.Enabled of wrong type bool

Check failure on line 198 in pkg/config/config.go

View workflow job for this annotation

GitHub Actions / build / build (macos-latest)

fmt.Sprintf format %s has arg c.Stats.Enabled of wrong type bool
fmt.Sprintf("SOFT_SERVE_STATS_LISTEN_ADDR=%s", c.Stats.ListenAddr),
fmt.Sprintf("SOFT_SERVE_LOG_FORMAT=%s", c.Log.Format),
fmt.Sprintf("SOFT_SERVE_LOG_TIME_FORMAT=%s", c.Log.TimeFormat),
Expand Down Expand Up @@ -318,6 +334,7 @@ func DefaultConfig() *Config {
Name: "Soft Serve",
DataPath: DefaultDataPath(),
SSH: SSHConfig{
Enabled: true,
ListenAddr: ":23231",
PublicURL: "ssh://localhost:23231",
KeyPath: filepath.Join("ssh", "soft_serve_host_ed25519"),
Expand All @@ -326,17 +343,20 @@ func DefaultConfig() *Config {
IdleTimeout: 10 * 60, // 10 minutes
},
Git: GitConfig{
Enabled: true,
ListenAddr: ":9418",
PublicURL: "git://localhost",
MaxTimeout: 0,
IdleTimeout: 3,
MaxConnections: 32,
},
HTTP: HTTPConfig{
Enabled: true,
ListenAddr: ":23232",
PublicURL: "http://localhost:23232",
},
Stats: StatsConfig{
Enabled: true,
ListenAddr: "localhost:23233",
},
Log: LogConfig{
Expand Down

0 comments on commit 45494f5

Please sign in to comment.