Skip to content

Add feature flags to spacetimedb-sdk to control TLS#3710

Closed
bfops wants to merge 5 commits intomasterfrom
bfops/optional-flag
Closed

Add feature flags to spacetimedb-sdk to control TLS#3710
bfops wants to merge 5 commits intomasterfrom
bfops/optional-flag

Conversation

@bfops
Copy link
Collaborator

@bfops bfops commented Nov 20, 2025

Description of Changes

Fixes #2878.

For some reason, even though I didn't change the default features, the Cargo.lock still got a bunch of new stuff. ChatGPT thinks it's because of something something unification with the dependencies that reqwest uses.

API and ABI breaking changes

None. Default features have not changed.

Expected complexity level and risk

1

Testing

  • No more openssl dependency when using the features!
    Default:
$ cargo tree -pspacetimedb-sdk -i openssl -e=no-dev
openssl v0.10.74
└── native-tls v0.2.14
    ├── tokio-native-tls v0.3.1
    │   └── tokio-tungstenite v0.27.0
    │       └── spacetimedb-sdk v1.8.0 (/home/lead/work/clockwork-localhd/SpacetimeDBPrivate/public/sdks/rust)
    ├── tokio-tungstenite v0.27.0 (*)
    └── tungstenite v0.27.0
        └── tokio-tungstenite v0.27.0 (*)

With the feature flag:

$ cargo tree -pspacetimedb-sdk -i openssl -e=no-dev --features tls-rustls-native --no-default-features
warning: nothing to print.

To find dependencies that require specific target platforms, try to use option `--target all` first, and then narrow your search scope accordingly.

@bfops bfops requested review from gefjon and jdetter November 20, 2025 18:17
@bfops bfops removed the request for review from jdetter November 20, 2025 18:23
@Soren8
Copy link

Soren8 commented Jan 31, 2026

Any possibility we could get this merged? Need this for Rust SDK wasm build targets. Thanks.

@bfops
Copy link
Collaborator Author

bfops commented Feb 2, 2026

Closing in favor of #4089

@bfops bfops closed this Feb 2, 2026
@bfops bfops deleted the bfops/optional-flag branch February 6, 2026 03:23
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

Successfully merging this pull request may close these issues.

Optional **rustls** TLS backend for spacetimedb-sdk

2 participants