[PR #1225] Warning on Spotify URI RegEx fail #1221

New issue

Open

opened 2026-02-28 00:04:09 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 00:04:09 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/spotipy-dev/spotipy/pull/1225
Author: @dieser-niko
Created: 12/23/2025
Status: 🔄 Open

Base: master ← Head: feature/invalid-uri-warning

📝 Commits (1)

051e6d4 add warning when regex for uri fails

📊 Changes

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

View changed files

📝 CHANGELOG.md (+2 -0)
📝 spotipy/client.py (+1 -0)

📄 Description

Closes #1221

While implementing, I noticed that such a warning would probably be more disruptive than helpful.
Currently there are multiple methods that should accept both tracks and episodes, but the regex only checks for tracks.

Any thoughts?

_{🔄 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/spotipy-dev/spotipy/pull/1225 **Author:** [@dieser-niko](https://github.com/dieser-niko) **Created:** 12/23/2025 **Status:** 🔄 Open **Base:** `master` ← **Head:** `feature/invalid-uri-warning` --- ### 📝 Commits (1) - [`051e6d4`](https://github.com/spotipy-dev/spotipy/commit/051e6d49af3a5f222bebc689ed2f63523ce359e7) add warning when regex for uri fails ### 📊 Changes **2 files changed** (+3 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+2 -0) 📝 `spotipy/client.py` (+1 -0) </details> ### 📄 Description Closes #1221 While implementing, I noticed that such a warning would probably be more disruptive than helpful. Currently there are multiple methods that *should* accept both tracks and episodes, but the regex only checks for tracks. Any thoughts? --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>