vector icon indicating copy to clipboard operation
vector copied to clipboard

feat(http_client source): #23338 Enable AWS authentication for the http_client source.

Open johannesfloriangeiger opened this issue 5 months ago • 2 comments

Summary

See title, enables AWS authentication for the http_client source.

Vector configuration

[sources.http_client]
type = "http_client"
endpoint = "<ENDPOINT>"
auth = {strategy = "aws", auth = {}, service = "aps"}
method = "POST"
headers.Content-Type = ["application/x-www-form-urlencoded"]
query.query = "up"

[sinks.blackhole]
type = "blackhole"
inputs = ["http_client"]

How did you test this PR?

Manually. Get yourself an AWS Account, create an Amazon Managed Service for Prometheus workspace, get the Endpoint - query URL and use it in the config above. Run Vector and see it working (and not failing with a 403 like the current version).

Change Type

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

Is this a breaking change?

  • [ ] Yes
  • [X] No

Does this PR include user facing changes?

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

References

  • Closes: #23338

Notes

  • Please read our Vector contributor resources.
  • Do not hesitate to use @vectordotdev/vector to reach out to us regarding this PR.
  • Some CI checks run only after we manually approve them.
    • We recommend adding a pre-push hook, please see this template.
    • Alternatively, we recommend running the following locally before pushing to the remote branch:
      • cargo fmt --all
      • cargo clippy --workspace --all-targets -- -D warnings
      • cargo nextest run --workspace (alternatively, you can run cargo test --all)
  • After a review is requested, please avoid force pushes to help us review incrementally.
    • Feel free to push as many commits as you want. They will be squashed into one before merging.
    • For example, you can run git merge origin master and git push.
  • If this PR introduces changes Vector dependencies (modifies Cargo.lock), please run cargo vdev build licenses to regenerate the license inventory and commit the changes (if any). More details here.

johannesfloriangeiger avatar Jul 04 '25 14:07 johannesfloriangeiger

Please resolve the merge conflicts and we will take a look

pront avatar Aug 27 '25 17:08 pront

Please resolve the merge conflicts and we will take a look

Thanks, merged master back into my branch - the reorg of the imports has broken git, I try to keep changes to existing code to a minimum in a PR.

johannesfloriangeiger avatar Sep 24 '25 10:09 johannesfloriangeiger