Skip to content

Conversation

@MattDevy
Copy link
Contributor

@MattDevy MattDevy commented Nov 6, 2025

Adds a Close method to the transport which cleans up resources.

  • Closes the client gracefully
    • Cancels running and scheduled background DiscoverNodes calls
    • Cancels scheduled connection resurrections
  • Forcefully stops the client on context timeout
  • Sets a timeout for DiscoverNodesContext calls from within scheduleDiscoverNodes loop to be equal to the DiscoverNodesInterval. DiscoverNodeTimeout is still respected

@MattDevy MattDevy requested a review from Anaethelion November 6, 2025 12:35
@prodsecmachine
Copy link

prodsecmachine commented Nov 6, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Licenses 0 0 0 0 0 issues
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@MattDevy MattDevy added the enhancement New feature or request label Nov 6, 2025
* Closes the client gracefully
* Forcefully stops the client on context timeout
@MattDevy MattDevy merged commit b2d94de into elastic:main Nov 7, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants