search

nate-strauser / wework

We Work Meteor, a job board and developer directory for Meteor specific work
https://www.weworkmeteor.com
MIT License
287 stars 145 forks source link

GitHub signup: "Internal server error" #24

Closed jorisbontje closed 9 years ago

jorisbontje commented 9 years ago

from the logs:

Exception while invoking method 'login' undefined

This happens both locally and when using http://www.weworkmeteor.com/sign-up

nate-strauser commented 9 years ago

i just deleted my github based user, then signed up successfully with it

i do see this in the logs though Exception while invoking method 'login' { stack: undefined, source: 'method' }

do you get anything more useful for debugging when the error happens locally?

jorisbontje commented 9 years ago

After changing overrideLoginErrors to false in lib/accounts.js, I get the following exception instead:

Exception while invoking method 'login' Error: Full Name is required
    at getErrorObject (packages/aldeed:collection2/collection2.js:363:1)
    at [object Object].doValidate (packages/aldeed:collection2/collection2.js:346:1)
    at [object Object].Mongo.Collection.(anonymous function) [as insert] (packages/aldeed:collection2/collection2.js:147:1)
    at Object.Accounts.insertUserDoc (packages/accounts-base/accounts_server.js:1042:1)
    at Object.Accounts.updateOrCreateUserFromExternalService (packages/accounts-base/accounts_server.js:1189:1)
    at [object Object].Package (packages/accounts-oauth/oauth_server.js:45:1)
    at packages/accounts-base/accounts_server.js:383:1
    at tryLoginMethod (packages/accounts-base/accounts_server.js:186:1)
    at runLoginHandlers (packages/accounts-base/accounts_server.js:380:1)
    at [object Object].Meteor.methods.login (packages/accounts-base/accounts_server.js:434:1)
Sanitized and reported to the client as: Full Name is required [400]
nate-strauser commented 9 years ago

ah ok - i can make that profile field optional

On Thu, Feb 26, 2015 at 10:19 AM, jorisbontje notifications@github.com wrote:

After changing overrideLoginErrors to false in lib/accounts.js, I get the following exception instead:

Exception while invoking method 'login' Error: Full Name is required at getErrorObject (packages/aldeed:collection2/collection2.js:363:1) at [object Object].doValidate (packages/aldeed:collection2/collection2.js:346:1) at [object Object].Mongo.Collection.(anonymous function) as insert at Object.Accounts.insertUserDoc (packages/accounts-base/accounts_server.js:1042:1) at Object.Accounts.updateOrCreateUserFromExternalService (packages/accounts-base/accounts_server.js:1189:1) at [object Object].Package (packages/accounts-oauth/oauth_server.js:45:1) at packages/accounts-base/accounts_server.js:383:1 at tryLoginMethod (packages/accounts-base/accounts_server.js:186:1) at runLoginHandlers (packages/accounts-base/accounts_server.js:380:1) at [object Object].Meteor.methods.login (packages/accounts-base/accounts_server.js:434:1) Sanitized and reported to the client as: Full Name is required [400]

— Reply to this email directly or view it on GitHub https://github.com/nate-strauser/wework/issues/24#issuecomment-76195824.

nate-strauser commented 9 years ago

try again, let me know if it is still an issue -- i must have more fields filled out on my github profile or something, guessing that's why it works for me

jorisbontje commented 9 years ago

yep, that works now. (even though the getCurrentUserDisplayName in topnav will be left empty if you dont have a full name setup on github).

thanks!