bacalhau-project / bacalhau

Compute over Data framework for public, transparent, and optionally verifiable computation
https://docs.bacalhau.org
Apache License 2.0
675 stars 88 forks source link

bacalhau WebUI does not work with TLS #3774

Closed frrist closed 3 months ago

frrist commented 5 months ago

The WebUI needs to accept a TLS config at a minimum, currently it does not. https://github.com/bacalhau-project/bacalhau/blob/860d58c19467903d7616a74cc5f1988e100b8050/cmd/cli/serve/serve.go#L312-L330

In order to enable the work done in https://github.com/bacalhau-project/bacalhau/pull/3711 this issue needs to be addressed as there is no point sharing a token with the WebUI over an unsecured connection.

frrist commented 5 months ago

I believe the simple/correct fix here is to re-use the API server config, or at least a portion of it, to instantiate the WebUI.