search

disqus / python-phabricator

Python bindings for Phabricator
Apache License 2.0
159 stars 66 forks source link

Support new style token-based authentication #23

Closed kov closed 9 years ago

kov commented 9 years ago

Phabricator has moved on from the clunky user+certificate-based authentication scheme into one that provides the API user with a single token that can be used directly. This change adds support for that and prefers it when available.

Fixes #22.

mikn commented 9 years ago

Is it possible to get this merged? :) You could move the argument during the merge, I agree it should be last. Right now we're ending up packaging it for our own repository because upstream isn't updated yet, otherwise it is quite cumbersome for new people who are starting and just now setting arcanist up.

kov commented 9 years ago

Hey, the commit above has the requested change, please merge it =)

xclaesse commented 9 years ago

That patch works for me, would be nice to get it merged.

polybuildr commented 9 years ago

Works for me! Thanks @gnoronha :D

BYK commented 9 years ago

Sorry for the long wait. Nice job with the patch, thanks!

mikn commented 9 years ago

@BYK It would be really nice if you could update the library on pypi as well!

BYK commented 9 years ago

Released 0.5.0: https://pypi.python.org/pypi/phabricator/0.5.0

Sorry for the wait!