Skip to content

auth: Allow adding HTTP Headers in LUA Record ifurlup#16955

Open
pieterlexis wants to merge 3 commits intoPowerDNS:masterfrom
pieterlexis:lua-headers
Open

auth: Allow adding HTTP Headers in LUA Record ifurlup#16955
pieterlexis wants to merge 3 commits intoPowerDNS:masterfrom
pieterlexis:lua-headers

Conversation

@pieterlexis
Copy link
Contributor

Short description

This PR adds support for a headers option in the ifurlup. In order to achieve this, I added an std::variant-based method of setting options to allow arrays as values in the options, that could be useful in the future.

I can recommend reviewing on a per-commit basis.

Closes: #8295
Closes: #11610

Checklist

I have:

  • read the CONTRIBUTING.md document
  • read and accepted the Developer Certificate of Origin document, including the AI Policy, and added a "Signed-off-by" to my commits
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)

miodvallat
miodvallat previously approved these changes Mar 6, 2026
Copy link
Contributor

@miodvallat miodvallat left a comment

Choose a reason for hiding this comment

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

Why not.

Copy link
Contributor

@miodvallat miodvallat left a comment

Choose a reason for hiding this comment

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

Why not.

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.

Add arbitrary headers to ifurlup() Lua healthchecks

2 participants