[PR #402] [MERGED] Add ureq-native-tls feature flag #437

Closed
opened 2026-02-27 20:24:44 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ramsayleung/rspotify/pull/402
Author: @jirutka
Created: 4/23/2023
Status: Merged
Merged: 4/26/2023
Merged by: @ramsayleung

Base: masterHead: native-tls


📝 Commits (1)

  • 75b67e5 Add ureq-native-tls feature flag

📊 Changes

3 files changed (+3 additions, -0 deletions)

View changed files

📝 CHANGELOG.md (+1 -0)
📝 Cargo.toml (+1 -0)
📝 rspotify-http/Cargo.toml (+1 -0)

📄 Description

Description

Allow building with ureq client and native-tls (i.e. without rustls).

Motivation and Context

I’m an Alpine Linux developer and I want to build ncspot with native TLS instead of rustls.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How has this been tested?

cargo test --no-default-features --features client-ureq,ureq-native-tls

Is this change properly documented?

Please make sure you've properly documented the changes you're making.

Don't forget to add an entry to the CHANGELOG if necessary (new features, breaking changes, relevant internal improvements).


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ramsayleung/rspotify/pull/402 **Author:** [@jirutka](https://github.com/jirutka) **Created:** 4/23/2023 **Status:** ✅ Merged **Merged:** 4/26/2023 **Merged by:** [@ramsayleung](https://github.com/ramsayleung) **Base:** `master` ← **Head:** `native-tls` --- ### 📝 Commits (1) - [`75b67e5`](https://github.com/ramsayleung/rspotify/commit/75b67e59004e7b57aaab663396ac2c1fb913a5cc) Add ureq-native-tls feature flag ### 📊 Changes **3 files changed** (+3 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+1 -0) 📝 `Cargo.toml` (+1 -0) 📝 `rspotify-http/Cargo.toml` (+1 -0) </details> ### 📄 Description ## Description Allow building with ureq client and native-tls (i.e. without rustls). ## Motivation and Context I’m an Alpine Linux developer and I want to build [ncspot](https://github.com/hrkfdn/ncspot) with native TLS instead of rustls. ## Type of change Please delete options that are not relevant. - [ ] Bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] This change requires a documentation update ## How has this been tested? ```sh cargo test --no-default-features --features client-ureq,ureq-native-tls ``` ## Is this change properly documented? Please make sure you've properly documented the changes you're making. Don't forget to add an entry to the CHANGELOG if necessary (new features, breaking changes, relevant internal improvements). --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 20:24:44 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/rspotify#437
No description provided.