ORCID / python-orcid

Python wrapper around ORCID API
BSD 3-Clause "New" or "Revised" License
92 stars 26 forks source link

token authentication url #20

Open btw1027 opened 7 years ago

btw1027 commented 7 years ago

I think the token url is incorrect for the PublicAPI class

is this: self._token_url = https://api.orcid.org/oauth/token

should be this instead? self._token_url = "https://orcid.org/oauth/token"

MSusik commented 7 years ago

@btw1027 What kind of error is displayed?

rcpeters commented 7 years ago

There wont be an error, both endpoints work the same. However we do recommend using https://orcid.org/oauth/token now.

Cheers, Rob

On Tue, Oct 31, 2017 at 5:53 AM, Mateusz Susik notifications@github.com wrote:

@btw1027 https://github.com/btw1027 What kind of error is displayed?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ORCID/python-orcid/issues/20#issuecomment-340752969, or mute the thread https://github.com/notifications/unsubscribe-auth/AALx5Frx5Nss-FK3UoF7jQhXRwN8IQ_iks5sxxg6gaJpZM4QBaJ5 .

-- Robert Peters info@rcpeters.com 805.440.9056

martijnende commented 2 years ago

Using https://api.orcid.org/oauth/token now leads to a 400 error. _token_url should be updated.