Merge pull request #15527 from yashykt/httpsclihelp

Log error message for debugging #13650
7 years ago · f9035befc6
parent bb144f083e 64321c0618
commit f9035befc6
2 changed files with 4 additions and 2 deletions
--- a/test/core/http/httpcli_test.cc
+++ b/test/core/http/httpcli_test.cc
@ -47,7 +47,8 @@ static void on_finish(void* arg, grpc_error* error) {
      "<body><p>This is a test</p></body></html>";
  grpc_http_response* response = static_cast<grpc_http_response*>(arg);
  GPR_ASSERT(response);
-  gpr_log(GPR_INFO, "response status %d", response->status);
+  gpr_log(GPR_INFO, "response status=%d error=%s", response->status,
+          grpc_error_string(error));
  GPR_ASSERT(response->status == 200);
  GPR_ASSERT(response->body_length == strlen(expect));
  GPR_ASSERT(0 == memcmp(expect, response->body, response->body_length));
--- a/test/core/http/httpscli_test.cc
+++ b/test/core/http/httpscli_test.cc
@ -49,7 +49,8 @@ static void on_finish(void* arg, grpc_error* error) {
      "<body><p>This is a test</p></body></html>";
  grpc_http_response* response = static_cast<grpc_http_response*>(arg);
  GPR_ASSERT(response);
-  gpr_log(GPR_INFO, "response status %d", response->status);
+  gpr_log(GPR_INFO, "response status=%d error=%s", response->status,
+          grpc_error_string(error));
  GPR_ASSERT(response->status == 200);
  GPR_ASSERT(response->body_length == strlen(expect));
  GPR_ASSERT(0 == memcmp(expect, response->body, response->body_length));