Open uricorin opened 5 days ago
Thanks for opening this request @uricorin ! I agree it would be useful functionality. Rather than adding a new config option, all_qp
, I'd suggest we model it like headers
which accepts wildcards so that you could just configure it like:
query_paramaters:
- '*'
Happy to see a PR!
Use cases
Hi everyone. We're using vector to aggregate HTTP requests. Currently, query parameters must be specified, by name, or they won't be outputted.
There's no easy way to support requests with varying arguments.
References
Discussion on a similar issue #16396
Solution
I've updated the code to output all query parameters, as a string, to field
query_parameters_string
. A configuration variable,all_qp
, enables this functionality - It's off by default.Here's an example configuration:
Proposal
I'd like to contribute this change back to the upstream project. If you approve opening a PR I'd appreciate suggestions regarding functionality and testing.
Notes
The Heroku Logplex and http server sources reuse functions so this change will add the capability to both.