Complete notification publishing migration #1624

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.

Sign up for GitHub

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

Merged

nscuro merged 1 commit into main from notification-workflow

Jan 28, 2026

Merged

Complete notification publishing migration #1624

Complete notification publishing migration

Codacy Production / Codacy Static Code Analysis succeeded Jan 28, 2026 in 0s

Your pull request is up to standards!

Codacy found no issues in your code

Annotations

Check notice on line 270 in apiserver/src/main/java/org/dependencytrack/dex/DexEngineInitializer.java

codacy-production / Codacy Static Code Analysis

apiserver/src/main/java/org/dependencytrack/dex/DexEngineInitializer.java#L270

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 281 in apiserver/src/main/java/org/dependencytrack/dex/DexEngineInitializer.java

codacy-production / Codacy Static Code Analysis

apiserver/src/main/java/org/dependencytrack/dex/DexEngineInitializer.java#L281

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check warning on line 122 in apiserver/src/main/java/org/dependencytrack/notification/PublishNotificationActivity.java

codacy-production / Codacy Static Code Analysis

apiserver/src/main/java/org/dependencytrack/notification/PublishNotificationActivity.java#L122

An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type.

Check warning on line 132 in notification/publishing/src/main/java/org/dependencytrack/notification/publishing/kafka/KafkaNotificationPublisherFactory.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/main/java/org/dependencytrack/notification/publishing/kafka/KafkaNotificationPublisherFactory.java#L132

The method 'runtimeConfigSpec()' has an NPath complexity of 240, current threshold is 200

Check notice on line 143 in notification/publishing/src/main/java/org/dependencytrack/notification/publishing/kafka/KafkaNotificationPublisherFactory.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/main/java/org/dependencytrack/notification/publishing/kafka/KafkaNotificationPublisherFactory.java#L143

These nested if statements could be combined

Check warning on line 189 in notification/publishing/src/main/java/org/dependencytrack/notification/publishing/kafka/KafkaNotificationPublisherFactory.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/main/java/org/dependencytrack/notification/publishing/kafka/KafkaNotificationPublisherFactory.java#L189

An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type.

Check notice on line 62 in notification/publishing/src/test/java/org/dependencytrack/notification/publishing/AbstractNotificationPublisherTest.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/test/java/org/dependencytrack/notification/publishing/AbstractNotificationPublisherTest.java#L62

Document empty method body

Check notice on line 65 in notification/publishing/src/test/java/org/dependencytrack/notification/publishing/AbstractNotificationPublisherTest.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/test/java/org/dependencytrack/notification/publishing/AbstractNotificationPublisherTest.java#L65

Document empty method body

Check warning on line 40 in notification/publishing/src/test/java/org/dependencytrack/notification/publishing/console/ConsoleNotificationPublisherTest.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/test/java/org/dependencytrack/notification/publishing/console/ConsoleNotificationPublisherTest.java#L40

Switch statements should be exhaustive, add a default case (or missing enum branches)

Check warning on line 77 in notification/publishing/src/test/java/org/dependencytrack/notification/publishing/email/EmailNotificationPublisherTest.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/test/java/org/dependencytrack/notification/publishing/email/EmailNotificationPublisherTest.java#L77

Switch statements should be exhaustive, add a default case (or missing enum branches)

Check warning on line 81 in notification/publishing/src/test/java/org/dependencytrack/notification/publishing/jira/JiraNotificationPublisherTest.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/test/java/org/dependencytrack/notification/publishing/jira/JiraNotificationPublisherTest.java#L81

Switch statements should be exhaustive, add a default case (or missing enum branches)

Check warning on line 71 in notification/publishing/src/test/java/org/dependencytrack/notification/publishing/mattermost/MattermostNotificationPublisherTest.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/test/java/org/dependencytrack/notification/publishing/mattermost/MattermostNotificationPublisherTest.java#L71

Switch statements should be exhaustive, add a default case (or missing enum branches)

Check warning on line 71 in notification/publishing/src/test/java/org/dependencytrack/notification/publishing/msteams/MsTeamsNotificationPublisherTest.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/test/java/org/dependencytrack/notification/publishing/msteams/MsTeamsNotificationPublisherTest.java#L71

Switch statements should be exhaustive, add a default case (or missing enum branches)

Check warning on line 71 in notification/publishing/src/test/java/org/dependencytrack/notification/publishing/slack/SlackNotificationPublisherTest.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/test/java/org/dependencytrack/notification/publishing/slack/SlackNotificationPublisherTest.java#L71

Switch statements should be exhaustive, add a default case (or missing enum branches)

Check warning on line 71 in notification/publishing/src/test/java/org/dependencytrack/notification/publishing/webex/WebexNotificationPublisherTest.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/test/java/org/dependencytrack/notification/publishing/webex/WebexNotificationPublisherTest.java#L71

Switch statements should be exhaustive, add a default case (or missing enum branches)

Check warning on line 76 in notification/publishing/src/test/java/org/dependencytrack/notification/publishing/webhook/WebhookNotificationPublisherTest.java

codacy-production / Codacy Static Code Analysis

notification/publishing/src/test/java/org/dependencytrack/notification/publishing/webhook/WebhookNotificationPublisherTest.java#L76

Switch statements should be exhaustive, add a default case (or missing enum branches)

View more details on Codacy Production