vertica / VerticaPy

VerticaPy is a Python library that exposes sci-kit like functionality to conduct data science projects on data stored in Vertica, thus taking advantage Vertica’s speed and built-in analytics and machine learning capabilities.
https://www.vertica.com/python/
Apache License 2.0
219 stars 45 forks source link

Token refresh bugfix #1203

Closed DMickens closed 6 months ago

DMickens commented 6 months ago

There was an issue where all Connection errors were treated as if they were caused by token refresh, prompting users for new information. This bleeds together cases where there may be an issue with the oauth token url causing the connection error, or any other reason which may be a bad host, port, connection refused, connection unexpectedly closed, connection reset, etc.

Now we do not prompt the user again during connection errors, but instead provide some context explaining that if they are using token refresh, the connection error could have been caused by invalid or malformed token url.

Additionally fixed initial oversight of using get_access_token_using_refresh_token() directly instead of do_token_refresh() which does some validation before calling get_access_token_using_refresh_token() itself.