Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make use of unpkg.com instead of wzrd.in #176

Closed
yaroslav-ilin opened this issue Feb 1, 2017 · 10 comments · Fixed by #323
Closed

Make use of unpkg.com instead of wzrd.in #176

yaroslav-ilin opened this issue Feb 1, 2017 · 10 comments · Fixed by #323

Comments

@yaroslav-ilin
Copy link

There is an option described in docs to use the package via wzrd.in.
I've just noticed that service responds with 502 code.

Will it be fine to make pull request to replace usage of wzrd.in with unpkg.com (ex-npmcdn)?

BTW, they seem to be fine with using their URLs in production so it might become recommended way to use uuid in browser.

@broofa
Copy link
Member

broofa commented Feb 1, 2017

Yup, looks like wzrd.in is down.

Unpkg looks like a pretty amateur effort. I don't mean that in a derogatory sense - only that it's one person's "best effort" solution, with no real SLA. Recommending that as a production-level solution seems optimistic. But I'm fine switching to that until someone recommends a better option.

@yaroslav-ilin
Copy link
Author

yaroslav-ilin commented Feb 7, 2017

Indeed, you are right, unpkg is not suited for production use as they clearly state on their website.

Unfortunately, there is another issue, which holds me back from making a pull request with suggested earlier edits: services like unpkg don't provide any ways to bundle CommonJS modules into components usable from browser out of the box (they don't bundle CommonJS into AMD or something and instead rely on package maintainers to provide npm package with prebuilt lib sources).

I checked issues and it seems that there is no goal to provide uuid npm package in a bundled state. Feel free to close this issue if I am right, otherwise I can take a look and try to make some prebulish npm script in order to make published npm packages consisting of bundled sources version as well.

@defunctzombie
Copy link
Contributor

I checked issues and it seems that there is no goal to provide uuid npm package in a bundled state. Feel free to close this issue if I am right

Correct, there is no desire to do this because packaged state means different things to different people.

The goal with putting wzrd.in on the readme was less about production and more to give folks a way to download a prebuilt version for use in their projects. If you aren't going to install and bundle this module with your deployment, then the next reliable thing is to copy the bundled code into your project and reference it from there.

@broofa
Copy link
Member

broofa commented Jun 16, 2017

sticking with wzrd.in for now.

@broofa broofa closed this as completed Jun 16, 2017
@PavelPolyakov
Copy link

https://wzrd.in/standalone/uuid/v4@latest

doesn't work

@broofa
Copy link
Member

broofa commented Aug 2, 2018

Seems like https://cdnjs.com/ is the go-to resource for this sort of thing these days. Would love to switch over to that, but uuid doesn't seem to be a supported library yet. Anyone want to tackle that?

Note: Ideally users should be able to link directly to the version-specific file within the uuid module. E.g. https://cdnjs.cloudflare.com/ajax/libs/uuid/3.3.2/v4.js

@broofa broofa reopened this Aug 2, 2018
@styfle
Copy link
Contributor

styfle commented Sep 3, 2018

Another alternative is www.jsdelivr.com which does tout "Built for production" since they use multiple CDN providers.

List of files from npm: https://cdn.jsdelivr.net/npm/uuid/

Direct link to a file: https://cdn.jsdelivr.net/npm/uuid@3.3.2/v4.js

They even have a badge with stats for your package

UPDATE: one thing to point out is that they don't do any conversion for CJS to UMD or ESM so you'll either need to publish a "compiled" file or the user must implement require in the browser.

@niftylettuce
Copy link

I will handle this, give me a few moments to fork and submit a PR

@niftylettuce
Copy link

PR submitted #293

ctavan added a commit to ctavan/node-uuid that referenced this issue Aug 26, 2019
The service wzrd.in seems to be unmaintained for over a year now, see
browserify/wzrd.in@2f7ea69

It currently throws 502 errors on all packages as reported in
uuidjs#302 or
https://talk.observablehq.com/t/wzrd-in-alternatives/571

As stated in
uuidjs#293 (comment)
there are no plans to support legacy bundlers.

Instead, we may consider to go for properly standardized ES6 modules in
the future as has been teased in uuidjs#317

However for the time being let's at least remove these broken
instructions from the README.

Closes uuidjs#176, uuidjs#302
This was referenced Aug 26, 2019
@ctavan
Copy link
Member

ctavan commented Aug 26, 2019

See #302 (comment) for an alternative that works as of today (but may cease to work in the future just like wzrd.in since it also seems to be rather a pet project than a professional service that you should be relying on).

broofa pushed a commit that referenced this issue Sep 1, 2019
* chore(doc): re-generate README

* fix(doc): remove wzrd.in instruction from README

The service wzrd.in seems to be unmaintained for over a year now, see
browserify/wzrd.in@2f7ea69

It currently throws 502 errors on all packages as reported in
#302 or
https://talk.observablehq.com/t/wzrd-in-alternatives/571

As stated in
#293 (comment)
there are no plans to support legacy bundlers.

Instead, we may consider to go for properly standardized ES6 modules in
the future as has been teased in #317

However for the time being let's at least remove these broken
instructions from the README.

Closes #176, #302
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants