User urn based on email should not be case-sensitive
See original GitHub issueDescribe the bug A clear and concise description of what the bug is.
To Reproduce Steps to reproduce the behavior:
- Go to datahub and have the identity provider create your user
- Ingest some metadata into datahub using the CLI, but have username email in all caps
- Observe you are not owner of the dataset you just ingested because it’s case-sensitive
Expected behavior
II think user@user.com
and user@USER.COM
should be the same. In general, we should not allow usernames that only differ in the casing.
Possible solution Lower-case all usernames that are emails.
If user user@domain.com
adds themselves as a dataset owner in an ingestion recipe, the dataset is not owned by them if their email in the identity provider is something like user@DOMAIN.COM
even though in terms of email address this is the same. I think in the case of users, the behavior should be to always lowercase, and we should not allow duplicate usernames in datahub which only differ by casing.
Issue Analytics
- State:
- Created 2 years ago
- Comments:5 (3 by maintainers)
Top Results From Across the Web
User email should not be case sensitive [#2490294] | Drupal.org
1. Storage must preserve case · 2. Assuming that email querying is case insensitive is a security liability · 3. Making querying by...
Read more >Are email addresses case sensitive? - Stack Overflow
So yes, the part before the "@" could be case-sensitive, since it is entirely under the control of the host system. In practice...
Read more >Are Email Addresses Case Sensitive? And Other ... - Mailchimp
Email addresses are not case sensitive. Learn about best practices for creating an email address and signature, whether it's a personal or business...
Read more >Are Email Addresses Case Sensitive? - Sendinblue
No. Email addresses are not case sensitive. Having letters in all lowercase makes the email address easier to read, but the oversight won't ......
Read more >Solved: How to apply a case insensitive condition to an em...
I'm trying to write a condition that if the email subject contains the words "new employee" that a flow is triggered. However it...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
I’m also running into this issue. My org doesn’t use a consistent casing for primary email identifiers in Azure AD, so it is not possible to transform ingested emails to the correct form without doing lookups. Being able to downcase usernames would solve this, either in all ingestion sources and identity providers, or as a setting in datahub itself. Alternatively, maybe we could have an option to use the ‘mail’ field that exists today in datahub as an identifier as well? In this case we’d know for sure that it’s an email and shouldn’t be case-sensitive.
I don’t use DataHub anymore 😦 If anyone faces this issue I assume they will create their own ticket on the new issue tracker.