|
Jakub Filak |
ece13c8 |
From c5be23b5c5efd1d8ec6df807fc2039454f22414d Mon Sep 17 00:00:00 2001
|
|
Jakub Filak |
ece13c8 |
From: Jakub Filak <jfilak@redhat.com>
|
|
Jakub Filak |
ece13c8 |
Date: Wed, 23 Apr 2014 18:13:47 +0200
|
|
Jakub Filak |
ece13c8 |
Subject: [LIBREPORT PATCH 2/2] Bugzilla: session parameters for XML RPC calls
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
The session parameters are added to every XMLRPC call.
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
abrt_xmlrpc_call*() functions expected formatting string in form of
|
|
Jakub Filak |
ece13c8 |
"({...})" for some good but unknown reason. Since now, the functions
|
|
Jakub Filak |
ece13c8 |
expects formatting string without the outer brackets.
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
() - means empty array (allowed in xmlrpc-c)
|
|
Jakub Filak |
ece13c8 |
{} - means empty structure (allowed in xmlrpc-c)
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
Signed-off-by: Jakub Filak <jfilak@redhat.com>
|
|
Jakub Filak |
ece13c8 |
---
|
|
Jakub Filak |
ece13c8 |
src/lib/abrt_xmlrpc.c | 95 ++++++++++++++++++++++++++++++++++++++++-----------
|
|
Jakub Filak |
ece13c8 |
src/lib/abrt_xmlrpc.h | 5 +--
|
|
Jakub Filak |
ece13c8 |
src/plugins/rhbz.c | 52 +++++++++++++---------------
|
|
Jakub Filak |
ece13c8 |
3 files changed, 102 insertions(+), 50 deletions(-)
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
diff --git a/src/lib/abrt_xmlrpc.c b/src/lib/abrt_xmlrpc.c
|
|
Jakub Filak |
ece13c8 |
index 24bdd9e..84ed50d 100644
|
|
Jakub Filak |
ece13c8 |
--- a/src/lib/abrt_xmlrpc.c
|
|
Jakub Filak |
ece13c8 |
+++ b/src/lib/abrt_xmlrpc.c
|
|
Jakub Filak |
ece13c8 |
@@ -20,6 +20,12 @@
|
|
Jakub Filak |
ece13c8 |
#include "abrt_xmlrpc.h"
|
|
Jakub Filak |
ece13c8 |
#include "proxies.h"
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
+struct abrt_xmlrpc_param_pair
|
|
Jakub Filak |
ece13c8 |
+{
|
|
Jakub Filak |
ece13c8 |
+ char *name;
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_value *value;
|
|
Jakub Filak |
ece13c8 |
+};
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
void abrt_xmlrpc_die(xmlrpc_env *env)
|
|
Jakub Filak |
ece13c8 |
{
|
|
Jakub Filak |
ece13c8 |
error_msg_and_die("fatal: %s", env->fault_string);
|
|
Jakub Filak |
ece13c8 |
@@ -106,12 +112,77 @@ void abrt_xmlrpc_free_client(struct abrt_xmlrpc *ax)
|
|
Jakub Filak |
ece13c8 |
if (ax->ax_client)
|
|
Jakub Filak |
ece13c8 |
xmlrpc_client_destroy(ax->ax_client);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
- if (ax->ax_session_data && ax->ax_session_data_free)
|
|
Jakub Filak |
ece13c8 |
- ax->ax_session_data_free(ax->ax_session_data);
|
|
Jakub Filak |
ece13c8 |
+ for (GList *iter = ax->ax_session_params; iter; iter = g_list_next(iter))
|
|
Jakub Filak |
ece13c8 |
+ {
|
|
Jakub Filak |
ece13c8 |
+ struct abrt_xmlrpc_param_pair *param_pair = (struct abrt_xmlrpc_param_pair *)iter->data;
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_DECREF(param_pair->value);
|
|
Jakub Filak |
ece13c8 |
+ free(param_pair->name);
|
|
Jakub Filak |
ece13c8 |
+ free(param_pair);
|
|
Jakub Filak |
ece13c8 |
+ }
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
+ g_list_free(ax->ax_session_params);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
free(ax);
|
|
Jakub Filak |
ece13c8 |
}
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
+void abrt_xmlrpc_client_add_session_param_string(xmlrpc_env *env, struct abrt_xmlrpc *ax,
|
|
Jakub Filak |
ece13c8 |
+ const char *name, const char *value)
|
|
Jakub Filak |
ece13c8 |
+{
|
|
Jakub Filak |
ece13c8 |
+ struct abrt_xmlrpc_param_pair *new_ses_param = xmalloc(sizeof(*new_ses_param));
|
|
Jakub Filak |
ece13c8 |
+ new_ses_param->name = xstrdup(name);
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
+ new_ses_param->value = xmlrpc_string_new(env, value);
|
|
Jakub Filak |
ece13c8 |
+ if (env->fault_occurred)
|
|
Jakub Filak |
ece13c8 |
+ abrt_xmlrpc_die(env);
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
+ ax->ax_session_params = g_list_append(ax->ax_session_params, new_ses_param);
|
|
Jakub Filak |
ece13c8 |
+}
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
+/* internal helper function */
|
|
Jakub Filak |
ece13c8 |
+static xmlrpc_value *abrt_xmlrpc_call_params_internal(xmlrpc_env *env, struct abrt_xmlrpc *ax, const char *method, xmlrpc_value *params)
|
|
Jakub Filak |
ece13c8 |
+{
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_value *array = xmlrpc_array_new(env);
|
|
Jakub Filak |
ece13c8 |
+ if (env->fault_occurred)
|
|
Jakub Filak |
ece13c8 |
+ abrt_xmlrpc_die(env);
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
+ bool destroy_params = false;
|
|
Jakub Filak |
ece13c8 |
+ if (xmlrpc_value_type(params) == XMLRPC_TYPE_NIL)
|
|
Jakub Filak |
ece13c8 |
+ {
|
|
Jakub Filak |
ece13c8 |
+ destroy_params = true;
|
|
Jakub Filak |
ece13c8 |
+ params = abrt_xmlrpc_params_new(env);
|
|
Jakub Filak |
ece13c8 |
+ }
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
+ if (xmlrpc_value_type(params) == XMLRPC_TYPE_STRUCT)
|
|
Jakub Filak |
ece13c8 |
+ {
|
|
Jakub Filak |
ece13c8 |
+ for (GList *iter = ax->ax_session_params; iter; iter = g_list_next(iter))
|
|
Jakub Filak |
ece13c8 |
+ {
|
|
Jakub Filak |
ece13c8 |
+ struct abrt_xmlrpc_param_pair *param_pair = (struct abrt_xmlrpc_param_pair *)iter->data;
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_struct_set_value(env, params, param_pair->name, param_pair->value);
|
|
Jakub Filak |
ece13c8 |
+ if (env->fault_occurred)
|
|
Jakub Filak |
ece13c8 |
+ abrt_xmlrpc_die(env);
|
|
Jakub Filak |
ece13c8 |
+ }
|
|
Jakub Filak |
ece13c8 |
+ }
|
|
Jakub Filak |
ece13c8 |
+ else
|
|
Jakub Filak |
ece13c8 |
+ {
|
|
Jakub Filak |
ece13c8 |
+ log_warning("Bug: not yet supported XML RPC call type.");
|
|
Jakub Filak |
ece13c8 |
+ }
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_array_append_item(env, array, params);
|
|
Jakub Filak |
ece13c8 |
+ if (env->fault_occurred)
|
|
Jakub Filak |
ece13c8 |
+ abrt_xmlrpc_die(env);
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_value *result = NULL;
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_client_call2(env, ax->ax_client, ax->ax_server_info, method,
|
|
Jakub Filak |
ece13c8 |
+ array, &result);
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
+ if (destroy_params)
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_DECREF(params);
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_DECREF(array);
|
|
Jakub Filak |
ece13c8 |
+ return result;
|
|
Jakub Filak |
ece13c8 |
+}
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
/* internal helper function */
|
|
Jakub Filak |
ece13c8 |
static
|
|
Jakub Filak |
ece13c8 |
xmlrpc_value *abrt_xmlrpc_call_full_va(xmlrpc_env *env, struct abrt_xmlrpc *ax,
|
|
Jakub Filak |
ece13c8 |
@@ -136,10 +207,8 @@ xmlrpc_value *abrt_xmlrpc_call_full_va(xmlrpc_env *env, struct abrt_xmlrpc *ax,
|
|
Jakub Filak |
ece13c8 |
suffix);
|
|
Jakub Filak |
ece13c8 |
}
|
|
Jakub Filak |
ece13c8 |
else
|
|
Jakub Filak |
ece13c8 |
- {
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_client_call2(env, ax->ax_client, ax->ax_server_info, method,
|
|
Jakub Filak |
ece13c8 |
- param, &result);
|
|
Jakub Filak |
ece13c8 |
- }
|
|
Jakub Filak |
ece13c8 |
+ result = abrt_xmlrpc_call_params_internal(env, ax, method, param);
|
|
Jakub Filak |
ece13c8 |
+
|
|
Jakub Filak |
ece13c8 |
xmlrpc_DECREF(param);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
return result;
|
|
Jakub Filak |
ece13c8 |
@@ -195,25 +264,13 @@ void abrt_xmlrpc_params_add_array(xmlrpc_env *env, xmlrpc_value *params, const c
|
|
Jakub Filak |
ece13c8 |
if (env->fault_occurred)
|
|
Jakub Filak |
ece13c8 |
abrt_xmlrpc_die(env);
|
|
Jakub Filak |
ece13c8 |
}
|
|
Jakub Filak |
ece13c8 |
-
|
|
Jakub Filak |
ece13c8 |
xmlrpc_value *abrt_xmlrpc_call_params(xmlrpc_env *env, struct abrt_xmlrpc *ax, const char *method, xmlrpc_value *params)
|
|
Jakub Filak |
ece13c8 |
{
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_value *array = xmlrpc_array_new(env);
|
|
Jakub Filak |
ece13c8 |
- if (env->fault_occurred)
|
|
Jakub Filak |
ece13c8 |
- abrt_xmlrpc_die(env);
|
|
Jakub Filak |
ece13c8 |
-
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_array_append_item(env, array, params);
|
|
Jakub Filak |
ece13c8 |
- if (env->fault_occurred)
|
|
Jakub Filak |
ece13c8 |
- abrt_xmlrpc_die(env);
|
|
Jakub Filak |
ece13c8 |
-
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_value *result = NULL;
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_client_call2(env, ax->ax_client, ax->ax_server_info, method,
|
|
Jakub Filak |
ece13c8 |
- array, &result);
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_value *result = abrt_xmlrpc_call_params_internal(env, ax, method, params);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
if (env->fault_occurred)
|
|
Jakub Filak |
ece13c8 |
abrt_xmlrpc_die(env);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_DECREF(array);
|
|
Jakub Filak |
ece13c8 |
return result;
|
|
Jakub Filak |
ece13c8 |
}
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
diff --git a/src/lib/abrt_xmlrpc.h b/src/lib/abrt_xmlrpc.h
|
|
Jakub Filak |
ece13c8 |
index ff7a65c..feaa04d 100644
|
|
Jakub Filak |
ece13c8 |
--- a/src/lib/abrt_xmlrpc.h
|
|
Jakub Filak |
ece13c8 |
+++ b/src/lib/abrt_xmlrpc.h
|
|
Jakub Filak |
ece13c8 |
@@ -23,6 +23,7 @@
|
|
Jakub Filak |
ece13c8 |
* include/xmlrpc-c/base.h: typedef int32_t xmlrpc_int32;
|
|
Jakub Filak |
ece13c8 |
*/
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
+#include <glib.h>
|
|
Jakub Filak |
ece13c8 |
#include <xmlrpc-c/base.h>
|
|
Jakub Filak |
ece13c8 |
#include <xmlrpc-c/client.h>
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
@@ -35,8 +36,7 @@ typedef void (*abrt_xmlrpc_destroy_fn)(void *);
|
|
Jakub Filak |
ece13c8 |
struct abrt_xmlrpc {
|
|
Jakub Filak |
ece13c8 |
xmlrpc_client *ax_client;
|
|
Jakub Filak |
ece13c8 |
xmlrpc_server_info *ax_server_info;
|
|
Jakub Filak |
ece13c8 |
- void *ax_session_data;
|
|
Jakub Filak |
ece13c8 |
- abrt_xmlrpc_destroy_fn ax_session_data_free;
|
|
Jakub Filak |
ece13c8 |
+ GList *ax_session_params;
|
|
Jakub Filak |
ece13c8 |
};
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
xmlrpc_value *abrt_xmlrpc_array_new(xmlrpc_env *env);
|
|
Jakub Filak |
ece13c8 |
@@ -49,6 +49,7 @@ void abrt_xmlrpc_params_add_array(xmlrpc_env *env, xmlrpc_value *params, const c
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
struct abrt_xmlrpc *abrt_xmlrpc_new_client(const char *url, int ssl_verify);
|
|
Jakub Filak |
ece13c8 |
void abrt_xmlrpc_free_client(struct abrt_xmlrpc *ax);
|
|
Jakub Filak |
ece13c8 |
+void abrt_xmlrpc_client_add_session_param_string(xmlrpc_env *env, struct abrt_xmlrpc *ax, const char *name, const char *value);
|
|
Jakub Filak |
ece13c8 |
void abrt_xmlrpc_die(xmlrpc_env *env) __attribute__((noreturn));
|
|
Jakub Filak |
ece13c8 |
void abrt_xmlrpc_error(xmlrpc_env *env);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
diff --git a/src/plugins/rhbz.c b/src/plugins/rhbz.c
|
|
Jakub Filak |
ece13c8 |
index bc0f87a..bad9ed4 100644
|
|
Jakub Filak |
ece13c8 |
--- a/src/plugins/rhbz.c
|
|
Jakub Filak |
ece13c8 |
+++ b/src/plugins/rhbz.c
|
|
Jakub Filak |
ece13c8 |
@@ -85,9 +85,8 @@ static GList *rhbz_comments(struct abrt_xmlrpc *ax, int bug_id)
|
|
Jakub Filak |
ece13c8 |
* <value><array>
|
|
Jakub Filak |
ece13c8 |
* ...
|
|
Jakub Filak |
ece13c8 |
*/
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_value *xml_response = abrt_xmlrpc_call(ax, "Bug.comments", "({s:(i),s:s})",
|
|
Jakub Filak |
ece13c8 |
- "ids", bug_id,
|
|
Jakub Filak |
ece13c8 |
- "Bugzilla_token", (char *)ax->ax_session_data);
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_value *xml_response = abrt_xmlrpc_call(ax, "Bug.comments", "{s:(i)}",
|
|
Jakub Filak |
ece13c8 |
+ "ids", bug_id);
|
|
Jakub Filak |
ece13c8 |
/* bugs
|
|
Jakub Filak |
ece13c8 |
* This is used for bugs specified in ids. This is a hash, where the
|
|
Jakub Filak |
ece13c8 |
* keys are the numeric ids of the bugs, and the value is a hash with a
|
|
Jakub Filak |
ece13c8 |
@@ -216,7 +215,7 @@ bool rhbz_login(struct abrt_xmlrpc *ax, const char *login, const char *password)
|
|
Jakub Filak |
ece13c8 |
func_entry();
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
xmlrpc_env env;
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_value *result = abrt_xmlrpc_call_full(&env, ax, "User.login", "({s:s,s:s})",
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_value *result = abrt_xmlrpc_call_full(&env, ax, "User.login", "{s:s,s:s}",
|
|
Jakub Filak |
ece13c8 |
"login", login, "password", password);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
if (env.fault_occurred)
|
|
Jakub Filak |
ece13c8 |
@@ -229,8 +228,13 @@ bool rhbz_login(struct abrt_xmlrpc *ax, const char *login, const char *password)
|
|
Jakub Filak |
ece13c8 |
return false;
|
|
Jakub Filak |
ece13c8 |
}
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
- ax->ax_session_data = rhbz_bug_read_item("token", result, RHBZ_READ_STR);
|
|
Jakub Filak |
ece13c8 |
- ax->ax_session_data_free = (abrt_xmlrpc_destroy_fn)free;
|
|
Jakub Filak |
ece13c8 |
+ char *token = rhbz_bug_read_item("token", result, RHBZ_READ_STR);
|
|
Jakub Filak |
ece13c8 |
+ if (token != NULL)
|
|
Jakub Filak |
ece13c8 |
+ {
|
|
Jakub Filak |
ece13c8 |
+ log_debug("Adding session param Bugzilla_token");
|
|
Jakub Filak |
ece13c8 |
+ abrt_xmlrpc_client_add_session_param_string(&env, ax, "Bugzilla_token", token);
|
|
Jakub Filak |
ece13c8 |
+ free(token);
|
|
Jakub Filak |
ece13c8 |
+ }
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
//TODO: with URL like http://bugzilla.redhat.com (that is, with http: instead of https:)
|
|
Jakub Filak |
ece13c8 |
//we are getting this error:
|
|
Jakub Filak |
ece13c8 |
@@ -301,7 +305,7 @@ unsigned rhbz_version(struct abrt_xmlrpc *ax)
|
|
Jakub Filak |
ece13c8 |
func_entry();
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
xmlrpc_value *result;
|
|
Jakub Filak |
ece13c8 |
- result = abrt_xmlrpc_call(ax, "Bugzilla.version", "()");
|
|
Jakub Filak |
ece13c8 |
+ result = abrt_xmlrpc_call(ax, "Bugzilla.version", "{}");
|
|
Jakub Filak |
ece13c8 |
char *version = NULL;
|
|
Jakub Filak |
ece13c8 |
if (result)
|
|
Jakub Filak |
ece13c8 |
version = rhbz_bug_read_item("version", result, RHBZ_READ_STR);
|
|
Jakub Filak |
ece13c8 |
@@ -476,9 +480,8 @@ struct bug_info *rhbz_bug_info(struct abrt_xmlrpc *ax, int bug_id)
|
|
Jakub Filak |
ece13c8 |
* <value><array><data>
|
|
Jakub Filak |
ece13c8 |
* ...
|
|
Jakub Filak |
ece13c8 |
*/
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_value *xml_bug_response = abrt_xmlrpc_call(ax, "Bug.get", "({s:(i),s:s})",
|
|
Jakub Filak |
ece13c8 |
- "ids", bug_id,
|
|
Jakub Filak |
ece13c8 |
- "Bugzilla_token", (char *)ax->ax_session_data);
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_value *xml_bug_response = abrt_xmlrpc_call(ax, "Bug.get", "{s:(i)}",
|
|
Jakub Filak |
ece13c8 |
+ "ids", bug_id);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
xmlrpc_value *bugs_memb = rhbz_get_member("bugs", xml_bug_response);
|
|
Jakub Filak |
ece13c8 |
xmlrpc_value *bug_item = rhbz_array_item_at(bugs_memb, 0);
|
|
Jakub Filak |
ece13c8 |
@@ -604,7 +607,6 @@ int rhbz_new_bug(struct abrt_xmlrpc *ax,
|
|
Jakub Filak |
ece13c8 |
abrt_xmlrpc_params_add_string(&env, params, "summary", (summary ? summary : bzsummary));
|
|
Jakub Filak |
ece13c8 |
abrt_xmlrpc_params_add_string(&env, params, "description", bzcomment);
|
|
Jakub Filak |
ece13c8 |
abrt_xmlrpc_params_add_string(&env, params, "status_whiteboard", status_whiteboard);
|
|
Jakub Filak |
ece13c8 |
- abrt_xmlrpc_params_add_string(&env, params, "Bugzilla_token", (char *)ax->ax_session_data);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
if(arch)
|
|
Jakub Filak |
ece13c8 |
abrt_xmlrpc_params_add_string(&env, params, "platform", arch);
|
|
Jakub Filak |
ece13c8 |
@@ -674,7 +676,7 @@ int rhbz_attach_blob(struct abrt_xmlrpc *ax, const char *bug_id,
|
|
Jakub Filak |
ece13c8 |
* 6 -> base64, two arguments (char* plain data which will be encoded by xmlrpc-c to base64,
|
|
Jakub Filak |
ece13c8 |
* size_t number of bytes to encode)
|
|
Jakub Filak |
ece13c8 |
*/
|
|
Jakub Filak |
ece13c8 |
- result = abrt_xmlrpc_call(ax, "Bug.add_attachment", "({s:(s),s:s,s:s,s:s,s:6,s:i,s:s})",
|
|
Jakub Filak |
ece13c8 |
+ result = abrt_xmlrpc_call(ax, "Bug.add_attachment", "{s:(s),s:s,s:s,s:s,s:6,s:i}",
|
|
Jakub Filak |
ece13c8 |
"ids", bug_id,
|
|
Jakub Filak |
ece13c8 |
"summary", fn,
|
|
Jakub Filak |
ece13c8 |
"file_name", filename,
|
|
Jakub Filak |
ece13c8 |
@@ -687,8 +689,7 @@ int rhbz_attach_blob(struct abrt_xmlrpc *ax, const char *bug_id,
|
|
Jakub Filak |
ece13c8 |
/* Undocumented argument but it works with Red Hat Bugzilla version 4.2.4-7
|
|
Jakub Filak |
ece13c8 |
* and version 4.4.rc1.b02
|
|
Jakub Filak |
ece13c8 |
*/
|
|
Jakub Filak |
ece13c8 |
- "nomail", nomail_notify,
|
|
Jakub Filak |
ece13c8 |
- "Bugzilla_token", (char *)ax->ax_session_data
|
|
Jakub Filak |
ece13c8 |
+ "nomail", nomail_notify
|
|
Jakub Filak |
ece13c8 |
);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
free(fn);
|
|
Jakub Filak |
ece13c8 |
@@ -745,8 +746,7 @@ void rhbz_logout(struct abrt_xmlrpc *ax)
|
|
Jakub Filak |
ece13c8 |
func_entry();
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
xmlrpc_env env;
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_value *result = abrt_xmlrpc_call_full(&env, ax, "User.logout", "({s:s})",
|
|
Jakub Filak |
ece13c8 |
- "Bugzilla_token", (char *)ax->ax_session_data);
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_value *result = abrt_xmlrpc_call_full(&env, ax, "User.logout", "{}");
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
if (env.fault_occurred)
|
|
Jakub Filak |
ece13c8 |
log_warning("xmlrpc fault: (%d) %s", env.fault_code, env.fault_string);
|
|
Jakub Filak |
ece13c8 |
@@ -798,11 +798,10 @@ void rhbz_mail_to_cc(struct abrt_xmlrpc *ax, int bug_id, const char *mail, int f
|
|
Jakub Filak |
ece13c8 |
);
|
|
Jakub Filak |
ece13c8 |
#endif
|
|
Jakub Filak |
ece13c8 |
/* Bugzilla 4.0+ uses this API: */
|
|
Jakub Filak |
ece13c8 |
- result = abrt_xmlrpc_call(ax, "Bug.update", "({s:i,s:{s:(s),s:i},s:s})",
|
|
Jakub Filak |
ece13c8 |
+ result = abrt_xmlrpc_call(ax, "Bug.update", "{s:i,s:{s:(s),s:i}}",
|
|
Jakub Filak |
ece13c8 |
"ids", bug_id,
|
|
Jakub Filak |
ece13c8 |
"cc", "add", mail,
|
|
Jakub Filak |
ece13c8 |
- "nomail", nomail_notify,
|
|
Jakub Filak |
ece13c8 |
- "Bugzilla_token", (char *)ax->ax_session_data
|
|
Jakub Filak |
ece13c8 |
+ "nomail", nomail_notify
|
|
Jakub Filak |
ece13c8 |
);
|
|
Jakub Filak |
ece13c8 |
if (result)
|
|
Jakub Filak |
ece13c8 |
xmlrpc_DECREF(result);
|
|
Jakub Filak |
ece13c8 |
@@ -836,10 +835,9 @@ void rhbz_add_comment(struct abrt_xmlrpc *ax, int bug_id, const char *comment,
|
|
Jakub Filak |
ece13c8 |
int nomail_notify = !!IS_NOMAIL_NOTIFY(flags);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
xmlrpc_value *result;
|
|
Jakub Filak |
ece13c8 |
- result = abrt_xmlrpc_call(ax, "Bug.add_comment", "({s:i,s:s,s:b,s:i,s:s})",
|
|
Jakub Filak |
ece13c8 |
+ result = abrt_xmlrpc_call(ax, "Bug.add_comment", "{s:i,s:s,s:b,s:i}",
|
|
Jakub Filak |
ece13c8 |
"id", bug_id, "comment", comment,
|
|
Jakub Filak |
ece13c8 |
- "private", private, "nomail", nomail_notify,
|
|
Jakub Filak |
ece13c8 |
- "Bugzilla_token", (char *)ax->ax_session_data);
|
|
Jakub Filak |
ece13c8 |
+ "private", private, "nomail", nomail_notify);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
if (result)
|
|
Jakub Filak |
ece13c8 |
xmlrpc_DECREF(result);
|
|
Jakub Filak |
ece13c8 |
@@ -850,15 +848,14 @@ void rhbz_set_url(struct abrt_xmlrpc *ax, int bug_id, const char *url, int flags
|
|
Jakub Filak |
ece13c8 |
func_entry();
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
const int nomail_notify = !!IS_NOMAIL_NOTIFY(flags);
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_value *result = abrt_xmlrpc_call(ax, "Bug.update", "({s:i,s:s,s:i,s:s})",
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_value *result = abrt_xmlrpc_call(ax, "Bug.update", "{s:i,s:s,s:i}",
|
|
Jakub Filak |
ece13c8 |
"ids", bug_id,
|
|
Jakub Filak |
ece13c8 |
"url", url,
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
/* Undocumented argument but it works with Red Hat Bugzilla version 4.2.4-7
|
|
Jakub Filak |
ece13c8 |
* and version 4.4.rc1.b02
|
|
Jakub Filak |
ece13c8 |
*/
|
|
Jakub Filak |
ece13c8 |
- "nomail", nomail_notify,
|
|
Jakub Filak |
ece13c8 |
- "Bugzilla_token", (char *)ax->ax_session_data
|
|
Jakub Filak |
ece13c8 |
+ "nomail", nomail_notify
|
|
Jakub Filak |
ece13c8 |
);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
if (result)
|
|
Jakub Filak |
ece13c8 |
@@ -886,10 +883,7 @@ xmlrpc_value *rhbz_search_duphash(struct abrt_xmlrpc *ax,
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
char *s = strbuf_free_nobuf(query);
|
|
Jakub Filak |
ece13c8 |
log_debug("search for '%s'", s);
|
|
Jakub Filak |
ece13c8 |
- xmlrpc_value *search = (ax->ax_session_data == NULL)
|
|
Jakub Filak |
ece13c8 |
- ? abrt_xmlrpc_call(ax, "Bug.search", "({s:s})", "quicksearch", s)
|
|
Jakub Filak |
ece13c8 |
- : abrt_xmlrpc_call(ax, "Bug.search", "({s:s,s:s})", "quicksearch", s,
|
|
Jakub Filak |
ece13c8 |
- "Bugzilla_token", (char *)ax->ax_session_data);
|
|
Jakub Filak |
ece13c8 |
+ xmlrpc_value *search = abrt_xmlrpc_call(ax, "Bug.search", "{s:s}", "quicksearch", s);
|
|
Jakub Filak |
ece13c8 |
|
|
Jakub Filak |
ece13c8 |
free(s);
|
|
Jakub Filak |
ece13c8 |
xmlrpc_value *bugs = rhbz_get_member("bugs", search);
|
|
Jakub Filak |
ece13c8 |
--
|
|
Jakub Filak |
ece13c8 |
1.8.3.1
|
|
Jakub Filak |
ece13c8 |
|