Skip to content
This repository was archived by the owner on Jan 21, 2026. It is now read-only.

Return the patched promise, not the original one#1470

Merged
punya merged 1 commit intogoogleapis:mainfrom
punya:fix-promise-patching
Aug 29, 2022
Merged

Return the patched promise, not the original one#1470
punya merged 1 commit intogoogleapis:mainfrom
punya:fix-promise-patching

Conversation

@punya
Copy link
Copy Markdown
Contributor

@punya punya commented Aug 29, 2022

Fixes #1319

@punya punya requested review from a team and aabmass August 29, 2022 01:17
@product-auto-label product-auto-label Bot added size: xs Pull request size is extra small. api: cloudtrace Issues related to the googleapis/cloud-trace-nodejs API. size: s Pull request size is small. and removed size: xs Pull request size is extra small. labels Aug 29, 2022
@generated-files-bot
Copy link
Copy Markdown

Warning: This pull request is touching the following templated files:

  • .github/workflows/ci.yaml - .github/workflows/ci.yaml (GitHub Actions) should be updated in synthtool

@product-auto-label product-auto-label Bot added size: xs Pull request size is extra small. and removed size: s Pull request size is small. labels Aug 29, 2022
@punya punya added the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 29, 2022
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 29, 2022
This ensures that asynchronous work done by patching gets awaited
properly. In particular, it prevents unhandled promise rejections.

Fixes #1319.
@punya punya added the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 29, 2022
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 29, 2022
@punya punya merged commit e79407e into googleapis:main Aug 29, 2022
@punya punya deleted the fix-promise-patching branch August 29, 2022 16:39
gcf-merge-on-green Bot pushed a commit that referenced this pull request Aug 29, 2022
🤖 I have created a release *beep* *boop*
---


## [7.1.1](v7.1.0...v7.1.1) (2022-08-29)


### Bug Fixes

* **pg:** Return patched promise instead of original ([#1470](#1470)) ([e79407e](e79407e))
* **redis:** Apply instrumentation only for Redis <4.0 ([b75f5d8](b75f5d8))
* remove pip install statements ([#1546](#1546)) ([#1468](#1468)) ([fbcc88c](fbcc88c))
* Remove unused type arg ([#1467](#1467)) ([ed422ed](ed422ed))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: cloudtrace Issues related to the googleapis/cloud-trace-nodejs API. size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Postgres unhandled promise rejection

2 participants