Merge pull request #61388 from AliMasarweh/wip-alimasa-bucket-logging-log-info

RGW\logging: added missing fields

Reviewed-by: yuvalif<ylifshit@redhat.com>

diff --cc src/rgw/rgw_asio_frontend.cc
Simple merge

diff --cc src/rgw/rgw_auth_s3.h
Simple merge

diff --cc src/rgw/rgw_bucket_logging.cc
index dd407f26e8c661499b32eb0f829941e4816cd229,e43196f9d482714c3dc4e2cac06fc5df0cb8a5aa..d183b54485f41cf97775744a1b663cc2c45c01f1
--- 1/src/rgw/rgw_bucket_logging.cc
--- 2/src/rgw/rgw_bucket_logging.cc
+++ b/src/rgw/rgw_bucket_logging.cc
@@@ -478,12 -409,17 +479,17 @@@ int log_record(rgw::sal::Driver* driver
      bucket_name = s->src_bucket_name;
    } else {
      bucket_owner = to_string( s->bucket->get_owner());
 -    bucket_name = s->bucket->get_name();
 +    bucket_name = full_bucket_name(s->bucket);
    }
  
+   using namespace rgw::auth::s3;
+   string aws_version("-");
+   string auth_type("-");
+   rgw::auth::s3::get_aws_version_and_auth_type(s, aws_version, auth_type);
+ 
    switch (conf.logging_type) {
      case LoggingType::Standard:
-       record = fmt::format("{} {} [{:%d/%b/%Y:%H:%M:%S %z}] {} {} {} {} {} \"{} {}{}{} HTTP/1.1\" {} {} {} {} {} {} {} {} {} {} {} {} {} {} {} {} {}",
+       record = fmt::format("{} {} [{:%d/%b/%Y:%H:%M:%S %z}] {} {} {} {} {} \"{} {}{}{} HTTP/1.1\" {} {} {} {} {} {} {} \"{}\" {} {} {} {} {} {} {} {} {}",
          dash_if_empty(bucket_owner),
          dash_if_empty(bucket_name),
          t,