Skip to content

Fixing issues with /api/v1/me #984

wants to merge 1 commit into from

2 participants

pyro2927 commented Mar 3, 2014

Before refactor, pre(self) was called as such: OAuth2ResourceController.pre(self). Matching syntax from before.

Current clients get returned 401s when trying to access /api/v1/me and passing up access token w/ bearer authorization.

@pyro2927 pyro2927 Fixing issues with /api/v1/me
Before refactor, `pre(self)` was called as such: `OAuth2ResourceController.pre(self)`.  Matching syntax from before.
kemitche commented Mar 3, 2014

While it's definitely possible I broke something during the refactoring, the proposed change won't have any effect. The function was called as OAuth2ResourceController.pre(self) previously due to how things were subclassed. Sub- and super-classes of OAuth2ResourceController also had pre methods, so it was previously necessary to explicitly call the pre method of the desired class.

After the refactor, there's only one authenticate_with_token method, so the subclassing is not a concern and the function can be called normally.

@kemitche kemitche closed this Mar 3, 2014
pyro2927 commented Mar 3, 2014

Gotchya. My understanding of python syntax is off then, guess I better freshen up!

Also, is your reddit avatar a dragon?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.