Merge lp:~rodrigo-moya/couchdb-glib/no-revision-in-auth-header into lp:couchdb-glib

Proposed by Rodrigo Moya
Status: Merged
Approved by: Tim Cole
Approved revision: 98
Merge reported by: Rodrigo Moya
Merged at revision: not available
Proposed branch: lp:~rodrigo-moya/couchdb-glib/no-revision-in-auth-header
Merge into: lp:couchdb-glib
Diff against target: 25 lines
2 files modified
couchdb-glib/couchdb-document.c (+1/-0)
couchdb-glib/utils.c (+3/-0)
To merge this branch: bzr merge lp:~rodrigo-moya/couchdb-glib/no-revision-in-auth-header
Reviewer Review Type Date Requested Status
Tim Cole (community) Approve
Guillermo Gonzalez Approve
Review via email: mp+12927@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Rodrigo Moya (rodrigo-moya) wrote :

Don't include non-OAuth URL parameters in OAuth header

Revision history for this message
Guillermo Gonzalez (verterok) :
review: Approve
Revision history for this message
Tim Cole (tcole) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'couchdb-glib/couchdb-document.c'
--- couchdb-glib/couchdb-document.c 2009-09-09 12:30:29 +0000
+++ couchdb-glib/couchdb-document.c 2009-10-06 15:05:21 +0000
@@ -178,6 +178,7 @@
178 parser = send_message_and_parse (document->couchdb, SOUP_METHOD_DELETE, url, NULL, error);178 parser = send_message_and_parse (document->couchdb, SOUP_METHOD_DELETE, url, NULL, error);
179 if (parser) {179 if (parser) {
180 g_object_unref (G_OBJECT (parser));180 g_object_unref (G_OBJECT (parser));
181 result = TRUE;
181182
182 g_signal_emit_by_name (document->couchdb, "document_deleted", document->dbname, id);183 g_signal_emit_by_name (document->couchdb, "document_deleted", document->dbname, id);
183 }184 }
184185
=== modified file 'couchdb-glib/utils.c'
--- couchdb-glib/utils.c 2009-09-09 14:53:00 +0000
+++ couchdb-glib/utils.c 2009-10-06 15:05:21 +0000
@@ -104,6 +104,9 @@
104 for (i = 0; params[i] != NULL; i++) {104 for (i = 0; params[i] != NULL; i++) {
105 gchar **url_param;105 gchar **url_param;
106106
107 if (!g_str_has_prefix (params[i], "oauth_"))
108 continue;
109
107#ifdef DEBUG_OAUTH110#ifdef DEBUG_OAUTH
108 g_debug ("%s\n", params[i]);111 g_debug ("%s\n", params[i]);
109#endif112#endif

Subscribers

People subscribed via source and target branches