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-21327] replace RUN_SCRIPTS with a new Scriptler RUN permission #29

Closed
wants to merge 2 commits into from

Conversation

imod
Copy link
Member

@imod imod commented May 15, 2017

this should replace the use of RUN_SCRIPT with a new Scriptler specific RUN permisson

@imod imod requested review from jglick and daniel-beck May 15, 2017 18:03
@daniel-beck daniel-beck self-assigned this May 15, 2017
Copy link
Member

@jglick jglick left a comment

Choose a reason for hiding this comment

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

A new permission? This just seems even more confusing than before. I guess I do not follow where you are going with this.

tokenmacro_ScriptDoesNotExist = No script with the id [{0}] could be found.
permissons_title = Scriptler
permissons_run_description = allow execution of scritpler scripts
Copy link
Member

Choose a reason for hiding this comment

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

typo

@@ -65,6 +68,9 @@
private final static String ALL = "(all)";
private final static String ALL_SLAVES = "(all slaves)";

public static final PermissionGroup SCRITPLER_PERMISSONS = new PermissionGroup(ScriptlerManagement.class, Messages._permissons_title());
Copy link
Member

Choose a reason for hiding this comment

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

typo

@imod
Copy link
Member Author

imod commented May 16, 2017

@jglick hmm, from my POV this is logical consequence if I want to allow some users (non admins) to execute/use scriptler scripts. RUN_SCRIPT obviously was wrong, but I don't quite understand your confusion with the new independent permission, can you explain?

@imod
Copy link
Member Author

imod commented May 31, 2017

@daniel-beck do you understand what @jglick is talking about? I really don't get it anymore some more context information in the answers would really help...

@daniel-beck
Copy link
Member

Probably best to try to make scripts AccessControlled and implement properly fine-grained permissions control if this is the route you want to go.

I hope I'll have the time in the next two weeks to investigate this further.

@daniel-beck
Copy link
Member

Superseded by #32, I think.

@daniel-beck daniel-beck removed their assignment Jan 22, 2018
@imod
Copy link
Member Author

imod commented Jan 24, 2018

closing because of #32

@imod imod closed this Jan 24, 2018
@imod imod deleted the JENKINS-21327_scriptler_run_permisson branch September 18, 2019 06:23
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.

3 participants