You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2021/04/21 21:42:03 UTC

[trafficserver] branch 9.0.x updated: uri_signing: fix warning which affects ubuntu:20.04 builds (#7717)

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new b5be79b  uri_signing: fix warning which affects ubuntu:20.04 builds (#7717)
b5be79b is described below

commit b5be79ba9678ae0f2eb958cae26fde4ef9f48070
Author: Brian Olsen <bn...@gmail.com>
AuthorDate: Mon Apr 19 15:27:05 2021 -0600

    uri_signing: fix warning which affects ubuntu:20.04 builds (#7717)
    
    (cherry picked from commit ab1adae4b7cfc43b21c5d444795d0a531d56b20e)
---
 plugins/experimental/uri_signing/uri_signing.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/plugins/experimental/uri_signing/uri_signing.c b/plugins/experimental/uri_signing/uri_signing.c
index fe6793b..d70bd56 100644
--- a/plugins/experimental/uri_signing/uri_signing.c
+++ b/plugins/experimental/uri_signing/uri_signing.c
@@ -355,11 +355,10 @@ check_auth:
   }
   return status;
 fail:
-  PluginDebug("Invalid JWT for %.*s", url_ct, url);
   TSHttpTxnStatusSet(txnp, TS_HTTP_STATUS_FORBIDDEN);
-  PluginDebug("Spent %" PRId64 " ns uri_signing verification of %.*s.", mark_timer(&t), url_ct, url);
-
   if (url != NULL) {
+    PluginDebug("Invalid JWT for %.*s", url_ct, url);
+    PluginDebug("Spent %" PRId64 " ns uri_signing verification of %.*s.", mark_timer(&t), url_ct, url);
     TSfree((void *)url);
   }
   if (strip_uri != NULL) {