Skip to content

Add support for configuring gRPC metadata#956

Merged
electron0zero merged 4 commits into
prometheus:masterfrom
averzicco:grpc-metadata
Dec 1, 2025
Merged

Add support for configuring gRPC metadata#956
electron0zero merged 4 commits into
prometheus:masterfrom
averzicco:grpc-metadata

Conversation

@averzicco
Copy link
Copy Markdown
Contributor

@averzicco averzicco commented Aug 7, 2022

What this PR does / Which issue(s) does the PR fix:

Hi, I've noticed there is an issue to request support for gRPC metadata (#946), so this PR is to add that support.

Fixes: #946

Does this PR introduce a user-facing change?

[FEATURE] Add support for configuring metadata in gRPC prober

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG added in release-notes section of PR Desc.

@aksenk
Copy link
Copy Markdown

aksenk commented Apr 26, 2023

Hi!

We also need this functionality. We can't use grpc prober with authorization without metadata (need headers).
Are you planning merge this?

Thanks.

@damoun
Copy link
Copy Markdown

damoun commented May 3, 2024

Hey 👋🏻 Any update on the review of this PR ? It would be useful for authentication.

@github-actions github-actions Bot added the stale label Sep 20, 2024
Comment thread config/config.go Outdated
Comment thread config/config.go
Comment thread prober/grpc_test.go Outdated
Comment thread CONFIGURATION.md Outdated
@electron0zero
Copy link
Copy Markdown
Member

please rebase with master as well, and resolve conflicts.

Signed-off-by: Alessandro Verzicco <8068317+averzicco@users.noreply.github.com>
Signed-off-by: averzicco <8068317+averzicco@users.noreply.github.com>
Signed-off-by: Alessandro Verzicco <8068317+averzicco@users.noreply.github.com>
Signed-off-by: Alessandro Verzicco <8068317+averzicco@users.noreply.github.com>
Signed-off-by: Alessandro Verzicco <8068317+averzicco@users.noreply.github.com>
Comment thread prober/grpc_test.go Outdated
Co-authored-by: Suraj Nath <9503187+electron0zero@users.noreply.github.com>
Signed-off-by: Alessandro Verzicco <8068317+averzicco@users.noreply.github.com>
Copy link
Copy Markdown
Member

@electron0zero electron0zero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks :)

@electron0zero
Copy link
Copy Markdown
Member

waiting to get more comments from other maintainers. feel free to ping me to merge this, if there are no comments in few weeks.

@averzicco
Copy link
Copy Markdown
Contributor Author

@electron0zero I guess now it's possible to merge this PR

@electron0zero electron0zero merged commit 33a1803 into prometheus:master Dec 1, 2025
5 checks passed
@jesusvazquez jesusvazquez mentioned this pull request Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Does blackbox_exporter support gRPC metadata?

4 participants