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

♻️ Refactor code. 重构插件支持钉钉机器人 #97

Merged
merged 1 commit into from
Feb 24, 2024
Merged

Conversation

721806280
Copy link
Owner

No description provided.

@721806280 721806280 merged commit b70c144 into main Feb 24, 2024
1 check passed
@721806280 721806280 deleted the 2.0.0 branch February 24, 2024 07:44
* @param secret The encryption key.
* @return Returns the test result. If the test passes, it returns FormValidation.respond(Kind.OK); otherwise, it returns an error message.
*/
public FormValidation doTest(@QueryParameter("id") String id, @QueryParameter("name") String name,

Check warning

Code scanning / Jenkins Security Scan

Stapler: Missing POST/RequirePOST annotation Warning

Potential CSRF vulnerability: If LarkRobotConfigDescriptor#doTest connects to user-specified URLs, modifies state, or is expensive to run, it should be annotated with @POST or @RequirePOST
* @param value Webhook key
* @return Validation result, returns FormValidation.ok() if validation passes, otherwise returns an error message
*/
public FormValidation doCheckWebhook(@QueryParameter String value) {

Check warning

Code scanning / Jenkins Security Scan

Stapler: Missing POST/RequirePOST annotation Warning

Potential CSRF vulnerability: If LarkRobotConfigDescriptor#doCheckWebhook connects to user-specified URLs, modifies state, or is expensive to run, it should be annotated with @POST or @RequirePOST
/**
* The unique key associated with the image. This key is typically used to fetch or reference the image.
*/
private String imgKey;

Check warning

Code scanning / Jenkins Security Scan

Jenkins: Plaintext password storage Warning

Field should be reviewed whether it stores a password and is serialized to disk: imgKey
/**
* Security key used for authentication.
*/
private String keys;

Check warning

Code scanning / Jenkins Security Scan

Jenkins: Plaintext password storage Warning

Field should be reviewed whether it stores a password and is serialized to disk: keys
public static class ImageContent {

@JsonProperty("image_key")
private String imageKey;

Check warning

Code scanning / Jenkins Security Scan

Jenkins: Plaintext password storage Warning

Field should be reviewed whether it stores a password and is serialized to disk: imageKey
/**
* The image key to be displayed in an IMAGE message.
*/
private String imageKey;

Check warning

Code scanning / Jenkins Security Scan

Jenkins: Plaintext password storage Warning

Field should be reviewed whether it stores a password and is serialized to disk: imageKey
* @param secret The encryption key.
* @return Returns the test result. If the test passes, it returns FormValidation.respond(Kind.OK); otherwise, it returns an error message.
*/
public FormValidation doTest(@QueryParameter("id") String id, @QueryParameter("name") String name,

Check warning

Code scanning / Jenkins Security Scan

Stapler: Missing permission check Warning

Potential missing permission check in LarkRobotConfigDescriptor#doTest
* @param value Webhook key
* @return Validation result, returns FormValidation.ok() if validation passes, otherwise returns an error message
*/
public FormValidation doCheckWebhook(@QueryParameter String value) {

Check warning

Code scanning / Jenkins Security Scan

Stapler: Missing permission check Warning

Potential missing permission check in LarkRobotConfigDescriptor#doCheckWebhook
* @param value Robot name
* @return Validation result, returns FormValidation.ok() if validation passes, otherwise returns an error message
*/
public FormValidation doCheckName(@QueryParameter String value) {

Check warning

Code scanning / Jenkins Security Scan

Stapler: Missing permission check Warning

Potential missing permission check in LarkRobotConfigDescriptor#doCheckName
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.

1 participant