Skip to content
This repository was archived by the owner on Jul 28, 2026. It is now read-only.

add sigv4 as a custom authentication method for remote_write data - #322

Merged
rfratto merged 5 commits into
grafana-cold-storage:masterfrom
rfratto:sigv4
Jan 13, 2021
Merged

add sigv4 as a custom authentication method for remote_write data#322
rfratto merged 5 commits into
grafana-cold-storage:masterfrom
rfratto:sigv4

Conversation

@rfratto

@rfratto rfratto commented Jan 11, 2021

Copy link
Copy Markdown
Contributor

PR Description

This PR adds a SigV4 mechanism for authenticating remote_write requests.

Which issue(s) this PR fixes

Notes to the Reviewer

PR Checklist

  • CHANGELOG updated
  • Documentation added
  • Tests updated

Comment thread pkg/prom/ha/codec_test.go Outdated
Comment thread pkg/prom/instance/instance.go
Comment thread pkg/prom/instance/instance.go
Comment thread pkg/prom/instance/remote_write.go
Comment thread pkg/prom/instance/sigv4.go Outdated
Comment thread pkg/prom/instance/sigv4.go Outdated
@rfratto
rfratto marked this pull request as ready for review January 12, 2021 19:31
@rfratto
rfratto requested a review from mattdurham January 12, 2021 19:32
@rfratto

rfratto commented Jan 12, 2021

Copy link
Copy Markdown
Contributor Author

@mattdurham the blast radius here is pretty big thanks to wrapping the remote write struct, let me know if we need to go over this together

Comment thread docs/configuration-reference.md
Comment thread docs/configuration-reference.md
@rfratto
rfratto merged commit 4a09b67 into grafana-cold-storage:master Jan 13, 2021
@rfratto
rfratto deleted the sigv4 branch January 13, 2021 14:20
@mattdurham mattdurham mentioned this pull request Sep 7, 2021
3 tasks
mattdurham pushed a commit that referenced this pull request Nov 11, 2021
* add sigv4 as a custom authentication method for remote_write data

* address review feedback i gave myself

* require region

* docs, changelog

* make region requirements more clear
@github-actions github-actions Bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Apr 21, 2024
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants