Skip to content

Conversation

@nfalco79
Copy link
Member

No description provided.

@nfalco79 nfalco79 requested a review from a team as a code owner June 24, 2025 08:30
@nfalco79 nfalco79 force-pushed the feature/standardize_code branch 2 times, most recently from c315987 to ab9912d Compare June 24, 2025 08:46
* @return an validation form for the given npmrc file identifier.
*/
public FormValidation doCheckConfigId(@Nullable @AncestorInPath ItemGroup<?> context, @CheckForNull @QueryParameter final String configId) {
public FormValidation doCheckConfigId(@Nullable @AncestorInPath ItemGroup<?> context,

Check warning

Code scanning / Jenkins Security Scan

Stapler: Missing POST/RequirePOST annotation Warning

Potential CSRF vulnerability: If DescriptorImpl#doCheckConfigId connects to user-specified URLs, modifies state, or is expensive to run, it should be annotated with @POST or @RequirePOST
* @return an validation form for the given npmrc file identifier.
*/
public FormValidation doCheckConfigId(@Nullable @AncestorInPath ItemGroup<?> context, @CheckForNull @QueryParameter final String configId) {
public FormValidation doCheckConfigId(@Nullable @AncestorInPath ItemGroup<?> context,

Check warning

Code scanning / Jenkins Security Scan

Stapler: Missing permission check Warning

Potential missing permission check in DescriptorImpl#doCheckConfigId
@nfalco79 nfalco79 force-pushed the feature/standardize_code branch from ab9912d to b68e2ed Compare June 24, 2025 08:49
@nfalco79 nfalco79 merged commit 2580545 into master Jun 24, 2025
17 checks passed
@nfalco79 nfalco79 deleted the feature/standardize_code branch June 24, 2025 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant