feat(aws_s3 source): Support vhost-style S3 bucket addressing

[sbalmos]

Summary

This adds support for vhost-style S3 bucket addressing in the aws_s3 source, equivalent to the support added in the aws_s3 sink in #21999.

Change Type

• Bug fix
• New feature
• Non-functional (chore, refactoring, docs)
• Performance

Is this a breaking change?

• Yes
• No

How did you test this PR?

Does this PR include user facing changes?

• Yes. Please add a changelog fragment based on our guidelines.
• No. A maintainer will apply the "no-changelog" label to this PR.

Checklist

• Please read our Vector contributor resources.
  • make check-all is a good command to run locally. This check is
    defined here. Some of these
    checks might not be relevant to your PR. For Rust changes, at the very least you should run:
    • cargo fmt --all
    • cargo clippy --workspace --all-targets -- -D warnings
    • cargo nextest run --workspace (alternatively, you can run cargo test --all)
• If this PR introduces changes Vector dependencies (modifies Cargo.lock), please
  run dd-rust-license-tool write to regenerate the license inventory and commit the changes (if any). More details here.

References

#21999

[pront]

Nit: It would be cleaner if this was an enum. I missed this #21999.

[pront]

Example:

vector/src/sinks/elasticsearch/config.rs

Lines 42 to 53 in 78adec7

	/// The Amazon OpenSearch service type, either managed or serverless; primarily, selects the
	/// correct AWS service to use when calculating the AWS v4 signature + disables features
	/// unsupported by serverless: Elasticsearch API version autodetection, health checks
	#[configurable_component]
	#[derive(Clone, Debug, Eq, PartialEq)]
	#[serde(deny_unknown_fields, rename_all = "lowercase")]
	pub enum OpenSearchServiceType {
	/// Elasticsearch or OpenSearch Managed domain
	Managed,
	/// OpenSearch Serverless collection
	Serverless,
	}

[sbalmos]

Yeah I was going to say, it's to be consistent with what the sink does. Do you want me to cross-implement in both the source and sink?

[pront]

I think we can accept as is (with the bool) and in the future deprecate all the bools and introduce enums. You don't have to do the later, but it would be greatly appreciated if you did :)

[sbalmos]

I'll leave this change for the next version round, since the bool version was already approved for the sink, and the bool version for the source is approved here. I think if things are okay with the first version, I'll concurrently change the default on both the sink and source to use vhost format. It'd be a breaking change anyway to switch from bool to an enum, so changing the default from path to vhost is about the same level I'd think.

[pront]

Agree to keep this as is. Note here, that we can avoid breaking the behavior here by introducing new fields and deprecate the old ones. See https://github.com/vectordotdev/vector/blob/master/docs/DEPRECATION.md

[buraizu]

Approving with some suggested updates

[pront]

Good to go after https://github.com/vectordotdev/vector/actions/runs/13474840462/job/37652711395?pr=22475 is fixed. Thanks @sbalmos

[sbalmos]

The spell checker seems to be hung up on my username in the authors of the changelog fragment file, and the use of the word vhosts in the filename. Not sure if I have rights to also push a commit that adds to the exclusions?

[sbalmos]

Heh, and now the origin update, removing shannon, broke spellchecker on that word. :(