diff options
author | jsdelfino <jsdelfino@13f79535-47bb-0310-9956-ffa450edef68> | 2011-02-26 20:59:35 +0000 |
---|---|---|
committer | jsdelfino <jsdelfino@13f79535-47bb-0310-9956-ffa450edef68> | 2011-02-26 20:59:35 +0000 |
commit | 7c018c6fb691b65ac1cb181a95f5766e2933eb3c (patch) | |
tree | b12eb4fd6fc05faebd9d481f1d57599cd0b2829e /sca-cpp/trunk/modules/oauth | |
parent | 0ee21c5d9657a1259a731722f53666ad135279bb (diff) |
Correctly pass query strings through component wiring redirects and improve format of XML and JSON response documents.
git-svn-id: http://svn.us.apache.org/repos/asf/tuscany@1074924 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'sca-cpp/trunk/modules/oauth')
-rw-r--r-- | sca-cpp/trunk/modules/oauth/mod-oauth1.cpp | 8 | ||||
-rw-r--r-- | sca-cpp/trunk/modules/oauth/mod-oauth2.cpp | 10 |
2 files changed, 9 insertions, 9 deletions
diff --git a/sca-cpp/trunk/modules/oauth/mod-oauth1.cpp b/sca-cpp/trunk/modules/oauth/mod-oauth1.cpp index 84de8f2d05..acf39601b3 100644 --- a/sca-cpp/trunk/modules/oauth/mod-oauth1.cpp +++ b/sca-cpp/trunk/modules/oauth/mod-oauth1.cpp @@ -193,7 +193,7 @@ const failable<int> authorize(const list<list<value> >& args, request_rec* r, co // Build the redirect URI const list<list<value> > redirargs = mklist<list<value> >(mklist<value>("mod_oauth1_step", "access_token"), tok, cid, info); - const string redir = httpd::url(r->uri, r) + string("?") + httpd::queryString(redirargs); + const string redir = httpd::url(r->uri, r) + string("?") + http::queryString(redirargs); debug(redir, "modoauth1::authorize::redir"); // Lookup client app configuration @@ -203,7 +203,7 @@ const failable<int> authorize(const list<list<value> >& args, request_rec* r, co list<value> appkey = cadr(app); // Build and sign the request token URI - const string requri = httpd::unescape(cadr(req)) + string("&") + httpd::queryString(mklist<list<value> >(mklist<value>("oauth_callback", httpd::escape(redir)))); + const string requri = httpd::unescape(cadr(req)) + string("&") + http::queryString(mklist<list<value> >(mklist<value>("oauth_callback", httpd::escape(redir)))); const list<string> srequri = sign("POST", requri, appkey, "", ""); debug(srequri, "modoauth1::authorize::srequri"); @@ -236,7 +236,7 @@ const failable<int> authorize(const list<list<value> >& args, request_rec* r, co return mkfailure<int>(reason(prc)); // Redirect to the authorize URI - const string authuri = httpd::unescape(cadr(auth)) + string("?") + httpd::queryString(mklist<list<value> >(tv)); + const string authuri = httpd::unescape(cadr(auth)) + string("?") + http::queryString(mklist<list<value> >(tv)); debug(authuri, "modoauth1::authorize::authuri"); return httpd::externalRedirect(authuri, r); } @@ -325,7 +325,7 @@ const failable<int> access_token(const list<list<value> >& args, request_rec* r, return mkfailure<int>(reason(sv)); // Build and sign access token request URI - const string tokuri = httpd::unescape(cadr(tok)) + string("?") + httpd::queryString(mklist<list<value> >(vv)); + const string tokuri = httpd::unescape(cadr(tok)) + string("?") + http::queryString(mklist<list<value> >(vv)); const list<string> stokuri = sign("POST", tokuri, appkey, cadr(tv), content(sv)); debug(stokuri, "modoauth1::access_token::stokuri"); diff --git a/sca-cpp/trunk/modules/oauth/mod-oauth2.cpp b/sca-cpp/trunk/modules/oauth/mod-oauth2.cpp index 51bd240177..a662333c9e 100644 --- a/sca-cpp/trunk/modules/oauth/mod-oauth2.cpp +++ b/sca-cpp/trunk/modules/oauth/mod-oauth2.cpp @@ -141,7 +141,7 @@ const failable<int> authorize(const list<list<value> >& args, request_rec* r, co // Build the redirect URI const list<list<value> > rargs = mklist<list<value> >(mklist<value>("mod_oauth2_step", "access_token"), tok, cid, info); - const string redir = httpd::url(r->uri, r) + string("?") + httpd::queryString(rargs); + const string redir = httpd::url(r->uri, r) + string("?") + http::queryString(rargs); debug(redir, "modoauth2::authorize::redir"); // Lookup client app configuration @@ -152,7 +152,7 @@ const failable<int> authorize(const list<list<value> >& args, request_rec* r, co // Redirect to the authorize URI const list<list<value> > aargs = mklist<list<value> >(mklist<value>("client_id", car(appkey)), mklist<value>("scope", "email"), mklist<value>("redirect_uri", httpd::escape(redir))); - const string uri = httpd::unescape(cadr(auth)) + string("?") + httpd::queryString(aargs); + const string uri = httpd::unescape(cadr(auth)) + string("?") + http::queryString(aargs); debug(uri, "modoauth2::authorize::uri"); return httpd::externalRedirect(uri, r); } @@ -192,12 +192,12 @@ const failable<int> access_token(const list<list<value> >& args, request_rec* r, // Build the redirect URI const list<list<value> > rargs = mklist<list<value> >(mklist<value>("mod_oauth2_step", "access_token"), tok, cid, info); - const string redir = httpd::url(r->uri, r) + string("?") + httpd::queryString(rargs); + const string redir = httpd::url(r->uri, r) + string("?") + http::queryString(rargs); debug(redir, "modoauth2::access_token::redir"); // Request access token const list<list<value> > targs = mklist<list<value> >(mklist<value>("client_id", car(appkey)), mklist<value>("redirect_uri", httpd::escape(redir)), mklist<value>("client_secret", cadr(appkey)), code); - const string turi = httpd::unescape(cadr(tok)) + string("?") + httpd::queryString(targs); + const string turi = httpd::unescape(cadr(tok)) + string("?") + http::queryString(targs); debug(turi, "modoauth2::access_token::tokenuri"); const failable<value> tr = http::get(turi, sc.cs); if (!hasContent(tr)) @@ -211,7 +211,7 @@ const failable<int> access_token(const list<list<value> >& args, request_rec* r, // Request user info // TODO Make this step configurable const list<list<value> > iargs = mklist<list<value> >(tv); - const string iuri = httpd::unescape(cadr(info)) + string("?") + httpd::queryString(iargs); + const string iuri = httpd::unescape(cadr(info)) + string("?") + http::queryString(iargs); debug(iuri, "modoauth2::access_token::infouri"); const failable<value> profres = http::get(iuri, sc.cs); if (!hasContent(profres)) |