search

drone29a / clj-oauth

OAuth Consumer support for Clojure
BSD 2-Clause "Simplified" License
273 stars 52 forks source link

Request- Update for clj-http "0.5.3" #20

Closed samrat closed 11 years ago

samrat commented 12 years ago

I need to use clj-http and found that clj-oauth starts failing when [clj-http "0.5.3"] is added as a dependency. Could someone please update clj-oauth to work with clj-http 0.5.3.

drone29a commented 12 years ago

Hey, how is it failing and is there an error message you can post?

samrat commented 12 years ago

I start getting a 401 status after I add the latest clj-http. Without that it works fine. I get that when using oauth.client/xauth-access-token

samrat commented 12 years ago

This is what I get:

ExceptionInfo clj-http: status 401 clj-http.client/wrap-exceptions/fn--1555 (client.clj:95)

drone29a commented 12 years ago

I cleaned up the code and wrote some xauth tests against Twitter's example (https://dev.twitter.com/docs/oauth/xauth). I also updated the project dependencies, including clj-http 0.5.3, and all tests pass. However, there may still be an integration issue with clj-http as I don't have an xauth service to test against.

If you update to head (or use 1.4.0-SNAPSHOT, from clojars) let me know if you still have problems. If you are, I would guess the newer version of clj-http has different encoding defaults.

drone29a commented 11 years ago

Closing due to inactivity. Let me know if you're experiencing this problem and will reopen.