Skip to content

[8.19](backport #45100) [Heartbeat] Add base64 encoding option to inline monitors#45257

Merged
emilioalvap merged 2 commits into8.19from
mergify/bp/8.19/pr-45100
Jul 10, 2025
Merged

[8.19](backport #45100) [Heartbeat] Add base64 encoding option to inline monitors#45257
emilioalvap merged 2 commits into8.19from
mergify/bp/8.19/pr-45100

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Jul 9, 2025

Proposed commit message

Add encoding: base64 option to inline monitors. This change allows inline monitors to support characters that would otherwise break yaml format.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

How to test this PR locally

  1. Build heartbeat locally
  2. Specify a base64-encoded inline script monitor.
  3. Execute heartbeat and check monitor is decoded at setup.

Related issues

This is an automatic backport of pull request #45100 done by [Mergify](https://mergify.com).
@emilioalvap @mergify
[Heartbeat] Add base64 encoding option to inline monitors (#45100)
70c9f95 
* Add base64 encoding option to inline monitors

* Fix linter

* Fix linter

* Remove unused Decode() method

* Fix linter

* Add changelog

(cherry picked from commit 2266395)
@mergify mergify bot requested a review from a team as a code owner July 9, 2025 15:04
@mergify mergify bot added the backport label Jul 9, 2025
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jul 9, 2025
@mergify mergify bot mentioned this pull request Jul 9, 2025
Merged
6 tasks
@github-actions github-actions bot added bug Team:obs-ds-hosted-services Label for the Observability Hosted Services team labels Jul 9, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Jul 9, 2025

Pinging @elastic/obs-ds-hosted-services (Team:obs-ds-hosted-services)
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jul 9, 2025
mcapell
mcapell approved these changes Jul 10, 2025
View reviewed changes
@emilioalvap @mcapell
Update CHANGELOG.next.asciidoc
19ec632 
Co-authored-by: Marc Capell <marc.capell@gmail.com>
@emilioalvap emilioalvap enabled auto-merge (squash) July 10, 2025 09:52
@emilioalvap emilioalvap merged commit e2eada2 into 8.19 Jul 10, 2025
26 checks passed
@emilioalvap emilioalvap deleted the mergify/bp/8.19/pr-45100 branch July 10, 2025 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport bug Team:obs-ds-hosted-services Label for the Observability Hosted Services team

3 participants

@elasticmachine @mcapell @emilioalvap