zendesk_create_user() breaks when email address contains capital letters

Created on 3 February 2016, about 9 years ago
Updated 27 February 2023, about 2 years ago

When a user registers with an email containing capital letters and already has an existing account at zendesk, the zendesk_create_user() function will fail to call zendesk_sync_user_back() and issue a watchdog error.

Although simply doing a strtolower($account->mail) will "fix" this bug, there is a better way to do this using zendesk API POST to /users/create_or_update.json and the function zendesk_sync_user_back() can just be deleted. The better way means updating the zendesk.lib.php to return headers for at least this particular call.

I actually chatted with the zendesk folks and they recommended using the header response and discouraged this kind of odd 'string comparison' of the $result->details->email[0]->description.

πŸ› Bug report
Status

Needs work

Version

3.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States blasthaus

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024