geocoder-php / geocoder-php/Geocoder
Take exception message from provider response in case of statusCode >= 400
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- PHP
- Estrellas
- 4k
- Forks
- 525
- Merge medio
- 8 min
- PR fusionados (30 d)
- 1
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.
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.
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.
Evaluación
- Stack tecnológico
- php
- Área
- backend
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 25/100