Skip to content
This repository has been archived by the owner on Dec 4, 2018. It is now read-only.

Dependency Bloat #146

Closed
tristanhoy opened this issue Jun 7, 2018 · 2 comments
Closed

Dependency Bloat #146

tristanhoy opened this issue Jun 7, 2018 · 2 comments

Comments

@tristanhoy
Copy link

Expected behavior

A simple package should be very lean to reduce:

  • Payload sizes
  • Build times
  • Vulnerability surface area

Observed behavior

Bugsnag has over 5mb of dependencies

Steps to reproduce

npm install bugsnag

Version

2.4.0

Additional information

  • Replace request (4MB) with isomorphic-fetch (500KB)
  • Upgrade cuid 1.3.8 (1.24MB) to cuid 2.1.1 (44KB)
@bengourley
Copy link
Contributor

You raise some good suggestions. I think the cuid upgrade would be a lot easier than the isomorphic fetch migration. I'll make a note to address this – though that might not happen before we merge our node/browser js notifiers, in which case it will be addressed anyway.

@bengourley
Copy link
Contributor

I'm closing off this issue because this package is now deprecated. @bugsnag/js is our new "universal" notifier for Node and Browser JS.

We still use request to send error reports from node because we need robust proxy support, which isomorphic-fetch (and other reasonably sized alternatives to request) doesn't have.

We will be looking at moving to just require('http')/require('https') combined with node-proxy-agent because there are still a ton of request's features that we don't use.

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