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

Buffer module ends up in production build #901

Closed
davscro opened this issue May 15, 2017 · 2 comments
Closed

Buffer module ends up in production build #901

davscro opened this issue May 15, 2017 · 2 comments

Comments

@davscro
Copy link

davscro commented May 15, 2017

Do you want to request a feature or report a bug?

Bug

What is the current behavior?

I'm not sure is this bug or excepted behaviour but I believe it's a bug. Running yarn build to generate production build, buffer module ends up in production build!

 "scripts": {
    "build": "set NODE_ENV=production rimraf build/dist && webpack -p --config webpack/webpack.config.js && npm run bundle_check",
    "bundle_check": "babel-node bundle_check.js"
  },

Could someone explain me why this package ends up in production build. Am I missing something?

screen shot 2017-05-13 at 00 43 18

yarn why buffer

screen shot 2017-05-13 at 00 45 25

yarn why node-libs-browser
screen shot 2017-05-13 at 00 45 57

And webpack is specified only as dev dependency.
I pointed this issue to WEBPACk but they answered it's connected with axios. Then I searched your repo and found the usage within the HTTP adapter

What is the expected behavior?
Buffer module should not be in production build.

Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.

axios: 0.16.1,
Webpack: 2.5.1
Node: 7.8.0
OS: OSX Sierra

Thanks in advance,
Davor

@nickuraltsev
Copy link
Member

Fixed via #887.

@davscro
Copy link
Author

davscro commented Jun 1, 2017

hey @nickuraltsev, #887 is not fixing this issue.

@axios axios locked and limited conversation to collaborators May 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants