Skip to content

Conversation

@killagu
Copy link
Contributor

@killagu killagu commented Oct 23, 2025

No description provided.

@fengmk2 fengmk2 requested a review from Copilot October 23, 2025 11:35
Copy link
Member

@fengmk2 fengmk2 left a comment

Choose a reason for hiding this comment

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

先不合并,我需要先将 auto release 修复。

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds error handling to prevent unhandled promise rejections when a connection timeout occurs. The change ensures that if a connection promise rejects after a timeout has already been thrown, the rejection is caught and silently ignored.

Key Changes

  • Added a .catch() handler to the connection promise in the timeout error path to prevent unhandled rejection errors

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@socket-security
Copy link

socket-security bot commented Oct 23, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedonelogger@​1.0.1931009278100

View full report

@fengmk2 fengmk2 enabled auto-merge October 23, 2025 12:24
@fengmk2 fengmk2 added this pull request to the merge queue Oct 23, 2025
Merged via the queue into ali-sdk:master with commit 897c866 Oct 23, 2025
8 checks passed
fengmk2 pushed a commit that referenced this pull request Oct 23, 2025
[skip ci]

## <small>6.6.1 (2025-10-23)</small>

* fix: add catch for get conn exception (#123) ([897c866](897c866)), closes [#123](#123)
* chore: enable auto merge ([a5e452e](a5e452e))
* chore: fix release workflow yml ([4459896](4459896))
@github-actions
Copy link

🎉 This PR is included in version 6.6.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants