rgw: rename ACL & policy IsPublic to is_public

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>

diff --git a/src/rgw/rgw_acl.cc b/src/rgw/rgw_acl.cc
index ff27a7ae676a5b2c64bdf7dc1c9cdb9acdda6e04..e1d1ecb442bb278b3ed322fea296648ddce2ce31 100644 (file)
--- a/src/rgw/rgw_acl.cc
+++ b/src/rgw/rgw_acl.cc
@@ -188,7 +188,7 @@ bool RGWAccessControlPolicy::verify_permission(const DoutPrefixProvider* dpp,
 }
 
 
-bool RGWAccessControlPolicy::IsPublic() const
+bool RGWAccessControlPolicy::is_public() const
 {
 
   static constexpr auto public_groups = {ACL_GROUP_ALL_USERS,

diff --git a/src/rgw/rgw_acl.h b/src/rgw/rgw_acl.h
index 278e8b4cbe936b144af98f830d99cd73d58bf7f8..aeb30e2f99c99deefe528abd1d72a1eb921f5c74 100644 (file)
--- a/src/rgw/rgw_acl.h
+++ b/src/rgw/rgw_acl.h
@@ -465,7 +465,7 @@ public:
   }
 
   virtual bool compare_group_name(string& id, ACLGroupTypeEnum group) { return false; }
-  bool IsPublic() const;
+  bool is_public() const;
 };
 WRITE_CLASS_ENCODER(RGWAccessControlPolicy)
 

diff --git a/src/rgw/rgw_iam_policy.cc b/src/rgw/rgw_iam_policy.cc
index 72c4f4ab6d51617014b7bec8e8349f3165ed669c..7325ec0854ba6f461ecf8818b7eb831181686a6d 100644 (file)
--- a/src/rgw/rgw_iam_policy.cc
+++ b/src/rgw/rgw_iam_policy.cc
@@ -1458,7 +1458,7 @@ struct IsPublicStatement
 };
 
 
-bool IsPublic(const Policy& p)
+bool is_public(const Policy& p)
 {
   return std::any_of(p.statements.begin(), p.statements.end(), IsPublicStatement());
 }

diff --git a/src/rgw/rgw_iam_policy.h b/src/rgw/rgw_iam_policy.h
index 78f363a145bde656c4458bd06ff641aa3a6b85a0..d66fd48dc9ee8a5f7a27457cf052de62efd84803 100644 (file)
--- a/src/rgw/rgw_iam_policy.h
+++ b/src/rgw/rgw_iam_policy.h
@@ -494,7 +494,7 @@ struct Policy {
 };
 
 std::ostream& operator <<(ostream& m, const Policy& p);
-bool IsPublic(const Policy& p);
+bool is_public(const Policy& p);
 
 }
 }

diff --git a/src/rgw/rgw_op.cc b/src/rgw/rgw_op.cc
index 6509d6b0eaae31d612c7f254bfc824d94d812e53..ae535e2f3de24d3a3991fdb726d7f34f8d9d1c67 100644 (file)
--- a/src/rgw/rgw_op.cc
+++ b/src/rgw/rgw_op.cc
@@ -5539,7 +5539,7 @@ void RGWPutACLs::execute()
 
   if (s->bucket_access_conf &&
       s->bucket_access_conf->block_public_acls() &&
-      new_policy.IsPublic()) {
+      new_policy.is_public()) {
     op_ret = -EACCES;
     return;
   }
@@ -7696,7 +7696,7 @@ void RGWPutBucketPolicy::execute()
     auto attrs = s->bucket_attrs;
     if (s->bucket_access_conf &&
         s->bucket_access_conf->block_public_policy() &&
-        rgw::IAM::IsPublic(p)) {
+        rgw::IAM::is_public(p)) {
       op_ret = -EACCES;
       return;
     }
@@ -8113,7 +8113,7 @@ int RGWGetBucketPolicyStatus::verify_permission()
 
 void RGWGetBucketPolicyStatus::execute()
 {
-  isPublic = (s->iam_policy && rgw::IAM::IsPublic(*s->iam_policy)) || s->bucket_acl->IsPublic();
+  isPublic = (s->iam_policy && rgw::IAM::is_public(*s->iam_policy)) || s->bucket_acl->is_public();
 }
 
 int RGWPutBucketPublicAccessBlock::verify_permission()