SonarSoftwareInc / customer_portal

A prebuilt customer portal for Sonar.
https://sonar.software
25 stars 52 forks source link

Two issues. Emails are case sensitive & Unable to update profile information #72

Open chacken opened 4 years ago

chacken commented 4 years ago
sonar-customerportal | May 27 13:40:10 cb9a49c9257e laravel[128]: production.INFO: Unable to find email address in the system. [] []
sonar-customerportal | May 27 13:40:59 cb9a49c9257e laravel[128]: production.INFO: Unable to find email address in the system. [] []
sonar-customerportal | May 27 13:43:23 cb9a49c9257e laravel[127]: production.ERROR: Array to string conversion [] []
sonar-customerportal | May 27 13:43:32 cb9a49c9257e laravel[128]: production.ERROR: Array to string conversion [] []
sonar-customerportal | May 27 13:43:39 cb9a49c9257e laravel[128]: production.ERROR: Array to string conversion [] []

Based on a quick scan of the code, the email issue seems like it's not fixable because it's getting the email from the Sonar instance. It'd be great if both the portal and sonar instance did a strtolower() before comparing account emails to see if the account exists.

As for the other issue, anytime you try to update information in a customers account in the portal, we get a "Failed to update profile. Please try again later." Likely due to the array issue above.