Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TLS Support by Reverse-Proxy for Basicstation #240

Open
1 task done
bastienvty opened this issue Oct 3, 2024 · 1 comment
Open
1 task done

TLS Support by Reverse-Proxy for Basicstation #240

bastienvty opened this issue Oct 3, 2024 · 1 comment

Comments

@bastienvty
Copy link

bastienvty commented Oct 3, 2024

  • I have searched the issues of this repository and believe that this is not a duplicate.

Summary

This feature allows the basicstation to be aware that TLS support is managed by a reverse-proxy with TLS termination. This means it doesn't have to manage certificates and only starts a basic server (ws://).

What is the use-case?

I use the reverse-proxy traefik and noticed that the gateways always ask for router-info of the basicstation. It then returns an URI with the scheme configured (ws or wss). However, chirpstack-gateway-bridge is configured to return wss only if TLS certificates have been referenced in the configuration file.
With a reverse-proxy, TLS is terminated at the network edge. The gateway-bridge therefore works with the ws scheme inside but is only accessible through wss from Internet. So, it would have to return wss even if there are no TLS certificates referenced.

Implementation description

Simply add a configuration variable to notify the gateway-bridge that TLS is managed by a reverse-proxy. It then modifies the scheme accordingly, even though it only starts an unsecure server.

Can you implement this by yourself and make a pull request?

I already implemented it and will create a pull-request in the hope it gets accepted.

@ezequielfalcon
Copy link

+1 on this feature. Using reverse proxies with TLS are a must strategy for HA production environments.
Thanks @bastienvty for providing details about this feature request (and for the PR too)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants