Skip to content

A Flag to deactivate the Escaping for Enviroment Variables#958

Open
Taikobo2 wants to merge 5 commits into
puppetlabs:mainfrom
Taikobo2:Escape_enviroment_flag
Open

A Flag to deactivate the Escaping for Enviroment Variables#958
Taikobo2 wants to merge 5 commits into
puppetlabs:mainfrom
Taikobo2:Escape_enviroment_flag

Conversation

@Taikobo2

Copy link
Copy Markdown

Summary

We created a flag to disable Escaping just for the Enviroment

Additional Context

Add any additional context about the problem here.

  • Our DOCKER_GID is dynamically set. With the Escaping the shell command does not work
  • Thats why we just wanted to Just not Escape the Enviroment.

Related Issues (if any)

The Pullrequest were the Implementation of the Escaping was:
#869

Checklist

  • I tested with and without the flag , locally in my puppet docker enviroment, and it worked.

@Taikobo2 Taikobo2 requested a review from a team as a code owner October 20, 2023 09:40
Comment thread manifests/run.pp Outdated
@ghost

ghost commented Aug 27, 2024

Copy link
Copy Markdown

Any chance moving forward with this?

@jst-cyr

jst-cyr commented Jun 24, 2026

Copy link
Copy Markdown
Member

@Taikobo2 : My apologies that nobody has responded on this for so much time. I'm going through our pull requests across modules and came across this request here. Is this still an issue for you?

@Taikobo2

Taikobo2 commented Jun 25, 2026 via email

Copy link
Copy Markdown
Author

@jst-cyr

jst-cyr commented Jun 25, 2026

Copy link
Copy Markdown
Member

Thank you for confirming @Taikobo2 ! I'll add this to our list of issues to review. We may need your help updating the PR with the latest from main once we get to this PR, but no need to take any actions yet!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants