Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Do not add generated comment header to already minified files.

See original GitHub issue

When I tried to access https://cdn.jsdelivr.net/npm/hyperapp I got this header with the code:

/**
 * Skipped minification because the original files appears to be already minified.
 * Do NOT use SRI with dynamically generated files! More information: https://www.jsdelivr.com/using-sri-with-dynamic-files
 */

Would you consider not adding this header by default?

Cheers!

Issue Analytics

State:
Created 6 years ago
Reactions:1
Comments:7 (5 by maintainers)

Top GitHub Comments

1reaction

MartinKolarikcommented, Sep 18, 2017

The header is important because it allows you to distinguish between files provided directly in the package and those generated on our end. You can always use the direct link to get the original file: https://cdn.jsdelivr.net/npm/hyperapp@0.12.1/dist/hyperapp.js (the one that’s served at the short link is https://cdn.jsdelivr.net/npm/hyperapp@0.12.1/dist/hyperapp.min.js)

0reactions

MartinKolarikcommented, Sep 18, 2017

In that case just use the direct link as I suggested before.