Skip to content

fix(samples): replace server-destroy with native methods#3865

Closed
raushan728 wants to merge 3 commits intogoogleapis:mainfrom
raushan728:fix/modernize-oauth2-sample
Closed

fix(samples): replace server-destroy with native methods#3865
raushan728 wants to merge 3 commits intogoogleapis:mainfrom
raushan728:fix/modernize-oauth2-sample

Conversation

@raushan728
Copy link

  • Replaced server-destroy with native server.closeAllConnections() (Node 18+).
  • Renamed opn to open for modern package naming.
  • Cleaned up unused eslint suppression in oauth2.js.

@raushan728 raushan728 requested a review from a team as a code owner January 13, 2026 13:54
@google-cla
Copy link

google-cla bot commented Jan 13, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@product-auto-label product-auto-label bot added size: s Pull request size is small. samples Issues that are directly related to samples. labels Jan 13, 2026
@quirogas
Copy link
Contributor

quirogas commented Feb 2, 2026

raushan728 I'm closing this PR since it has been open for three weeks, and you have not yet signed the CLA. For future contributions, please review our contribution guidelines.

@quirogas quirogas closed this Feb 2, 2026
@raushan728
Copy link
Author

@quirogas there must have been a misunderstanding as I signed the CLA 3 weeks ago and the ci checks passed too!

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

Labels

samples Issues that are directly related to samples. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants