Skip to content

Use rustls instead of OpenSSL; release 0.2.1#33

Merged
saramaebee merged 1 commit into
mainfrom
release-0.2.1
Jun 12, 2026
Merged

Use rustls instead of OpenSSL; release 0.2.1#33
saramaebee merged 1 commit into
mainfrom
release-0.2.1

Conversation

@saramaebee

Copy link
Copy Markdown
Collaborator

Summary

The v0.2.0 release build failed on x86_64-unknown-linux-musl: reqwest's default native-tls feature pulls in openssl-sys, which can't link when cross-compiling to musl (no musl OpenSSL on the runner). The gnu/macOS builds passed because they have system OpenSSL.

Switch reqwest to rustls-tls with default-features = false (keeping json/gzip/brotli/deflate/charset/http2). This removes openssl-sys from the tree entirely, so musl builds cleanly and no target depends on system OpenSSL anymore. Bumps version to 0.2.1.

Test plan

  • cargo build and the full test suite (103 + integration tests) pass.
  • openssl-sys gone from Cargo.lock; rustls present.
  • musl cross-compile proof comes from the v0.2.1 release run after tagging.

After merge

Tag v0.2.1 on main to trigger the release (the v0.2.0 tag's build failed, so nothing was published under 0.2.0).

🤖 Generated with Claude Code

The v0.2.0 release build failed on x86_64-unknown-linux-musl: reqwest's
default native-tls pulls in openssl-sys, which can't link when
cross-compiling to musl (no musl OpenSSL on the runner).

Switch reqwest to rustls-tls with default-features disabled (keeping
json/gzip/brotli/deflate/charset/http2). This drops openssl-sys
entirely, so musl builds cleanly and the other targets no longer depend
on system OpenSSL. Bump to 0.2.1.

Verified: cargo build + full test suite pass, openssl-sys gone from the
lockfile, rustls present.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@saramaebee saramaebee merged commit 03227a0 into main Jun 12, 2026
5 checks passed
@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 54db0b9f-ff5c-478f-aa6d-3405baabb3d4

📥 Commits

Reviewing files that changed from the base of the PR and between 9078519 and e1b63d5.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • Cargo.toml

Walkthrough

This PR updates the Cargo.toml configuration for the fossapi crate. The package version is bumped from 0.2.0 to 0.2.1. The reqwest HTTP client dependency is reconfigured to disable default features and explicitly enable rustls-tls alongside json, gzip, brotli, deflate, and http2 features. This configuration change allows the crate to use Rust-native TLS instead of the default OpenSSL for cross-platform compatibility.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main changes: switching to rustls from OpenSSL and releasing version 0.2.1, which directly correspond to the Cargo.toml modifications.
Description check ✅ Passed The description provides detailed context about the build failure, the fix rationale, testing approach, and next steps, all directly related to the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant