Zone.js higher than 0.6.21 breaks angular-universal https calls #484

AMar4enko · 2016-10-12T12:29:13Z

Calls to https:// end up with error code: ECONNRESET, as described in angular/universal-starter#172

JiaLiPassion · 2016-11-05T07:13:46Z

This is because the EventEmitter.once is not handled properly, I have create a pull request, you can use my repository for test, https://github.com/JiaLiPassion/zone.js.git.

…moveAllListeners

…atch. (#500) * fix long-stack-trace zone will not render correctly when reject a promise * fix issue #484 and #491, patch EventEmitter.once and removeAllListeners * add event emitter once test case * prependlistener should add listener at the beginning of the listener array. add test cases for prepend listener and once. * use newest fetch to test

John0x · 2016-11-23T14:47:53Z

Any updates on this?

JiaLiPassion · 2016-11-23T14:49:57Z

This has been fixed, you can use the latest zone.js 0.7.1 to test.

…tch process.nextTick, fix removeAllListeners bug (#516) * fix long-stack-trace zone will not render correctly when reject a promise * fix issue #484 and #491, patch EventEmitter.once and removeAllListeners * add event emitter once test case * prependlistener should add listener at the beginning of the listener array. add test cases for prepend listener and once. * use newest fetch to test * patch process.nextTick * restore ZoneAwareError captureStackTrace function * move captureStackTrace test into node * use hasOwnProperty for check captureStackTrace exist or not * change var to const * add process.spec.ts into node_tests.ts target * add done in process.spec.ts * change var to let * add nexttick order case * add prepareStackTrace callback to ZoneAwareError * fix when EventEmitter removeAllListeners has no event type parameter, should remove all listeners * change some var to let/const, remove unnecessary cancelTask call * modify testcases * remove typo * use zone.scheduleMicrotask to patch process.nextTick * forget use let/const again * add comment to removeAllListeners patch, and remove useCapturing parameter cause it is not needed * update fetch to 2.0.1

…moveAllListeners

JiaLiPassion mentioned this issue Nov 5, 2016

Fix EventEmitter.once issue and add EventEmitter removeAllListeners patch. #500

Merged

JiaLiPassion added a commit to JiaLiPassion/zone.js that referenced this issue Nov 9, 2016

fix issue angular#484 and angular#491, patch EventEmitter.once and re…

da482b9

…moveAllListeners

AMar4enko closed this as completed Nov 23, 2016

JiaLiPassion added a commit to JiaLiPassion/zone.js that referenced this issue Dec 6, 2016

fix issue angular#484 and angular#491, patch EventEmitter.once and re…

59c318a

…moveAllListeners

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Zone.js higher than 0.6.21 breaks angular-universal https calls #484

Zone.js higher than 0.6.21 breaks angular-universal https calls #484

AMar4enko commented Oct 12, 2016

JiaLiPassion commented Nov 5, 2016

John0x commented Nov 23, 2016

JiaLiPassion commented Nov 23, 2016

Zone.js higher than 0.6.21 breaks angular-universal https calls #484

Zone.js higher than 0.6.21 breaks angular-universal https calls #484

Comments

AMar4enko commented Oct 12, 2016

JiaLiPassion commented Nov 5, 2016

John0x commented Nov 23, 2016

JiaLiPassion commented Nov 23, 2016