Skip to content

Form validation#5

Open
vwagh-dev wants to merge 6 commits intomainfrom
level2
Open

Form validation#5
vwagh-dev wants to merge 6 commits intomainfrom
level2

Conversation

@Symbol("OnboardingPlugin")
public class OnboardingPluginGlobalConfiguration extends GlobalConfiguration {

public FormValidation doCheckName(@QueryParameter String name) {

Check warning

Code scanning / Jenkins Security Scan

Stapler: Missing permission check

Potential missing permission check in OnboardingPluginGlobalConfiguration#doCheckName
@Symbol("OnboardingPlugin")
public class OnboardingPluginGlobalConfiguration extends GlobalConfiguration {

public FormValidation doCheckName(@QueryParameter String name) {

Check warning

Code scanning / Jenkins Security Scan

Stapler: Missing POST/RequirePOST annotation

Potential CSRF vulnerability: If OnboardingPluginGlobalConfiguration#doCheckName connects to user-specified URLs, modifies state, or is expensive to run, it should be annotated with @POST or @RequirePOST
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