rgw: make the get_v4_canonical_request_hash identified shorter.

Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>

diff --git a/src/rgw/rgw_auth_s3.cc b/src/rgw/rgw_auth_s3.cc
index 4afcae3e8035d94bbaf65d509cae6f0cc217f122..8aee844aa4b7201c733baf3f906779bd35b6c131 100644 (file)
--- a/src/rgw/rgw_auth_s3.cc
+++ b/src/rgw/rgw_auth_s3.cc
@@ -634,14 +634,16 @@ static std::string assemble_v4_canonical_request(
 
 /*
  * create canonical request for signature version 4
+ *
+ * http://docs.aws.amazon.com/general/latest/gr/sigv4-create-canonical-request.html
  */
-std::string get_v4_canonical_request_hash(CephContext* cct,
-                                          const std::string& http_verb,
-                                          const std::string& canonical_uri,
-                                          const std::string& canonical_qs,
-                                          const std::string& canonical_hdrs,
-                                          const std::string& signed_hdrs,
-                                          const std::string& request_payload_hash)
+std::string get_v4_canon_req_hash(CephContext* cct,
+                                  const std::string& http_verb,
+                                  const std::string& canonical_uri,
+                                  const std::string& canonical_qs,
+                                  const std::string& canonical_hdrs,
+                                  const std::string& signed_hdrs,
+                                  const std::string& request_payload_hash)
 {
   ldout(cct, 10) << "payload request hash = " << request_payload_hash << dendl;
 

diff --git a/src/rgw/rgw_auth_s3.h b/src/rgw/rgw_auth_s3.h
index e41c44a3cd9cffd6255d55514619ad8389269022..b80fad3139f1cbbb569fdaef3b53e202f9e62964 100644 (file)
--- a/src/rgw/rgw_auth_s3.h
+++ b/src/rgw/rgw_auth_s3.h
@@ -194,13 +194,13 @@ boost::optional<std::string> get_v4_canonical_headers(const req_info& info,
 
 std::string hash_string_sha256(const char* data, int len);
 
-std::string get_v4_canonical_request_hash(CephContext* cct,
-                                          const std::string& http_verb,
-                                          const std::string& canonical_uri,
-                                          const std::string& canonical_qs,
-                                          const std::string& canonical_hdrs,
-                                          const std::string& signed_hdrs,
-                                          const std::string& request_payload_hash);
+std::string get_v4_canon_req_hash(CephContext* cct,
+                                  const std::string& http_verb,
+                                  const std::string& canonical_uri,
+                                  const std::string& canonical_qs,
+                                  const std::string& canonical_hdrs,
+                                  const std::string& signed_hdrs,
+                                  const std::string& request_payload_hash);
 
 std::string get_v4_string_to_sign(CephContext* cct,
                                   const std::string& algorithm,

diff --git a/src/rgw/rgw_rest_s3.cc b/src/rgw/rgw_rest_s3.cc
index 8649f602b85aea9c502c6749cb3905c8547554ed..b40754fdbebd78a6b9378aecf7adadafc8e5bc7d 100644 (file)
--- a/src/rgw/rgw_rest_s3.cc
+++ b/src/rgw/rgw_rest_s3.cc
@@ -3535,13 +3535,13 @@ int RGW_Auth_S3::authorize_v4(RGWRados *store, struct req_state *s, bool force_b
 
   /* craft canonical request */
   std::string canonical_req_hash = \
-    rgw::auth::s3::get_v4_canonical_request_hash(s->cct,
-                                                 s->info.method,
-                                                 canonical_uri,
-                                                 canonical_qs,
-                                                 *canonical_headers,
-                                                 signed_hdrs,
-                                                 expected_request_payload_hash);
+    rgw::auth::s3::get_v4_canon_req_hash(s->cct,
+                                         s->info.method,
+                                         canonical_uri,
+                                         canonical_qs,
+                                         *canonical_headers,
+                                         signed_hdrs,
+                                         expected_request_payload_hash);
 
   /*
    * create a string to sign