Take exception message from provider response in case of statusCode >= 400
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 25/100
Rechercherichtung
Beginne mit src/Http/Provider/AbstractHttpProvider.php, insbesondere mit getParsedResponse und dem InvalidServerResponse-Pfad rund um die Zeilen 68-87. Überprüfe, wie sich die providerspezifischen Antwortstrukturen unterscheiden, und definiere anschließend eine Strategie, die für Statuscodes >=400 die Nachricht des Providers statt des generischen Fehlers offenlegt; abgeschlossen ist die Aufgabe, wenn dieses Verhalten mit dem ausgewählten Design konsistent ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Hey guys.
So I as a developer would like to see message from provider response rather than generic The geocoder server returned an invalid response (%d) for query "%s". We could not parse it. message.
We faced with an issue which is pretty hard to debug on dockerized application. It would be much easier to understand what went wrong if we get message from particular provider response in case InvalidServerResponse is thrown on line: https://github.com/geocoder-php/Geocoder/blob/master/src/Http/Provider/AbstractHttpProvider.php#L78
The main issue that each provider has their own response structure in case of bad request occurs. I don't see a good solution here so far and I have only 2 thoughts so far:
getParsedResponsemethod need to be an abstract so each provider will implement their own mechanism of catching errors.
https://github.com/geocoder-php/Geocoder/blob/master/src/Http/Provider/AbstractHttpProvider.php#L68-L87- We could add another abstract method like
abstract protected function throwClientException(string $message)method inside of anAbstractHttpProviderclass. And each provider have to implement it.
I suppose it is kind of feature request.
- Vorherrschende Sprache
- PHP
- Sterne
- 4k
- Forks
- 527
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus geocoder-php/Geocoder
-
question
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
geocoder-php/Geocoder#1267 · 2 Kommentare ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
geocoder-php/Geocoder#1266 ·
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 38/100
geocoder-php/Geocoder#1244 · 2 Kommentare · 2 Reaktionen ·
-
Add Support for IPQueryOffenprovider
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 30/100
geocoder-php/Geocoder#1243 ·
-
Mapbox Geocoding v6Evtl. vergeben @davidbarratt hat das vor 42 Tagen übernommen. Offenprovider
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
geocoder-php/Geocoder#1237 · 1 Kommentar · 1 Reaktion ·
Alle Issues in geocoder-php/Geocoder
Ähnliche Issues
-
sync-en
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
sync-en
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 4 Tagen
-
Перевод устарел
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
Combination form: image thumbnails collapse to 0×0 when a stylesheet sets `img { max-width: 100% }`Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
PrestaShop/PrestaShop#43200 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
roadrunner-php/cli#69 ·
Maintainer antworten meist innerhalb von 1 Tag