UCL-INGI / INGInious

INGInious is a secure and automated exercises assessment platform using your own tests, also providing a pluggable interface with your existing LMS.
http://www.inginious.org
Other
202 stars 139 forks source link

Github LTI authentification does not use primary github email #1004

Open AlexandreDoneux opened 3 months ago

AlexandreDoneux commented 3 months ago

Describe the bug When using GitHub to sign up the email address was set to one of my non-primary email addresses. It seems like it is using the oldest email address added to my GitHub account.

After suppressing the unwanted email addresses and trying again, this time the complete name was not pre-completed and the email address was one of an already existing user in DB. This may be caused by a lookup in DB with an empty username (complete name).

INGInious installation details

To Reproduce Steps to reproduce the behavior:

  1. Have a GitHub account with multiple email addresses and the first one not being the primary address.
  2. Click on 'sign in'.
  3. Select the GitHub logo below 'Sign in with'
  4. See error

Expected behavior Display the correct complete name and the primary GitHub email address.

Screenshots

Screenshot from 2024-05-28 11-26-39