This repository was archived by the owner on Aug 11, 2022. It is now read-only.

error-message: Fix messaging for 401 errors that lack a reason #20032

Closed

iarna wants to merge 1 commit into release-next from iarna/fix-generic-401

Contributor

iarna commented Mar 13, 2018

Prevously they were falling through to 404 handling and being printed as:

npm ERR! 404 Registry returned 401 for DELETE on …


          error-message: Fix messaging for 401 errors that lack a reason

8d7a07a

Prevously they were falling through to 404 handling and being printed as:

npm ERR! 404 Registry returned 401 for DELETE on …

iarna requested a review from a team as a code owner

March 13, 2018 01:33

zkat approved these changes

View reviewed changes

Contributor

zkat left a comment

🐑

iarna force-pushed the release-next branch 3 times, most recently from bcc4670 to 4a65b3b Compare

March 23, 2018 21:22

zkat force-pushed the release-next branch from 748c06e to ada9c6c Compare

April 4, 2018 00:58

iarna added a commit that referenced this pull request


          error-message: Fix messaging for 401 errors that lack a reason

c4e1fe7

Prevously they were falling through to 404 handling and being printed as:

npm ERR! 404 Registry returned 401 for DELETE on …

Credit: @iarna
Reviewed-By: @zkat
PR-URL: #20032

iarna added a commit that referenced this pull request


          error-message: Fix messaging for 401 errors that lack a reason

d69f3e7

Prevously they were falling through to 404 handling and being printed as:

npm ERR! 404 Registry returned 401 for DELETE on …

Credit: @iarna
Reviewed-By: @zkat
PR-URL: #20032

iarna force-pushed the release-next branch from 1a78f57 to 2a71d18 Compare

April 10, 2018 01:50

iarna added a commit that referenced this pull request


          error-message: Fix messaging for 401 errors that lack a reason

10f5442

Prevously they were falling through to 404 handling and being printed as:

npm ERR! 404 Registry returned 401 for DELETE on …

Credit: @iarna
Reviewed-By: @zkat
PR-URL: #20032

iarna force-pushed the release-next branch from 2a71d18 to d2d290b Compare

April 11, 2018 01:25

iarna added a commit that referenced this pull request


          error-message: Fix messaging for 401 errors that lack a reason

857c213

Prevously they were falling through to 404 handling and being printed as:

npm ERR! 404 Registry returned 401 for DELETE on …

Credit: @iarna
Reviewed-By: @zkat
PR-URL: #20032

iarna added a commit that referenced this pull request


          error-message: Fix messaging for 401 errors that lack a reason

0f14aa8

Prevously they were falling through to 404 handling and being printed as:

npm ERR! 404 Registry returned 401 for DELETE on …

Credit: @iarna
Reviewed-By: @zkat
PR-URL: #20032

iarna added a commit that referenced this pull request


          error-message: Fix messaging for 401 errors that lack a reason

5d17c87

Prevously they were falling through to 404 handling and being printed as:

npm ERR! 404 Registry returned 401 for DELETE on …

Credit: @iarna
Reviewed-By: @zkat
PR-URL: #20032

iarna force-pushed the release-next branch 3 times, most recently from 2e76172 to fe31e66 Compare

April 13, 2018 20:20

iarna force-pushed the release-next branch 2 times, most recently from 63c9452 to 1b021d0 Compare

April 21, 2018 00:09

Contributor Author

iarna commented May 3, 2018

This shipped in 6.0.0

iarna closed this

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

Labels

None yet