Token authorization + test coverage #6

Merged
merged 9 commits into from Sep 7, 2012

Conversation

Projects
None yet
3 participants
  1. Token authentication implemented following the latest Pinboard API update (http://blog.pinboard.in/2012/07/api_authentication_tokens/). Backward compatibility is ok.
  2. Basic unit tests added for token and canonical username/password-base authentication.

+1

Thanks a lot @dreikanter, I was about to launch myself into the task of adding token auth support, when I saw your pull request.

Owner

mgan59 commented Sep 7, 2012

Looked over the commits, they look great. Thanks for the test and the cleanup in various places.

@mgan59 mgan59 added a commit that referenced this pull request Sep 7, 2012

@mgan59 mgan59 Merge pull request #6 from dreikanter/master
Token authorization + test coverage
2538fcc

@mgan59 mgan59 merged commit 2538fcc into mgan59:master Sep 7, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment