Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JENKINS-63254][JENKINS-47101] Watch sensitive variables #105

Merged
merged 14 commits into from
Oct 14, 2020

Conversation

car-roll
Copy link
Contributor

@car-roll car-roll commented Jul 28, 2020

JENKINS-63254

PoC version 2 for groovy string interpolation interception.
Relies on changes to EnvironmentExpander. Unlike version 1, does not require any changes to core.

Requires: jenkinsci/workflow-step-api-plugin#57
Upstream: jenkinsci/workflow-cps-plugin#370

TODO:
- Add unit test(s)

CC @dwnusbaum @bitwiseman

@car-roll car-roll changed the title Binding callback [JENKINS-63254] Binding callback Jul 31, 2020
@car-roll car-roll changed the title [JENKINS-63254] Binding callback [JENKINS-63254] Watch sensitive variables Jul 31, 2020
@car-roll car-roll closed this Aug 4, 2020
@car-roll car-roll reopened this Aug 4, 2020
@car-roll car-roll closed this Aug 4, 2020
@car-roll car-roll reopened this Aug 4, 2020
@car-roll car-roll closed this Aug 5, 2020
@car-roll car-roll reopened this Aug 5, 2020
@car-roll car-roll closed this Aug 5, 2020
@car-roll car-roll reopened this Aug 5, 2020
@jglick jglick self-requested a review August 19, 2020 16:53
@car-roll car-roll marked this pull request as ready for review September 15, 2020 17:13
@car-roll car-roll closed this Sep 15, 2020
@car-roll car-roll reopened this Sep 15, 2020
@car-roll car-roll changed the title [JENKINS-63254] Watch sensitive variables [JENKINS-63254][JENKINS-47101] Watch sensitive variables Sep 29, 2020
pom.xml Outdated Show resolved Hide resolved
@car-roll car-roll requested a review from jglick October 14, 2020 19:53
pom.xml Show resolved Hide resolved
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.

4 participants