Sick-Beard icon indicating copy to clipboard operation
Sick-Beard copied to clipboard

Published 20 hours ago • verified publisher icon midgetspy

Fixed the Trakt notifier to use basic authentication

Open jaredhocutt opened this issue 12 years ago • 3 comments

The Trakt notifier was using POST authentication which does not appear to work even though the Trakt API documentation says it is supported. I modified the code to use basic authentication instead so that the Trakt notifier will authenticate and work.

jaredhocutt avatar Oct 12 '12 03:10 jaredhocutt

"The Trakt notifier was using POST authentication which does not appear to work even though the Trakt API documentation says it is supported."

still don't working?

fernandog avatar Aug 20 '13 19:08 fernandog

new code goes into development branch not master. plz update pull request to reflect that

thezoggy avatar Aug 20 '13 19:08 thezoggy

Just checked and it appears to work again. Seems that Trakt may have been having issues with accepting username/password as POST parameters at the time I made this pull request. It still probably wouldn't be a bad idea to use basic authentication. Would you like for me to still update the pull request for the development branch or go ahead and close this pull request?

jaredhocutt avatar Aug 20 '13 22:08 jaredhocutt