question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

User urn based on email should not be case-sensitive

See original GitHub issue

Describe the bug A clear and concise description of what the bug is.

To Reproduce Steps to reproduce the behavior:

  1. Go to datahub and have the identity provider create your user
  2. Ingest some metadata into datahub using the CLI, but have username email in all caps
  3. 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:closed
  • Created 2 years ago
  • Comments:5 (3 by maintainers)

github_iconTop GitHub Comments

1reaction
iasooncommented, Jan 13, 2022

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.

0reactions
dingobarcommented, Oct 31, 2022

I don’t use DataHub anymore 😦 If anyone faces this issue I assume they will create their own ticket on the new issue tracker.

Read more comments on GitHub >

github_iconTop 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 >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found