Skip to content

fix(telegraf): escape dollar in here-doc to prevent parameter expansion#6659

Merged
jstirnaman merged 5 commits intoinfluxdata:masterfrom
spike77453:master
Dec 22, 2025
Merged

fix(telegraf): escape dollar in here-doc to prevent parameter expansion#6659
jstirnaman merged 5 commits intoinfluxdata:masterfrom
spike77453:master

Conversation

@spike77453
Copy link
Contributor

Escapes the dollar symbol in $basearch to prevent parameter expansion in the here-document.

Without escaping the baseurl parameter in the resulting file is malformed:

[influxdata]
name = InfluxData Repository - Stable
baseurl = https://repos.influxdata.com/stable//main
enabled = 1
gpgcheck = 1
gpgkey = file:///etc/pki/rpm-gpg/RPM-GPG-KEY-influxdata

See also:

@jstirnaman jstirnaman self-requested a review December 22, 2025 19:31
Copy link
Contributor

@jstirnaman jstirnaman left a comment

Choose a reason for hiding this comment

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

Thanks for submitting the PR!

@jstirnaman jstirnaman merged commit 8045b67 into influxdata:master Dec 22, 2025
6 of 8 checks passed
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.

2 participants