New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update api requesting issue catching #275
Conversation
catch other error codes
better error catcher
if len(words) == 1 and len(words[0]) > 33: | ||
final_text = "\n" + words[0][:30] + "..." | ||
if len(words) == 1 and len(words[0]) > 22: | ||
final_text = "\n" + words[0][:19] + "..." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
à quel ticket cette modif est-elle liée ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
" in 'Preferences/Options/Network'.", | ||
context=__class__.__name__, | ||
), | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Plusieurs choses :
- l'alerte est contraire à ce qui est indiqué dans la docstring. Déclenche t'on une erreur s'il y a un proxy sur l'OS mais dans QGIS ? ou l'inverse ? ou les deux ?
- seul HTTP est testé alors que l'API fonctionne essentiellement en HTTPS. Certes, les proxy gèrent cependant souvent les 2 protocoles sur la même URL mais pourquoi ne pas tester ?
- de plus comme indiqué sur Slack, cela ne teste que la case cochée ou pas. Or, dans QGIS, la case peut être cochée (donc test=True) mais indiquer d'utiliser le proxy OS (DefaultProxy) https://docs.qgis.org/testing/en/docs/user_manual/introduction/qgis_configuration.html#network-settings
As part of the #266 bug investigation.
In case of a problem with the response of the Isogeo API, the error code Qt and the status code http are written in the logs. The objective is to identify more different causes of error.
Bonus : close #273 by replacing
types
withdatatype