Skip to content
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

discogs: support simple auth. #2447

Merged
merged 1 commit into from
Feb 21, 2017
Merged

Conversation

robbat2
Copy link
Contributor

@robbat2 robbat2 commented Feb 21, 2017

The official OAuth authentication seems to have broken, so allow usage
of simple configuration instead.

See-Also: discogs/discogs_client#78
Signed-off-by: Robin H. Johnson robbat2@gentoo.org

@sampsyo
Copy link
Member

sampsyo commented Feb 21, 2017

Oh no, that's bad news. Any chance you could add some documentation in discogs.rst to explain what's going on (and how to get a user token)?

The official OAuth authentication seems to have broken, so allow usage
of simple configuration instead.

See-Also: discogs/discogs_client#78
Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>
@robbat2
Copy link
Contributor Author

robbat2 commented Feb 21, 2017

Now includes documentation.

@sampsyo
Copy link
Member

sampsyo commented Feb 21, 2017

OK, thanks! This looks good, and the documentation is helpful too.

For what it's worth, OAuth-based authentication is still working fine on my machine. I'm not sure what's going wrong on your setup, but it can't hurt to have an alternative authentication mechanism.

@sampsyo sampsyo merged commit 3e4c9b8 into beetbox:master Feb 21, 2017
sampsyo added a commit that referenced this pull request Feb 21, 2017
sampsyo added a commit that referenced this pull request Feb 21, 2017
sampsyo added a commit that referenced this pull request Feb 21, 2017
@robbat2
Copy link
Contributor Author

robbat2 commented Feb 21, 2017

What version of requests and oauthlib do you have locally? It felt like something was broken there when I was debugging it, that a bunch of keys weren't being included in the POST body anymore.

@sampsyo
Copy link
Member

sampsyo commented Feb 21, 2017

Looks like that was with discogs-client 2.0.2 and requests 2.12.4:

>>> discogs_client.__version__
'2.0.2'
>>> requests.__version__
'2.12.4'

That seemed a bit of out of date, but I checked and I think discogs-client is just misreporting its version—pip thinks it's up to date. (This was on Python 3.6, by the way.)

@robbat2
Copy link
Contributor Author

robbat2 commented Feb 22, 2017

I have requests-2.12.5 & discogs-client-2.2.1 (from pypi, the tag is missing in github); wondering if it's a regression in discogs-client then.

@sampsyo
Copy link
Member

sampsyo commented Feb 22, 2017

For what it's worth, I just double-checked, and it seems like I do have 2.2.1—this issue suggests that they released a wrong value in that __version__ variable. discogs/discogs_client#64

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants