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

Enable issue reporter configs. #1

Merged
merged 1 commit into from
Aug 28, 2024
Merged

Conversation

random1223
Copy link
Contributor

No description provided.

}

public void setEnableConsoleIssueReporter(boolean enableConsoleIssueReporter) {
this.enableConsoleIssueReporter = enableConsoleIssueReporter;

Choose a reason for hiding this comment

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

Codety
CodetyBot:

  • 🔴 [security] [vault-token] Detected potential exposure of sensitive information

if(analyzerRequest.enableSlackNotification) {
analyzerRequest.slackOauthToken = (envMap.get(CodetyConstant.ENV_SLACK_OAUTH_TOKEN));
analyzerRequest.slackConversationId = (envMap.get(CodetyConstant.ENV_SLACK_CONVERSATION_ID));
}

analyzerRequest.enableConsoleIssueReporter = !"false".equals(envMap.get(CodetyConstant.ENV_CODETY_ISSUE_REPORTER_CONSOLE));

Choose a reason for hiding this comment

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

Codety
CodetyBot:

  • 🔴 [Error Prone] [AvoidDuplicateLiterals] The String literal 'false' appears 4 times in this file; the first occurrence is on line 95

Copy link

Codety
Issues found in this pull request based on Codety's default factory settings:

Java source code

File Issue
..io/codety/scanner/service/dto/AnalyzerRequest.java 🔴 [Error Prone] [AvoidDuplicateLiterals] The String literal 'false' appears 4 times in this file; the first occurrence is on line 95 view

Secrets detection

File Issue
..io/codety/scanner/service/dto/AnalyzerRequest.java 🔴 [security] [vault-token] Detected potential exposure of sensitive information view

The above content will be updated automatically after a scan pushed into this pull request.

@random1223 random1223 merged commit f163160 into main Aug 28, 2024
1 check passed
@random1223 random1223 deleted the enable_issue_reporter_configs branch August 28, 2024 05:49
random1223 added a commit that referenced this pull request Oct 13, 2024
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