Skip to content
This repository has been archived by the owner on Nov 4, 2022. It is now read-only.

X-GitHub-OTP Error #328

Merged
merged 1 commit into from
Dec 17, 2014
Merged

X-GitHub-OTP Error #328

merged 1 commit into from
Dec 17, 2014

Conversation

henvic
Copy link
Member

@henvic henvic commented Dec 17, 2014

I just installed and I'm trying to authenticate and I'm getting:

Cannot set property 'X-GitHub-OTP' of undefined

img
(I deleted my password *s and auth code from the image)

What am I missing?

@tjconcept
Copy link

I just hit the same.

@paulirish
Copy link

Same here.

@eduardolundgren
Copy link
Member

/cc @henvic

@henvic
Copy link
Member

henvic commented Dec 16, 2014

Working on this.

@henvic
Copy link
Member

henvic commented Dec 17, 2014

Pull request submitted to #328.

:octocat: Sent from GH.

@henvic henvic closed this Dec 17, 2014
@henvic
Copy link
Member

henvic commented Dec 17, 2014

Just to let you know: there's still another bug that's happening just after authentication and I'm still working on that.

@henvic henvic reopened this Dec 17, 2014
henvic added a commit that referenced this pull request Dec 17, 2014
@henvic henvic merged commit 582f269 into master Dec 17, 2014
@henvic henvic deleted the pr-328 branch December 17, 2014 00:15
@wfalkwallace
Copy link
Author

@henvic, I just retried and I'm still getting the same:

/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules/rx/dist/rx.all.js:9229
        throw e;
              ^
TypeError: Cannot set property 'X-GitHub-OTP' of undefined
    at PromptUI.completed (/usr/local/lib/node_modules/gh/lib/cmds/user.js:179:45)
    at PromptUI.onCompletion (/usr/local/lib/node_modules/gh/node_modules/inquirer/lib/ui/prompt.js:68:10)
    at AnonymousObserver.Rx.AnonymousObserver.AnonymousObserver.completed (/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules/rx/dist/rx.all.js:1878:12)
    at AnonymousObserver.Rx.internals.AbstractObserver.AbstractObserver.onCompleted (/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules/rx/dist/rx.all.js:1815:14)
    at AutoDetachObserverPrototype.completed (/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules/rx/dist/rx.all.js:9247:23)
    at AutoDetachObserver.Rx.internals.AbstractObserver.AbstractObserver.onCompleted (/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules/rx/dist/rx.all.js:1815:14)
    at AnonymousObserver._onCompleted (/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules

@henvic
Copy link
Member

henvic commented Dec 17, 2014

We still need to release a new version.

@zenorocha, can you do that for us?

@wfalkwallace
Copy link
Author

my mistake, thanks!

@thulioph
Copy link

thulioph commented Feb 1, 2015

Hey @wfalkwallace how do you resolved this problem?

@wfalkwallace
Copy link
Author

@thulioph, I didn't yet 😿

@henvic
Copy link
Member

henvic commented Feb 1, 2015

Have you guys tried updating gh already?

@thulioph
Copy link

thulioph commented Feb 2, 2015

Yep. I tried today, but not working.

@henvic
Copy link
Member

henvic commented Feb 2, 2015

Are you stuck on any prior version or are you telling me version 1.9.4 doesn't work? Use npm -v gh to verify what version you're currently on.

@henvic
Copy link
Member

henvic commented Feb 2, 2015

If you're stuck, I suggest you try removing the package before reinstalling it.

I won't have time to work on anything related to gh for the next 1 1/2 weeks.

If you're feeling adventurous (or just nothing works already), I encourage you to try the 2.0.x branch here.

@thulioph
Copy link

Sorry @henvic, but i forgot this. I checked my version and is 2.4.1.

I don't know how to help with this problem, so i'm waiting 😞

@zbeekman
Copy link

I’m having this problem too. Also, it seems that npm -v gh is returning the version of NodeJS, not gh.

$ npm -v gh
2.5.1
$ node -v
v0.10.36
$ npm -v NodeJS
2.5.1

$ gh help
gh First we need authorization to use GitHub's API. Login with your GitHub account.
? Enter your GitHub user: zbeekman
? Enter your GitHub password: ********
? Enter your two-factor code: 713383

/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules/rx/dist/rx.js:4586
        throw e;
              ^
TypeError: Cannot set property 'X-GitHub-OTP' of undefined
    at PromptUI.completed (/usr/local/lib/node_modules/gh/lib/cmds/user.js:179:45)
    at PromptUI.onCompletion (/usr/local/lib/node_modules/gh/node_modules/inquirer/lib/ui/prompt.js:68:10)
    at AnonymousObserver.Rx.AnonymousObserver.AnonymousObserver.completed (/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules/rx/dist/rx.js:1929:12)
    at AnonymousObserver.Rx.internals.AbstractObserver.AbstractObserver.onCompleted (/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules/rx/dist/rx.js:1866:14)
    at AutoDetachObserverPrototype.completed (/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules/rx/dist/rx.js:4604:23)
    at AutoDetachObserver.Rx.internals.AbstractObserver.AbstractObserver.onCompleted (/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules/rx/dist/rx.js:1866:14)
    at AnonymousObserver._onCompleted (/usr/local/lib/node_modules/gh/node_modules/inquirer/node_modules/rx/dist/rx.js:3)

I reported it as a new bug, since there only seems to be this PR, see #347

@henvic henvic self-assigned this Feb 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants