Skip to content

cid-3709: Add socket is connected before send a webhook message#116

Closed
henriq-amaral-leanix wants to merge 1 commit intomainfrom
feature/cid-3709-fix-failure-send-message
Closed

cid-3709: Add socket is connected before send a webhook message#116
henriq-amaral-leanix wants to merge 1 commit intomainfrom
feature/cid-3709-fix-failure-send-message

Conversation

@henriq-amaral-leanix
Copy link
Contributor

🛠 Changes made

Added validation to check if agent is connected to the websocket server before send a message

✨ Type of change

  • Bug fix (non-breaking change which fixes an issue)

🧪 How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

  • 👌 Modified tests

🏎 Checklist:

  • My code follows the style guidelines
  • I have performed a self-review of my own code

@henriq-amaral-leanix henriq-amaral-leanix requested a review from a team as a code owner May 13, 2025 15:25
@henriq-amaral-leanix henriq-amaral-leanix self-assigned this May 13, 2025
@github-actions
Copy link

LeanIX GitHub Agent Code Coverage

File Coverage [81.36%] 🍏
GitHubWebhookService.kt 99.25% 🍏
WebSocketService.kt 54.02%
Total Project Coverage 67.85% 🍏

@mohamedlajmileanix mohamedlajmileanix added the enhancement New feature or request label May 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Development

Successfully merging this pull request may close these issues.

2 participants