Uploaded image for project: 'CouchDB'
  1. CouchDB
  2. COUCHDB-1144

oauth requests with non-percent-encoded realms result in function_clause error in HTTP request



    • Bug
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 1.0.2, 1.1
    • 1.0.3, 1.1
    • HTTP Interface
    • None
    • OSX - branch 1.1 - r1095237


      As illustrated in this Wireshark packet dump, lines 53ff, any OAuth request with a non-percent-encoded URL as the realm will return a 500 error:

      Erlang stacktrace is similar to:

      [error] [<0.189.0>] function_clause error in HTTP request [Wed, 27 Apr 2011 23:31:46 GMT] [info] [<0.189.0>] Stacktrace:

      {oauth_uri,decode, ["://","ptth"]}




      {oauth_uri, '-params_from_header_string/1-lc$^0/1-0-', 1}











      ] [Wed, 27 Apr 2011 23:31:46 GMT] [info] [<0.189.0>] - - 'PUT'
      /test_c_project/16f74c25-d641-4710-8f38-18295a8a69b1 500

      Chatted with benoitc today and he suggested this may be because the realm is not being properly encoded before being passed on.

      By default, some OAuth libraries such as python's popular oauth2 library always set the realm of a request to the URL of the resource being accessed, such as http://localhost:5984/ . (In fact, oauth2 library does not support overriding this setting.) The OAuth spec @ http://oauth.net/core/1.0/ also shows realms specified as URLs. RFC5849 states "The OPTIONAL "realm" parameter MAY be added and interpreted per [RFC2617] section 1.2.", which in turn says the realm is any quoted-string.

      It seems that this may already be fixed in trunk simply because trunk has a newer version of the upstream erlang-oauth.

      A JS test could be added in futon to validate this by modifying function oauthRequest to accept a realm parameter, then passing that down to OAuth.getAuthorizationHeader on line 56.


        1. oauth_uri.erl
          3 kB
          Joan Touzet
        2. oauth-realm-test.patch
          1 kB
          Joan Touzet



            Unassigned Unassigned
            wohali Joan Touzet
            1 Vote for this issue
            0 Start watching this issue