synced/coder
1
0
Fork 0
mirror of https://github.com/coder/coder.git synced 2025-07-12 00:14:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

docs: clarify access URL in install flow (#5626)

Browse Source 
* fix: TLS disabled copy

* clarify default access URL

* add docs for coder address
This commit is contained in:
Ben Potter
2023-01-18 15:23:55 -08:00
committed by GitHub
parent 0374af23b2
commit a040bcc0cf
5 changed files with 30 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cli/server.go
View File

@ -112,7 +112,7 @@ func Server(vip *viper.Viper, newAPI func(context.Context, *coderd.Options) (*co
return xerrors.Errorf("TLS address must be set if TLS is enabled")
}
if !cfg.TLS.Enable.Value && cfg.HTTPAddress.Value == "" {
return xerrors.Errorf("either HTTP or TLS must be enabled")
return xerrors.Errorf("TLS is disabled. Enable with --tls-enable or specify a HTTP address")
}
// Disable rate limits if the `--dangerous-disable-rate-limits` flag

2
cli/server_test.go
View File

@ -742,7 +742,7 @@ func TestServer(t *testing.T) {
)
err := root.ExecuteContext(ctx)
require.Error(t, err)
require.ErrorContains(t, err, "either HTTP or TLS must be enabled")
require.ErrorContains(t, err, "TLS is disabled. Enable with --tls-enable or specify a HTTP address")
})
t.Run("NoTLSAddress", func(t *testing.T) {