use named export + misc cleanup

Author: eli-darkly

CHANGELOG.md

@@ -3,8 +3,11 @@
 All notable changes to this package will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org).
 
-## [1.0.8] - 2019-01-29
-First release from this fork. Changes from the 1.0.7 release of the upstream code are as follows:
+## [1.0.0] - 2019-01-29
+First release from this fork. Changes from the previous release of the upstream code (1.0.7) are as follows:
 
 ### Added:
 - The optional `method` and `body` properties of the constructor options allow you to specify a different HTTP method from the default of `GET`, and to provide a request body if the specified method allows a body.
+
+### Changed:
+- The EventSource constructor is now a named export, not a default export. This is necessary in order to avoid a problem that  can happen when using Babel with ES6 code: the logic for converting CJS imports to ES6 imports does not work correctly if the default import has properties (`CONNECTING`, `OPEN`, `CLOSED`) but is also a function. Note that this is a breaking change if you were previously using the upstream code, but the only thing that needs to be changed is the import statement.

CONTRIBUTING.md

@@ -4,7 +4,7 @@ If you add or fix something, add tests.
 
 ## Release process
 
-Update `HISTORY.md`, Then:
+Update `CHANGELOG.md`, Then:
 
     npm outdated --depth 0 # See if you can upgrade something
     npm run polyfill

HISTORY.md

@@ -1,11 +1,11 @@
-# EventSource [![npm version](http://img.shields.io/npm/v/launchdarkly-eventsource.svg?style=flat-square)](http://browsenpm.org/package/launchdarkly-eventsource)[![Build Status](http://img.shields.io/travis/launchdarkly/js-eventsource/master.svg?style=flat-square)](https://travis-ci.org/launchdarkly/js-eventsource)[![NPM Downloads](https://img.shields.io/npm/dm/laumchdarkly-eventsource.svg?style=flat-square)](http://npm-stat.com/charts.html?package=launchdarkly-eventsource&from=2015-09-01)[![Dependencies](https://img.shields.io/david/launchdarkly/js-eventsource.svg?style=flat-square)](https://david-dm.org/launchdarkly/js-eventsource)
+# EventSource [![npm version](http://img.shields.io/npm/v/launchdarkly-eventsource.svg?style=flat-square)](http://browsenpm.org/package/launchdarkly-eventsource)[![Circle CI](https://circleci.com/gh/launchdarkly/js-eventsource/tree/master.svg?style=svg)](https://circleci.com/gh/launchdarkly/js-eventsource/tree/master)[![NPM Downloads](https://img.shields.io/npm/dm/laumchdarkly-eventsource.svg?style=flat-square)](http://npm-stat.com/charts.html?package=launchdarkly-eventsource&from=2015-09-01)[![Dependencies](https://img.shields.io/david/launchdarkly/js-eventsource.svg?style=flat-square)](https://david-dm.org/launchdarkly/js-eventsource)
 
 This library is a pure JavaScript implementation of the [EventSource](https://html.spec.whatwg.org/multipage/server-sent-events.html#server-sent-events) client. The API aims to be W3C compatible.
 
 You can use it with Node.js or as a browser polyfill for
 [browsers that don't have native `EventSource` support](http://caniuse.com/#feat=eventsource).
 
-This is a fork of the original [EventSource](https://github.com/EventSource/eventsource) project by Aslak Hellesøy, with minimal additions to support the requirements of the LaunchDarkly Node and Electron SDKs.
+This is a fork of the original [EventSource](https://github.com/EventSource/eventsource) project by Aslak Hellesøy, with additions to support the requirements of the LaunchDarkly Node and Electron SDKs. Note that as described in the [changelog](CHANGELOG.md), the API is _not_ backward-compatible with the original package, although it can be used with minimal changes.
 
 ## Install
 
@@ -51,6 +51,15 @@ var eventSourceInitDict = {headers: {'Cookie': 'test=test'}};
 var es = new EventSource(url, eventSourceInitDict);
 ```
 
+### Setting HTTP request method/body
+
+By default, EventSource makes a `GET` request. You can specify a different HTTP verb and/or a request body:
+
+```javascript
+var eventSourceInitDict = {method: 'POST', body: 'n=100'};
+var es = new EventSource(url, eventSourceInitDict);
+```
+
 ### Allow unauthorized HTTPS requests
 
 By default, https requests that cannot be authorized will cause the connection to fail and an exception

README.md

@@ -303,7 +303,9 @@ function EventSource (url, eventSourceInitDict) {
   }
 }
 
-module.exports = EventSource
+module.exports = {
+  EventSource: EventSource
+}
 
 util.inherits(EventSource, events.EventEmitter)
 EventSource.prototype.constructor = EventSource; // make stacktraces readable

lib/eventsource.js

@@ -1,6 +1,6 @@
 {
   "name": "launchdarkly-eventsource",
-  "version": "1.0.8",
+  "version": "1.0.0-rc.1",
   "description": "Fork of eventsource package - W3C compliant EventSource client for Node.js and browser (polyfill)",
   "keywords": [
     "eventsource",

package.json

@@ -1,5 +1,5 @@
 /* eslint-disable no-new */
-var EventSource = require('../lib/eventsource')
+var EventSource = require('../lib/eventsource').EventSource
 var bufferFrom = require('buffer-from')
 var path = require('path')
 var http = require('http')