Replies: 1 comment
-
|
Hello @bcspragu ! 'just performed the very same migration as you, and I second your
(syncstorage-rs v0.22.2 / Fennec v148) Bye ! 👋 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Hi there!
I just replaced my old self-hosted sync server (the Python one) with this new one, backed by Postgres. I ran into a few issues that I got sorted out, and just documenting them here in case anyone else hits these:
X-Forwarded-Proto: httpsin my reverse proxy - I don't actually know if this is required or not, but desktop Firefox was failing to sync with 404 errors IIRC, and after I added the header, everything was syncing successfully. In Caddy, the config looks something like this:<your sync server> { reverse_proxy <local sync server address> { header_up X-Forwarded-Proto "https" } }/tokenon mobile Firefox - Even after I got desktop working, Android Firefox Nightly (version ~150.0) was failing to sync. Runningadb logcat, I was able to see it was getting a 404 trying to hit/token/1.0/sync/1.5. I had definitely set the URL correctly following the official instructions, so I don't know if this was a caching bug or something else going on, but I restarted Firefox and the phone several times, triple checking that I had configured the URL to<sync server URI>/1.0/sync/1.5, so I decided to "fix it in post" with my reverse proxy, which looked something like this in Caddy:<your sync server> { handle_path /token/* { reverse_proxy <local sync server address> { ... } } ... }Which basically says "strip the
/token/prefix from the URL and forward it to the backend. So<host>/token/testwould go to the backend with the path re-written to/test. Once I deployed this change, mobile sync started working immediately (as noted by both server and client logs, as well as manually checking Postgres)I don't know if there are one or more bugs here, or if this is user error on my part, but it works now so that's fun!
Beta Was this translation helpful? Give feedback.
All reactions