|
 |
a47729 |
From 7c5540bb47799b4db95673d22f61995ad5c56440 Mon Sep 17 00:00:00 2001
|
|
|
a47729 |
From: Rob Crittenden <rcritten@redhat.com>
|
|
|
a47729 |
Date: Mon, 31 Jan 2022 17:31:50 -0500
|
|
|
a47729 |
Subject: [PATCH] Remove ipa-join errors from behind the debug option
|
|
|
a47729 |
|
|
|
a47729 |
This brings it inline with the previous XML-RPC output which
|
|
|
a47729 |
only hid the request and response from the output and not
|
|
|
a47729 |
any errors returned.
|
|
|
a47729 |
|
|
|
a47729 |
https://pagure.io/freeipa/issue/9103
|
|
|
a47729 |
|
|
|
a47729 |
Signed-off-by: Rob Crittenden <rcritten@redhat.com>
|
|
|
a47729 |
Reviewed-By: Florence Blanc-Renaud <frenaud@redhat.com>
|
|
|
a47729 |
Reviewed-By: Peter Keresztes Schmidt <carbenium@outlook.com>
|
|
|
a47729 |
---
|
|
|
a47729 |
client/ipa-join.c | 27 +++++++++------------------
|
|
|
a47729 |
1 file changed, 9 insertions(+), 18 deletions(-)
|
|
|
a47729 |
|
|
|
a47729 |
diff --git a/client/ipa-join.c b/client/ipa-join.c
|
|
|
a47729 |
index d98739a9a..5888a33bf 100644
|
|
|
a47729 |
--- a/client/ipa-join.c
|
|
|
a47729 |
+++ b/client/ipa-join.c
|
|
|
a47729 |
@@ -743,8 +743,7 @@ jsonrpc_request(const char *ipaserver, const json_t *json, curl_buffer *response
|
|
|
a47729 |
|
|
|
a47729 |
json_str = json_dumps(json, 0);
|
|
|
a47729 |
if (!json_str) {
|
|
|
a47729 |
- if (debug)
|
|
|
a47729 |
- fprintf(stderr, _("json_dumps() failed\n"));
|
|
|
a47729 |
+ fprintf(stderr, _("json_dumps() failed\n"));
|
|
|
a47729 |
|
|
|
a47729 |
rval = 17;
|
|
|
a47729 |
goto cleanup;
|
|
|
a47729 |
@@ -758,8 +757,7 @@ jsonrpc_request(const char *ipaserver, const json_t *json, curl_buffer *response
|
|
|
a47729 |
CURLcode res = curl_easy_perform(curl);
|
|
|
a47729 |
if (res != CURLE_OK)
|
|
|
a47729 |
{
|
|
|
a47729 |
- if (debug)
|
|
|
a47729 |
- fprintf(stderr, _("JSON-RPC call failed: %s\n"), curl_easy_strerror(res));
|
|
|
a47729 |
+ fprintf(stderr, _("JSON-RPC call failed: %s\n"), curl_easy_strerror(res));
|
|
|
a47729 |
|
|
|
a47729 |
rval = 17;
|
|
|
a47729 |
goto cleanup;
|
|
|
a47729 |
@@ -769,8 +767,7 @@ jsonrpc_request(const char *ipaserver, const json_t *json, curl_buffer *response
|
|
|
a47729 |
curl_easy_getinfo(curl, CURLINFO_RESPONSE_CODE, &resp_code);
|
|
|
a47729 |
|
|
|
a47729 |
if (resp_code != 200) {
|
|
|
a47729 |
- if (debug)
|
|
|
a47729 |
- fprintf(stderr, _("JSON-RPC call failed with status code: %li\n"), resp_code);
|
|
|
a47729 |
+ fprintf(stderr, _("JSON-RPC call failed with status code: %li\n"), resp_code);
|
|
|
a47729 |
|
|
|
a47729 |
if (!quiet && resp_code == 401)
|
|
|
a47729 |
fprintf(stderr, _("JSON-RPC call was unauthorized. Check your credentials.\n"));
|
|
|
a47729 |
@@ -848,8 +845,7 @@ jsonrpc_parse_response(const char *payload, json_t** j_result_obj, bool quiet) {
|
|
|
a47729 |
|
|
|
a47729 |
j_root = json_loads(payload, 0, &j_error);
|
|
|
a47729 |
if (!j_root) {
|
|
|
a47729 |
- if (debug)
|
|
|
a47729 |
- fprintf(stderr, _("Parsing JSON-RPC response failed: %s\n"), j_error.text);
|
|
|
a47729 |
+ fprintf(stderr, _("Parsing JSON-RPC response failed: %s\n"), j_error.text);
|
|
|
a47729 |
|
|
|
a47729 |
rval = 17;
|
|
|
a47729 |
goto cleanup;
|
|
|
a47729 |
@@ -864,8 +860,7 @@ jsonrpc_parse_response(const char *payload, json_t** j_result_obj, bool quiet) {
|
|
|
a47729 |
|
|
|
a47729 |
*j_result_obj = json_object_get(j_root, "result");
|
|
|
a47729 |
if (!*j_result_obj) {
|
|
|
a47729 |
- if (debug)
|
|
|
a47729 |
- fprintf(stderr, _("Parsing JSON-RPC response failed: no 'result' value found.\n"));
|
|
|
a47729 |
+ fprintf(stderr, _("Parsing JSON-RPC response failed: no 'result' value found.\n"));
|
|
|
a47729 |
|
|
|
a47729 |
rval = 17;
|
|
|
a47729 |
goto cleanup;
|
|
|
a47729 |
@@ -897,8 +892,7 @@ jsonrpc_parse_join_response(const char *payload, join_info *join_i, bool quiet)
|
|
|
a47729 |
&tmp_hostdn,
|
|
|
a47729 |
"krbprincipalname", &tmp_princ,
|
|
|
a47729 |
"krblastpwdchange", &tmp_pwdch) != 0) {
|
|
|
a47729 |
- if (debug)
|
|
|
a47729 |
- fprintf(stderr, _("Extracting the data from the JSON-RPC response failed: %s\n"), j_error.text);
|
|
|
a47729 |
+ fprintf(stderr, _("Extracting the data from the JSON-RPC response failed: %s\n"), j_error.text);
|
|
|
a47729 |
|
|
|
a47729 |
rval = 17;
|
|
|
a47729 |
goto cleanup;
|
|
|
a47729 |
@@ -941,8 +935,7 @@ join_krb5_jsonrpc(const char *ipaserver, const char *hostname, char **hostdn, co
|
|
|
a47729 |
"nshardwareplatform", uinfo.machine);
|
|
|
a47729 |
|
|
|
a47729 |
if (!json_req) {
|
|
|
a47729 |
- if (debug)
|
|
|
a47729 |
- fprintf(stderr, _("json_pack_ex() failed: %s\n"), j_error.text);
|
|
|
a47729 |
+ fprintf(stderr, _("json_pack_ex() failed: %s\n"), j_error.text);
|
|
|
a47729 |
|
|
|
a47729 |
rval = 17;
|
|
|
a47729 |
goto cleanup;
|
|
|
a47729 |
@@ -990,8 +983,7 @@ jsonrpc_parse_unenroll_response(const char *payload, bool* result, bool quiet) {
|
|
|
a47729 |
|
|
|
a47729 |
if (json_unpack_ex(j_result_obj, &j_error, 0, "{s:b}",
|
|
|
a47729 |
"result", result) != 0) {
|
|
|
a47729 |
- if (debug)
|
|
|
a47729 |
- fprintf(stderr, _("Extracting the data from the JSON-RPC response failed: %s\n"), j_error.text);
|
|
|
a47729 |
+ fprintf(stderr, _("Extracting the data from the JSON-RPC response failed: %s\n"), j_error.text);
|
|
|
a47729 |
|
|
|
a47729 |
rval = 20;
|
|
|
a47729 |
goto cleanup;
|
|
|
a47729 |
@@ -1021,8 +1013,7 @@ jsonrpc_unenroll_host(const char *ipaserver, const char *host, bool quiet) {
|
|
|
a47729 |
host);
|
|
|
a47729 |
|
|
|
a47729 |
if (!json_req) {
|
|
|
a47729 |
- if (debug)
|
|
|
a47729 |
- fprintf(stderr, _("json_pack_ex() failed: %s\n"), j_error.text);
|
|
|
a47729 |
+ fprintf(stderr, _("json_pack_ex() failed: %s\n"), j_error.text);
|
|
|
a47729 |
|
|
|
a47729 |
rval = 17;
|
|
|
a47729 |
goto cleanup;
|
|
|
a47729 |
--
|
|
|
a47729 |
2.34.1
|
|
|
a47729 |
|