angular / angular-cli

CLI tool for Angular
https://cli.angular.io
MIT License
26.71k stars 11.98k forks source link

[bug] Uncaught ReferenceError: global is not defined #8160

Closed amitport closed 6 years ago

amitport commented 6 years ago

Bug Report or Feature Request (mark with an x)

- [X] bug report -> please search issues before submitting
- [ ] feature request

Versions.

Angular CLI: 1.5.0-rc.3
Node: 8.7.0
OS: win32 x64
Angular: 4.4.6
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router, tsc-wrapped, upgrade

@angular/cdk: 2.0.0-beta.12
@angular/cli: 1.5.0-rc.3
@angular/flex-layout: 2.0.0-beta.9
@angular/material: 2.0.0-beta.12
@angular-devkit/build-optimizer: 0.0.30
@angular-devkit/core: 0.0.20
@angular-devkit/schematics: 0.0.34
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.8.0-rc.3
@schematics/angular: 0.0.48
typescript: 2.3.4
webpack: 3.8.1

Repro steps.

create angular/cli project add dependency and import socket.io-client@2.0.4 ng serve --aot open browser

The log given by the failure.

index.js:10 Uncaught ReferenceError: global is not defined
    at eval (index.js:10)
    at Object.../../../../has-binary2/index.js (vendor.bundle.js:2914)
    at __webpack_require__ (inline.bundle.js:55)
    at eval (index.js:8)
    at Object.../../../../socket.io-parser/index.js (vendor.bundle.js:9569)
    at __webpack_require__ (inline.bundle.js:55)
    at eval (index.js:7)
    at Object.../../../../socket.io-client/lib/index.js (vendor.bundle.js:9527)
    at __webpack_require__ (inline.bundle.js:55)
    at eval (io-service.ts:1)
dottodot commented 6 years ago

I seeing the same issue as well with ng-socket-io but also with import 'intl'; in polyfills.ts

amitport commented 6 years ago
dottodot commented 6 years ago

Strange I've reverted to 1.5.0-rc.2 and still get it, no idea what's going on.

amitport commented 6 years ago

@dottodot oh sorry, I actually reverted to 1.5.0-rc.1 in any case, make sure it's a clean revert. try deleting node_modules and packge/yarn-lock before npm i

dottodot commented 6 years ago

Thanks that worked. I've gone back to 1.5.0-rc.2 and that's ok for me so seems like the issue is in 1.5.0-rc.3

Martin-Luft commented 6 years ago

Same problem here since 1.5.0-rc.3, with 1.5.0-rc.2 no problems...

robisim74 commented 6 years ago

Same error using rc.3 Uncaught ReferenceError: global is not defined also importing another library (jsrsasing). It worked fine until rc.2.

RicardoVaranda commented 6 years ago

Hi guys, could you please provide a simple repo of this issue, that way it's easier and faster for the team to be able to debug the issue.

dottodot commented 6 years ago

@RicardoVaranda https://github.com/dottodot/ang5-test

All i've done is create a new project with 1.5.0-rc.3 and included import 'intl'; in polyfills.ts and you get the error

filipesilva commented 6 years ago

This is definitely related to https://github.com/angular/angular-cli/pull/8132 as we've removed the node global object from the compilation environment. I'm investigating what can be done about it.

filipesilva commented 6 years ago

Just wanted to let you know we're looking into this. For the intl case using import 'intl/dist/intl'; instead fixes it, but this might not be possible for all libs. socket.io-client also seems to have a dist in socket.io-client/dist/socket.io.js. I don't think the same can be used with ng-socket-io however, since it's importing the node package inside of it...

MrBlaise commented 6 years ago

Hey, I just wanted to chime in. I have the same issue upgrading to 1.5.0-rc.3 causes this issue, 1.5.0-rc.2 works fine. Didn't change anything just upgraded thew version. I use the following polyfills:

import 'core-js/es6/symbol';
import 'core-js/es6/object';
import 'core-js/es6/function';
import 'core-js/es6/parse-int';
import 'core-js/es6/parse-float';
import 'core-js/es6/number';
import 'core-js/es6/math';
import 'core-js/es6/string';
import 'core-js/es6/date';
import 'core-js/es6/array';
import 'core-js/es6/regexp';
import 'core-js/es6/map';
import 'core-js/es6/set';

/** IE10 and IE11 requires the following for NgClass support on SVG elements */
import 'classlist.js';  // Run `npm install --save classlist.js`.

/** IE10 and IE11 requires the following to support `@angular/animation`. */
import 'web-animations-js';  // Run `npm install --save web-animations-js`.

/** Evergreen browsers require these. **/
import 'core-js/es6/reflect';
import 'core-js/es7/reflect';

/***************************************************************************************************
 * Zone JS is required by Angular itself.
 */
import 'zone.js/dist/zone';  // Included with Angular CLI.
bogacg commented 6 years ago

With Rc.6 still having same problem.

dottodot commented 6 years ago

There's a pull request which I assume will fix it guess we just need to wait for it to be added

https://github.com/angular/angular-cli/pull/8189

amitport commented 6 years ago

This is fixed for me with release 1.5.0-rc.8 (https://github.com/angular/angular-cli/pull/8250). Thanks!

StojanovicM commented 6 years ago

Hi, I am using Dragula version 1.5.0 with Angular 5, and still having problem "ReferenceError: global is not defined"

I have just installed it, imported and added to NgModule -> Imports, and my app crashes with this error.

Is there any step I am missing??

filipesilva commented 6 years ago

On version 6 of Angular CLI we are removing the shim for global and other node built-ins. You can read more about why this change was made in https://github.com/angular/angular-cli/issues/9827#issuecomment-369578814.

If you are using a library that assumes these globals are present, you can try manually shimming it inside your polyfills.ts file:

// Add global to window, assigning the value of window itself.
(window as any).global = window;

I ran a quick test with socket.io-client and this seemed to allow it to run.

gioragutt commented 6 years ago

@filipesilva Thank you so much!

krishnareddy226 commented 6 years ago

declare var $: any; I am getting the error $ not defined

bluefire2121 commented 6 years ago

In my polyfills.ts, the import references to intl were still there. Comment out or delete these if you get stuck, then add (window as any).global= window as @filipesilva stated above.

tobias74 commented 6 years ago

It seems that all the import-statements in the polyfill.ts are hoisted in the compiled polyfill.js. Therefore putting

(window as any).global = window;

at the top of the file polyfill.ts did not solve the issue for me, since in the compiled polyfill.js this command was located at the very end of the file (after all the imports).

Solution

I had to put the line

(window as any).global = window;

in a new file (for example "global-shim.ts")

and import this file at the very top of the polyfill.ts

import 'global-shim';

gioragutt commented 6 years ago

@tobias74 I put it in the end of the file, it worked for me.

yulm6 commented 6 years ago

When I adding sockjs-client, I got the same issue. npm install sockjs-client --save

I did as @tobias74 ,maybe it worked,but the brower refreshes frequenly.

chetanbirajdar commented 6 years ago

@tobias74 Thanks that worked

Evllis commented 6 years ago

@filipesilva Thank you!!!!!!!!! ^_^

SamiHK commented 6 years ago

I removed all these things. import 'intl'; (window as any).global = window; // Runnpm install --save intl. import 'intl/locale-data/jsonp/en';

AND NOW :D This polyfill.ts is working for me.

`import 'core-js/es6/symbol'; import 'core-js/es6/object'; import 'core-js/es6/function'; import 'core-js/es6/parse-int'; import 'core-js/es6/parse-float'; import 'core-js/es6/number'; import 'core-js/es6/math'; import 'core-js/es6/string'; import 'core-js/es6/date'; import 'core-js/es6/array'; import 'core-js/es6/regexp'; import 'core-js/es6/map'; import 'core-js/es6/weak-map'; import 'core-js/es6/set';

import 'core-js/es7/array';

/* IE10 and IE11 requires the following for NgClass support on SVG elements / import 'classlist.js'; // Run npm install --save classlist.js.

/* IE10 and IE11 requires the following for the Reflect API. / import 'core-js/es6/reflect';

/ Evergreen browsers require these. / // Used for reflect-metadata in JIT. If you use AOT (and only Angular decorators), you can remove. import 'core-js/es7/reflect';

/**

/***

/***

/**

avinexus7 commented 5 years ago

On version 6 of Angular CLI we are removing the shim for global and other node built-ins. You can read more about why this change was made in #9827 (comment).

If you are using a library that assumes these globals are present, you can try manually shimming it inside your polyfills.ts file:

// Add global to window, assigning the value of window itself.
(window as any).global = window;

I ran a quick test with socket.io-client and this seemed to allow it to run.

Is this the right way of implementing socket.io in Angular 2? I mean does the above line give any side affects that may be obtrusive in the future?

BMLande commented 5 years ago

solution of @filipesilva is worked . i have issue when i am migrating ng5 to ng6 . thanks @filipesilva

fkolar commented 5 years ago

@filipesilva I see this is pretty old issue but even today, newly generated project and I am testing apolo graphql does not add the global.

so I have to add to my polyfills.ts the (window as any).global = window;

Is this normal ?

clydin commented 5 years ago

Yes. global does not exist in a browser. A library that supports web browsers should not be using Node.js specific functionality.

willkara commented 5 years ago

Will this ever be fixed in the main module? Still apparently happens in new projects.

I've tried adding a basic AwsAmplify project and this happens.

amitport commented 5 years ago

@willkara, read the comment right before yours.

Follow the stack trace to see which package uses global. submit an actionable, reproducible issue on that package repo

(also, "happens in new projects." is very different from happened when you "tried adding a basic AwsAmplify")

willkara commented 5 years ago

@amitport fair comment. I'll follow the trace again and see where it happens.

danieldanielecki commented 5 years ago

In case, if your target is node in webpack (target: 'node'), because you want to fix "Can't resolve 'fs'. Then you're getting the following error Fix: "Uncaught ReferenceError: global is not defined" do it as follows node: { global: true, fs: 'empty' }. Bonus: if you got error "Uncaught ReferenceError: exports is not defined". simply add libraryTarget: 'umd'. The complete webpack config code is below.

const webpackConfig = {
  node: { global: true, fs: 'empty' }, // Fix: "Uncaught ReferenceError: global is not defined", and "Can't resolve 'fs'".
  output: {
    libraryTarget: 'umd' // Fix: "Uncaught ReferenceError: exports is not defined".
  }
};

module.exports = webpackConfig; // Export all custom Webpack configs.
amitport commented 5 years ago

@filipesilva can you lock this issue? seem to keep "sprouting" in all directions even though it's closed.

ar1stio commented 4 years ago

just add (window as any).global = window; in polyfills.ts in ionic

angular-automatic-lock-bot[bot] commented 4 years ago

This issue has been automatically locked due to inactivity. Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.