Skip to content

Conversation

@cjihrig
Copy link
Contributor

This commit tests the scenario where a dgram socket closes during a call to Socket#bind().

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

test

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Feb 14, 2017
@cjihrigcjihrig added the dgram Issues and PRs related to the dgram subsystem / UDP. label Feb 14, 2017
Copy link
Member

@TrottTrott left a comment

Choose a reason for hiding this comment

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

LGTM

@targos
Copy link
Member

This commit tests the scenario where a dgram socket closes during a call to Socket#bind(). PR-URL: nodejs#11383 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
@cjihrigcjihrig merged commit a196895 into nodejs:masterFeb 16, 2017
@cjihrigcjihrig deleted the close-during-bind branch February 16, 2017 18:12
italoacasas pushed a commit to italoacasas/node that referenced this pull request Feb 20, 2017
This commit tests the scenario where a dgram socket closes during a call to Socket#bind(). PR-URL: nodejs#11383 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
italoacasas pushed a commit that referenced this pull request Feb 22, 2017
This commit tests the scenario where a dgram socket closes during a call to Socket#bind(). PR-URL: #11383 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
@italoacasasitaloacasas mentioned this pull request Feb 25, 2017
@jasnell
Copy link
Member

Requires the common.mustNotCall() function to be backported to v6 and v4 before this can be backported.

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

Labels

dgramIssues and PRs related to the dgram subsystem / UDP.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@cjihrig@targos@jasnell@Trott@nodejs-github-bot