[PR #429] [MERGED] Enable token refreshing by default #449

New issue

Closed

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

kerem commented

2026-02-27 20:24:46 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ramsayleung/rspotify/pull/429
Author: @ramsayleung
Created: 6/26/2023
Status: ✅ Merged
Merged: 7/15/2023
Merged by: @ramsayleung

Base: master ← Head: ramsay/enable_token_refreshing_by_default

📝 Commits (5)

d990557 Enable token refreshing by default
6f218aa Update related examples.
6188183 Update README
586d039 Fix test_auth_header test.
8d60e3b Merge branch 'master' into ramsay/enable_token_refreshing_by_default

📊 Changes

5 files changed (+3 additions, -7 deletions)

View changed files

📝 CHANGELOG.md (+1 -0)
📝 examples/with_auto_reauth.rs (+0 -1)
📝 examples/with_token_callback_fn.rs (+0 -3)
📝 src/clients/mod.rs (+1 -2)
📝 src/lib.rs (+1 -1)

📄 Description

Description

Enable token refreshing by default.

Motivation and Context

https://github.com/ramsayleung/rspotify/issues/413

Dependencies

List any dependencies that are required for this change.

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 run --example with_auto_reauth --features env-file,cli,client-reqwest

Is this change properly documented?

Yes

_{🔄 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/429 **Author:** [@ramsayleung](https://github.com/ramsayleung) **Created:** 6/26/2023 **Status:** ✅ Merged **Merged:** 7/15/2023 **Merged by:** [@ramsayleung](https://github.com/ramsayleung) **Base:** `master` ← **Head:** `ramsay/enable_token_refreshing_by_default` --- ### 📝 Commits (5) - [`d990557`](https://github.com/ramsayleung/rspotify/commit/d9905578c4f19e86d814462dfdda9674d4298d4b) Enable token refreshing by default - [`6f218aa`](https://github.com/ramsayleung/rspotify/commit/6f218aa3df20cce0d9b2633e9503ee0c4a17ed8c) Update related examples. - [`6188183`](https://github.com/ramsayleung/rspotify/commit/6188183fa5c5017832736af80c71fc74f468483e) Update README - [`586d039`](https://github.com/ramsayleung/rspotify/commit/586d03975dd674a5210e698b95cc4119f8b84d43) Fix test_auth_header test. - [`8d60e3b`](https://github.com/ramsayleung/rspotify/commit/8d60e3bf24f8fcbde3654093d9908369409357fe) Merge branch 'master' into ramsay/enable_token_refreshing_by_default ### 📊 Changes **5 files changed** (+3 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+1 -0) 📝 `examples/with_auto_reauth.rs` (+0 -1) 📝 `examples/with_token_callback_fn.rs` (+0 -3) 📝 `src/clients/mod.rs` (+1 -2) 📝 `src/lib.rs` (+1 -1) </details> ### 📄 Description ## Description Enable token refreshing by default. ## Motivation and Context https://github.com/ramsayleung/rspotify/issues/413 ## Dependencies List any dependencies that are required for this change. ## 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) - [x] 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 run --example with_auto_reauth --features env-file,cli,client-reqwest` ## Is this change properly documented? Yes --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>