nodejs-github-bot · GitHub

@nodejs-github-bot added the dgram

Issues and PRs related to the dgram subsystem / UDP.

label

Dec 27, 2016

@boneskull

@boneskull boneskull changed the title dgram: improve Socket docs and invalid arg behavior dgram: improve "address" parameter behavior in Socket.prototype.send

Jan 9, 2017

mscdex

mscdex

sam-github

@sam-github sam-github added the semver-minor

PRs that contain new features and should be released in the next minor version.

label

Jan 11, 2017

@sam-github sam-github added semver-major

PRs that contain breaking changes and should be released in the next major version.

and removed semver-minor

PRs that contain new features and should be released in the next minor version.

labels

Jan 11, 2017

@boneskull

sam-github

@boneskull

- Add coverage around valid, undocumented types for `address` parameter.
- Add coverage around known invalid, but uncovered, types for `address`
  parameter.

@boneskull

thefourtheye

thefourtheye

thefourtheye

- Do not require presence of `address` parameter to use `callback`
  parameter; `address` is *always* optional
- Improve exception messaging if `address` is invalid type
- If `address` is an invalid type, guarantee a synchronously thrown
  exception
- Update documentation to reflect signature changes

@boneskull

cjihrig

Trott pushed a commit to Trott/io.js that referenced this pull request

Feb 17, 2017
- Do not require presence of `address` parameter to use `callback`
  parameter; `address` is *always* optional
- Improve exception messaging if `address` is invalid type
- If `address` is an invalid type, guarantee a synchronously thrown
  exception
- Update documentation to reflect signature changes
- Add coverage around valid, undocumented types for `address` parameter.
- Add coverage around known invalid, but uncovered, types for `address`
  parameter.
PR-URL: nodejs#10473
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

Closed

@Trott Trott mentioned this pull request

Dec 19, 2017

Closed

Read the original on github.com ↗