Skip to content
This repository has been archived by the owner on Sep 2, 2021. It is now read-only.

Fix the internal server error #57

Merged
merged 1 commit into from
Aug 6, 2014
Merged

Conversation

ingorichter
Copy link
Contributor

There was no bug filed for this issue, but this error happened after we deployed the latest version of the registry with the download data tracking feature.
The order of statements is important for passport, otherwise user couldn't authenticate with github and the server returned with a 500 internal server error.

@ingorichter
Copy link
Contributor Author

@dangoor There was an issue 2 weeks ago and this little change fixed it. I have applied those changes on the production machine, but haven't yet created the PR.

@ingorichter
Copy link
Contributor Author

Merged, since this is already running on production for more than a month.

ingorichter added a commit that referenced this pull request Aug 6, 2014
@ingorichter ingorichter merged commit 4d13856 into master Aug 6, 2014
@dangoor dangoor deleted the ingo/fix-internal-server-error branch August 6, 2014 20:47
@dangoor
Copy link
Contributor

dangoor commented Aug 6, 2014

Ahh, sorry for not getting to this. It's a tiny change and perfectly sensible.

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.

2 participants