publish: Better message when an "OTP" error occurs

PR-URL: #19580
Credit: @jdeniau
Reviewed-By: @iarna

lib/publish.js

@@ -209,7 +209,7 @@ function upload (arg, pkg, isRetry, cached) {
     if (err.code !== 'EOTP' && !(err.code === 'E401' && /one-time pass/.test(err.message))) throw err
     // we prompt on stdout and read answers from stdin, so they need to be ttys.
     if (!process.stdin.isTTY || !process.stdout.isTTY) throw err
-    return readUserInfo.otp('Enter OTP:  ').then((otp) => {
+    return readUserInfo.otp().then((otp) => {
       npm.config.set('otp', otp)
       return upload(arg, pkg, isRetry, cached)
     })

lib/utils/read-user-info.js

@@ -19,7 +19,15 @@ function read (opts) {
 }
 
 function readOTP (msg, otp, isRetry) {
-  if (!msg) msg = 'Enter OTP: '
+  if (!msg) {
+    msg = [
+      'There was an error while trying authentication due to OTP (One-Time-Password).',
+      'The One-Time-Password is generated via applications like Authy or',
+      'Google Authenticator, for more information see:',
+      'https://docs.npmjs.com/getting-started/using-two-factor-authentication',
+      'Enter OTP: '
+    ].join('\n')
+  }
   if (isRetry && otp && /^[\d ]+$|^[A-Fa-f0-9]{64,64}$/.test(otp)) return otp.replace(/\s+/g, '')
 
   return read({prompt: msg, default: otp || ''})