FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

update https-browserify to ^1.0.0 by feross · Pull Request #1710 · browserify/browserify · GitHub

update https-browserify to ^1.0.0 - #1710

Merged
feross merged 1 commit into
masterfrom
https-browserify-1
Apr 4, 2017
Merged

feross merged 1 commit into
masterfrom
https-browserify-1

Conversation

feross commented Apr 4, 2017 •
edited
Loading

Copy link
Copy Markdown
Member

See: https://github.com/substack/https-browserify/commit/7295a7cf5c0928f8fd7e24ed8aae17801726e88b

Match Node.js behavior exactly

  • Remove unused 'scheme' property
  • Throw exception if non-https URL is passed in
  • Support string URL arguments
  • First argument is always required, so remove code that sets default
    object if params doesn't exist

See:
https://github.com/substack/https-browserify/commit/7295a7cf5c0928f8fd7e
24ed8aae17801726e88b

Match Node.js behavior exactly

- Remove unused 'scheme' property
- Throw exception if non-https URL is passed in
- Support string URL arguments
- First argument is always required, so remove code that sets default
object if params doesn't exist

feross commented Apr 4, 2017

Copy link
Copy Markdown
Member Author

Tests are failing due to nodejs/nodejs.org#1191, so merging.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL