René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Bug in token refresh logic makes library practically unusable","articleBody":"There is a bug in the token refresh logic that results in permanent silent failure. This makes the entire `ebapi` library pointless to use because authorization tokens require frequent refreshes.\r\n\r\nCalling the `refresh()` method on a `FileTokens` instances (used by `ApiConnection`) results in a `NameError`, but is masked by a try/except block that ignores any instance of `Exception` that it catches.\r\n\r\nComment added in code snippet below ([copied from tokens.py here](https://github.com/ecobee/python_api_sdk/blob/42801b032cf67b85edd7636bbbaf456042ebd146/ebapi/tokens.py#L119-L122)) identifies where the bug is located:\r\n\r\n```python\r\n# snip\r\ntry:\r\n    self.user.loc[idx] = refresh_token(ref)  # \u003c-- refresh_token() is not defined (NameError)\r\nexcept Exception:\r\n    pass  # \u003c-- bug is completely silenced\r\n# snip\r\n```\r\n","author":{"url":"https://github.com/joxl","@type":"Person","name":"joxl"},"datePublished":"2021-07-25T04:28:54.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/2/python_api_sdk/issues/2"}