Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug > Email Query > Cannot filter by "From" when keyword is domain name #3945

Open
tlle14 opened this issue Oct 23, 2020 · 8 comments
Open
Labels
bug minor Priority of a bug - QA priorization tool moderate Difficulty of a bug - QA priorization tool

Comments

@tlle14
Copy link

tlle14 commented Oct 23, 2020

Actual Result

When using the filter (from/to/cc/bcc), JMAP cannot filter emails by the special domain name like qa.open-paas.org - which has 2 dots and a hyphen, while able to filter with other domain names (linagora.com or gmail.com)

Cannot filter by domain name
image

Able to filter by a normal domain name
image
image
image

Expected Result

Please investigate. It should able to filter a valid domain name

Environment

DEV environment

@tlle14 tlle14 added bug minor Priority of a bug - QA priorization tool labels Oct 23, 2020
@chibenwa
Copy link
Member

What's the full email of the user in from field of mail "fb3..."?

Can you share the EML of that mail?

@chibenwa
Copy link
Member

Functionally speaking filtering by domain part is likely uneeded.

@chibenwa chibenwa added the moderate Difficulty of a bug - QA priorization tool label Oct 26, 2020
@tlle14
Copy link
Author

tlle14 commented Oct 26, 2020

Functionally speaking filtering by domain part is likely uneeded.

I was thinking about the case: user wants to search all email from the specified domain in the mailbox

@chibenwa
Copy link
Member

Yes it is a "nice to have", not more IMO

@tlle14
Copy link
Author

tlle14 commented Oct 26, 2020

Hi @chibenwa

I need to update the bug description since it happens only with the domain contains special characters like qa.open-paas.org
I'm still able to filter with other domain, such as gmail.com or linagora.com.

Could you help to confirm?

@chibenwa
Copy link
Member

This is then due to our ES tokenizer: - is likelyconsidered as a separator.

I will clarify the impact (low) and difficulty (hard) as changing mapping and reindexing all data is needed to solve this.

Thanks for the clarification, we will add failing unit test cases to document this bug but likely will not fix it directly.

@chibenwa
Copy link
Member

@chibenwa
Copy link
Member

#3972 demonstrates the issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug minor Priority of a bug - QA priorization tool moderate Difficulty of a bug - QA priorization tool
Projects
None yet
Development

No branches or pull requests

2 participants