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

Added help for GROOVY token macro #18

merged 1 commit into from Jul 26, 2015


None yet
3 participants
Copy link

patbos commented Jul 26, 2015

If token macro processing via Token Macro Plugin is allowed, the evaluation of macro is done in System Groovy, therefore any user can run arbitrary system script, regardless he has administer permission!

This comment has been minimized.


KostyaSha Jul 26, 2015


Why not just use static html?

This comment has been minimized.


KostyaSha Jul 26, 2015


new line

This comment has been minimized.


patbos Jul 26, 2015

Author Contributor

Needs to be jelly for some reason, tried html but it did not work.

@patbos patbos force-pushed the patbos:help-token-macro branch from f0abc59 to d7d5e3d Jul 26, 2015

olivergondza added a commit that referenced this pull request Jul 26, 2015

Merge pull request #18 from patbos/help-token-macro
Added help for GROOVY token macro

@olivergondza olivergondza merged commit bf27542 into jenkinsci:master Jul 26, 2015

1 check passed

Jenkins This pull request looks good

@patbos patbos deleted the patbos:help-token-macro branch Aug 3, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.