Js-ipfs: jsDelivr complains about package size exceeded the configured limit of 50 MB

Created on 15 Aug 2018  ·  4Comments  ·  Source: ipfs/js-ipfs

Package size exceeded the configured limit of 50 MB

I think it is important as links in the readme are to jsDelivr

https://www.jsdelivr.com/package/npm/ipfs

image

P2 dieasy help wanted

Most helpful comment

Hi, Martin from jsDelivr here. Technically we are able to serve packages even if they exceed this limit and I could add an exception for ipfs.

The reason for having a limit by default is that npm only provides tarballs with all files so we always have to fetch everything, even if nothing else than dist/ is needed. The bigger the package, the higher the chance this takes too long and some requests occasionally fail.

We also find that very few packages actually need to exceed this limit. In your case, 63 MB out of 80 MB are just the examples. In my opinion, those do not belong to the npm package at all and should be added to .npmignore.

All 4 comments

Is there a way to configure this to only care about/load the dist directory?

On Aug 15, 2018, at 2:31PM, Kevin Simper notifications@github.com wrote:

Package size exceeded the configured limit of 50 MB

I think it is important as links in the readme are to jsDelivr

https://www.jsdelivr.com/package/npm/ipfs https://www.jsdelivr.com/package/npm/ipfs
https://user-images.githubusercontent.com/1126497/44174389-4d516380-a0e3-11e8-8691-112ec268adc6.png

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/ipfs/js-ipfs/issues/1510, or mute the thread https://github.com/notifications/unsubscribe-auth/AAACQ88GrBzu9hHh2ccwGQGN8pHS4YVJks5uRJMmgaJpZM4V-2EM.

Hi, Martin from jsDelivr here. Technically we are able to serve packages even if they exceed this limit and I could add an exception for ipfs.

The reason for having a limit by default is that npm only provides tarballs with all files so we always have to fetch everything, even if nothing else than dist/ is needed. The bigger the package, the higher the chance this takes too long and some requests occasionally fail.

We also find that very few packages actually need to exceed this limit. In your case, 63 MB out of 80 MB are just the examples. In my opinion, those do not belong to the npm package at all and should be added to .npmignore.

fixed in https://github.com/ipfs/js-ipfs/pull/1513 and released in 0.31.7

@alanshaw Awesome!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

mnts picture mnts  ·  3Comments

Ntmf picture Ntmf  ·  4Comments

daviddias picture daviddias  ·  3Comments

daviddias picture daviddias  ·  3Comments

dryajov picture dryajov  ·  3Comments