brave / brave/go-sync

TLS support

Open
#347 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
367
Forks
83
Avg merge
3d 8h
Merged PRs (30d)
4

Description

The experimental flag that allows to override the go-sync server from the UI was merged to Nightly, but with the caveat of enforcing HTTPS usage for most deployment scenarios: https://github.com/brave/brave-core/pull/28463

I assume you use some reverse-proxy in production with go-sync, so it would be useful to at least mention that in the README that TLS is basically required or, ideally, add optional caddy/nginx setup to `docker-compose.yaml` for home-brew TLS with Lets Encrypt.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the README and docker-compose.yaml, then review the linked brave-core pull request for the HTTPS requirement. Determine whether the contribution will document the TLS requirement or add an optional Caddy/nginx setup, and consider the work done when the chosen deployment guidance is present and accurate.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker-compose, go, nginx
Domain
devops, documentation, infrastructure
Issue type
Documentation
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.