Skip to content
This repository was archived by the owner on Oct 8, 2024. It is now read-only.
This repository was archived by the owner on Oct 8, 2024. It is now read-only.

Npm package broken #28

Description

@jeremylevy

The NPM package seems to be broken :

npm install hideShowPassword
fetch failed https://registry.npmjs.org/hideShowPassword/-/hideShowPassword-2.0.4.tgz
npm ERR! Error: 404 Not Found

By the way, thanks for the good work. It works very well.

Activity

  1. tylersticka commented on May 13, 2015

    @tylersticka
    Member

    Hrm, I'm encountering the same error. Thanks for letting us know, we'll investigate...

  2. tylersticka commented on May 13, 2015

    @tylersticka
    Member

    Huh, it worked for me a moment ago, but for @erikjung it did not.

    According to NPM's Troubleshooting page:

    • It's most likely a temporary npm registry glitch. Check npm server status and try again later.
    • If the error persists, perhaps the published package is corrupt. Contact the package owner and have them publish a new version of the package.

    But the status is currently green. The fact that it installs fine some of the time (and if I do an npm install ~/my/local/folder/hideShowPassword, that works fine, too) suggests that corruption is unlikely.

    Some pages online say you need to set the npm registry, but running npm config get registry for me outputs the correct registry URL (https://registry.npmjs.org/).

    Most perplexing, one of our other npm modules doesn't seem to have this problem at all...

    Still investigating....... 💭

  3. jeremylevy commented on May 13, 2015

    @jeremylevy
    Author

    It worked for me BEFORE my npm cache was cleaned with:

    npm cache clean
    

    Once cleaned, the npm install has broken.

    Thanks for the support !

  4. tylersticka commented on May 13, 2015

    @tylersticka
    Member

    Yeah, that seems to be the case for me as well... cleaning the cache makes it break again.

    @lyzadanger pointed out that it's perplexing to see zero downloads on the npm page, compared to another package which has many downloads despite being far less popular (it has about 1% of the amount of GitHub stars). Something's not right there!

  5. self-assigned this
    on May 13, 2015
  6. tylersticka commented on May 13, 2015

    @tylersticka
    Member

    I'm comparing the two package.json files and noticing a discrepancy in the way the repository.url field is specified.

    For Leveller, it's defined as https://github.com/cloudfour/leveller.git.

    For hideShowPassword, it's defined as git://github.com/cloudfour/hideShowPassword.git.

    The NPM docs show examples that use the former (https), not the latter (git).

    I'm going to modify the package.json and push a new version to see if that remedies the issue or not.

  7. tylersticka commented on May 13, 2015

    @tylersticka
    Member

    Did not mean to close this.

    So, I implemented the package.json change, but then there's a new wrinkle... I found out that npm doesn't allow publishing of packages with capital letters in their names. I'm not sure how we got a registry page at all before, but apparently that was a fluke or an oversight or a hiccup or something.

    Still developing...

  8. added a commit that references this issue on May 13, 2015
  9. tylersticka commented on May 13, 2015

    @tylersticka
    Member

    @jeremylevy What a long, strange trip it's been! Thanks for bearing with us.

    Can you try npm install hideshowpassword (all lower-case) and see if that resolves your issue?

  10. jeremylevy commented on May 14, 2015

    @jeremylevy
    Author

    It works ! You rock ! 👍

  11. tylersticka commented on May 14, 2015

    @tylersticka
    Member

    @jeremylevy Great, thanks so much for reporting this issue! It's embarrassing to think how long this npm package may have been broken. 😓

    I spoke with the folks at NPM and they recommended deprecating hideShowPassword. So now, if you attempt to install the camel-case version, you should see a warning that advises you to use the lowercase version instead:

    $ npm install hideShowPassword
    npm WARN deprecated [email protected]: use lowercase version: hideshowpassword
    ...
    

    Closing this issue now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions