Skip to content

Commit

Permalink
Merge pull request #2694 from mcb30/s3_no_explicit_default_port
Browse files Browse the repository at this point in the history
Do not include an explicit default port number within S3 upload URI
  • Loading branch information
jblomer committed Mar 22, 2021
2 parents 4c9b753 + 654d49d commit c3cc41d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cvmfs/upload_s3.cc
Expand Up @@ -120,7 +120,7 @@ bool S3Uploader::ParseSpoolerDefinition(
if (options_manager.GetValue("CVMFS_S3_PORT", &parameter)) {
host_name_port_ = host_name_ + ":" + parameter;
} else {
host_name_port_ = host_name_ + ":" + StringifyInt(kDefaultPort);
host_name_port_ = host_name_;
}

if (!options_manager.GetValue("CVMFS_S3_ACCESS_KEY", &access_key_)) {
Expand Down

0 comments on commit c3cc41d

Please sign in to comment.