mirror of
https://github.com/jwilsson/spotify-web-api-php.git
synced 2026-04-27 07:55:49 +03:00
[GH-ISSUE #121] request.php broken on line 194 #73
Labels
No labels
bug
docs
enhancement
enhancement
enhancement
feedback wanted
good first issue
help wanted
help wanted
help wanted
invalid
pull-request
question
question
upstream
upstream
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/spotify-web-api-php#73
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @outspokenmagazine on GitHub (Apr 24, 2018).
Original GitHub issue: https://github.com/jwilsson/spotify-web-api-php/issues/121
original line 194
$url .= '/?' . $parameters;
was ending up every search request in a 404 response for me.
I removed the / to:
$url .= '?' . $parameters;
now all works
I did that by following the spotify web api reference call
@jwilsson commented on GitHub (Apr 24, 2018):
Hmm, strange.
When I try, everything works. I tried with cURL as well, both with and without a slash before the question mark.
Could you post the code you're using? And which version of this library are you using?
@outspokenmagazine commented on GitHub (Apr 25, 2018):
Hey, you are right. I tested with straight curl and it does not make any difference there.
Thats strange indeed.
I downloaded your package via composer and I am currently using 2.2.0 (in regards to what the changelog says)
and here is what I am doing in my code in detail:
Maybe I just did a rookies mistake and misunderstand how to use it. The last line was ending up in the 404 response from spotify, which I then was able to track down what I wrote in the initial incident.
@jwilsson commented on GitHub (Apr 25, 2018):
I tested your code and that works just fine. I'm guessing you've tested again since yesterday? Cause it seems Spotify had some issues with the search endpoint.
@outspokenmagazine commented on GitHub (Apr 26, 2018):
I did just retest the old code, and indeed today it is not giving me any 404 responses. So you are right, it must have been an issue in regards to Spotify endpoints problems.
Thank you for looking into it anyway and of course many thanks for the package in general!