use rustls in reqwest dep

[kevinheavey]

This should remove the need for libssl, or at least remove two things that require it

[vercel]

@kevinheavey is attempting to deploy a commit to the coral-xyz Team on Vercel.

A member of the Team first needs to authorize it.

[Henry-E]

Great, so these changes just narrow down the features that anchor uses? Can you say more about what rustls is. Is this worthy of a changelog?

[kevinheavey]

The default native-tls feature of reqwest relies on OpenSSL being installed, which is not something Cargo can take care of for you (hence why the Anchor docs tell you to install a bunch of Linux packages). rustls not the default because it results in bigger binaries and is less mature.

Solana crates like cargo-build-bpf already use the rustls feature though so I'm not worried about those disadvantages.

A changelog entry would be good so people can take a step out of their workflows if needed

[Henry-E]

Ok, well if it's good enough for cargo-build-bpf, then it's probably good enough for us.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated
anchor-docs	⬜️ Ignored (Inspect)			Feb 6, 2023 at 6:23PM (UTC)

[Henry-E]

Merge attempt failed
Changes must be made through a pull request.

Getting this error when trying to squash and merge. Very odd

[Henry-E]

Github keeps erroring when trying to merge this PR, so just going to remake it in a new PR and merge that instead.