clockworklabs
Add support for Gzip compression on server
Description of Changes
Removed the brotli dependency and replaced it with the flate2 library already being included.
API and ABI breaking changes
Changes the communications between the STDB server and its SDK clients, but opaque to users.
Expected complexity level and risk
- Straightforward change. Needs matching updates for the client repos.
Testing
Running the sdk test suite
Note, I updated the PR title to reflect the decision to support Gzip in addition to Brotli compression.
Adding the api-break label because it seems like this requires SDK changes to provide the choice of compression? Not sure though, I didn't look to closely. Feel free to remove.
Nevermind, I read a bit more and it looks like we gracefully default to brotli.
@lcodes this looks like it's ready although the sdk tests appear to be failing. Also if it's not too difficult, it would be really nice to add a test exercising gzip specifically.
Had issues with getting sdk tests to pass on this branch, moved the impl back in parts to a new branch until tests pass again; moving the PR there https://github.com/clockworklabs/SpacetimeDB/pull/1642