Take exception message from provider response in case of statusCode >= 400
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
Línea de trabajo
Comienza con src/Http/Provider/AbstractHttpProvider.php, especialmente con getParsedResponse y la ruta InvalidServerResponse alrededor de las líneas 68-87. Revisa en qué se diferencian las estructuras de respuesta específicas de cada provider y, después, define una estrategia que exponga el mensaje del provider para los códigos de estado >=400 en lugar del error genérico; estará terminado cuando ese comportamiento sea coherente con el diseño seleccionado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- PHP
- Estrellas
- 4k
- Forks
- 525
- Merge medio
- 8 min
- PR fusionados (30 d)
- 1
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de geocoder-php/Geocoder
-
Wrong LocationIQ Bounds Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
geocoder-php/Geocoder#1272 ·
-
question
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
geocoder-php/Geocoder#1267 · 2 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
geocoder-php/Geocoder#1266 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 38/100
geocoder-php/Geocoder#1244 · 2 comentarios · 2 reacciones ·
-
Add Support for IPQuery Abiertoprovider
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
geocoder-php/Geocoder#1243 ·
Todos los issues de geocoder-php/Geocoder
Issues similares
-
status/awaiting_triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
WordPress/plugin-check#1486 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
az-digital/az_quickstart#6019 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
grokability/snipe-it#19688 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100