Skip to content

EPMDEDP-17177: refactor: extract duplicated literals and hostname for matting in EventListener creation#293

Merged
SergK merged 1 commit into
masterfrom
fix-sonar-findings
Jul 7, 2026
Merged

EPMDEDP-17177: refactor: extract duplicated literals and hostname for matting in EventListener creation#293
SergK merged 1 commit into
masterfrom
fix-sonar-findings

Conversation

@SergK

@SergK SergK commented Jul 7, 2026

Copy link
Copy Markdown
Member

Extract duplicated string literals and event listener hostname formatting logic into constants and a helper function to reduce code duplication and improve maintainability. Addresses SonarQube code smell findings for the EventListener exposure code (Ingress, HTTPRoute, and Route variants).

  • Extract gitServer label string to gitServerLabel constant
  • Extract DNS wildcard error message to errGetDNSWildcard constant
  • Extract event listener hostname formatting to generateEventListenerHost() helper to centralize and prevent divergence across exposure methods

@SergK SergK requested a review from a team as a code owner July 7, 2026 14:50
…matting in EventListener creation

Extract duplicated string literals and event listener hostname formatting logic
into constants and a helper function to reduce code duplication and improve
maintainability. Addresses SonarQube code smell findings for the EventListener
exposure code (Ingress, HTTPRoute, and Route variants).

- Extract gitServer label string to gitServerLabel constant
- Extract DNS wildcard error message to errGetDNSWildcard constant
- Extract event listener hostname formatting to generateEventListenerHost()
  helper to centralize and prevent divergence across exposure methods

Signed-off-by: Sergiy Kulanov <sergiy_kulanov@epam.com>
@SergK SergK force-pushed the fix-sonar-findings branch from f2a8fae to ab02441 Compare July 7, 2026 15:02
@SergK SergK merged commit 73460eb into master Jul 7, 2026
5 checks passed
@SergK SergK deleted the fix-sonar-findings branch July 7, 2026 15:10
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