Skip to content

CXXCBC-633: ensure DNS-SRV resolver reports timeouts#703

Merged
avsej merged 1 commit intocouchbase:mainfrom
avsej:CXXCBC-633-fix-timeouts-in-DNS-SRV-resolver
Dec 9, 2024
Merged

CXXCBC-633: ensure DNS-SRV resolver reports timeouts#703
avsej merged 1 commit intocouchbase:mainfrom
avsej:CXXCBC-633-fix-timeouts-in-DNS-SRV-resolver

Conversation

@avsej
Copy link
Copy Markdown
Member

@avsej avsej commented Dec 9, 2024

  • catch asio::error::operation_aborted in every handler, and return immediately giving chance to deadline_ handler to report timeout error code

  • make sure that UDP round will always fallback to TCP in case of UDP timeout.

* catch asio::error::operation_aborted in every handler, and return
  immediately giving chance to deadline_ handler to report timeout error
  code

* make sure that UDP round will always fallback to TCP in case of UDP
  timeout.
@avsej avsej requested review from a team, mikereiche and thejcfactor December 9, 2024 18:28
Copy link
Copy Markdown
Contributor

@thejcfactor thejcfactor left a comment

Choose a reason for hiding this comment

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

Nice! Checked out in the Node.js client. Much better now. 👍

@avsej avsej merged commit fe4f36f into couchbase:main Dec 9, 2024
@avsej avsej deleted the CXXCBC-633-fix-timeouts-in-DNS-SRV-resolver branch December 9, 2024 19:55
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.

2 participants