Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[backport] PR #8209 to 5.0 - [optimizer] move to querystring-browser package for up-to-date api #8215

Merged
merged 1 commit into from
Sep 10, 2016

Conversation

elastic-jasper
Copy link
Contributor

Backport PR #8209

Commit 1:
[optimizer] move to querystring-browser package for up-to-date api

Webpack automatically injects libraries to support many node.js APIs in the browser using the https://github.com/webpack/node-libs-browser package. This package uses https://github.com/mike-spainhower/querystring for the querystring module, but that module hasn't been updated in 4 years and doesn't support the modern querystring api.

This change points to the https://github.com/mathiasvr/querystring package instead, which is up to date (specifically it supports the encodeURIComponent option of querystring.stringify.

---------

**Commit 1:**
[optimizer] move to querystring-browser package for up-to-date api

Webpack automatically injects libraries to support many node.js APIs in the browser using the https://github.com/webpack/node-libs-browser package. This package uses https://github.com/mike-spainhower/querystring for the querystring module, but that module hasn't been updated in 4 years and doesn't support the modern querystring api.

This change points to the https://github.com/mathiasvr/querystring package instead, which is up to date (specifically it supports the `encodeURIComponent` option of [`querystring.stringify`](https://nodejs.org/api/querystring.html#querystring_querystring_stringify_obj_sep_eq_options).

* Original sha: 1030689
* Authored by spalger <spalger@users.noreply.github.com> on 2016-09-09T21:10:50Z
@epixa epixa changed the title [backport] PR #8209 to 5.0 [backport] PR #8209 to 5.0 - [optimizer] move to querystring-browser package for up-to-date api Sep 10, 2016
@epixa epixa merged commit db15eee into 5.0 Sep 10, 2016
@epixa epixa deleted the jasper/backport/8209/5.0 branch September 10, 2016 13:22
alexwizp added a commit to alexwizp/kibana that referenced this pull request Sep 9, 2024
alexwizp added a commit to alexwizp/kibana that referenced this pull request Sep 10, 2024
alexwizp added a commit to alexwizp/kibana that referenced this pull request Sep 11, 2024
alexwizp added a commit to alexwizp/kibana that referenced this pull request Sep 13, 2024
alexwizp added a commit to alexwizp/kibana that referenced this pull request Sep 18, 2024
alexwizp added a commit to alexwizp/kibana that referenced this pull request Sep 24, 2024
alexwizp added a commit to alexwizp/kibana that referenced this pull request Sep 27, 2024
alexwizp added a commit to alexwizp/kibana that referenced this pull request Sep 29, 2024
alexwizp added a commit to alexwizp/kibana that referenced this pull request Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants