Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sign up returns error #14

Closed
msolomonTMG opened this issue Jun 5, 2018 · 3 comments · Fixed by #23
Closed

Sign up returns error #14

msolomonTMG opened this issue Jun 5, 2018 · 3 comments · Fixed by #23

Comments

@msolomonTMG
Copy link
Owner

msolomonTMG commented Jun 5, 2018

Sometimes, when a user clicks a signup link from Slack, they are presented with an error rather than the Jira OAuth screen.

Steps to Reproduce

  1. Message the slackbot with "signup" with a slack user who is not yet signed up
  2. Click the signup link that is returned
    Expected: user is presented with Jira OAuth screen
    Actual: user is presented with an error message

Observations:

  • mongodb shows the user signed up with their slack username but not their Jira username
  • if the user is deleted from the mongodb and tries signing up again, signup is successful
  • this signup issue appears to be related to the amount of time it takes heroku to wake up after going to sleep because signups that occur when heroku is awake tend to be successful
@edahlseng
Copy link
Collaborator

I've seen a similar error before as well, without using Heroku. It may be related to an uncaught promise, if I remember right. In any case, I'll check it out more fully.

@jholmstadt
Copy link

I spent a little time trying to figure out a fix for this, and came up with something that seems to work fine for our deployment: teamlakana@ced3e71

I don't know much about nodeJS, however it seemed to me that if the user exists in the DB, then all that happens is that it logs to the console, and eventually the request times out because there is no response. Instead, I made it respond with a redirect to the /auth URL so that the user can complete the JIRA authorization. That could lead to multiple JIRA auth attempts, but that doesn't seem to harm anything.

@agfa555
Copy link

agfa555 commented Dec 12, 2018

Hi, first of all thank you for this.
We find this idea super useful but I'm stuck in this same step.

Everything was configured correctly but when trying to signup I get the error message.
Also, if I try to signup again I get the message "You're already signed up!" and a link: "Click here to adjust your settings"
If I click that I get to a screen that says forbidden.

I also tried @jholmstadt fork with the same result. :'(

Any help would be greatly appreciated.

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

Successfully merging a pull request may close this issue.

4 participants