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

Update OutlookRecipient.java #68

Merged
merged 1 commit into from
Oct 24, 2023
Merged

Conversation

dwlabcube
Copy link
Contributor

Correction for the X500 Address In Exchange. Now it is on the correct if statement. #67

Correction for the X500 Address In Exchange. Now it is on the correct if statement.
@bbottema bbottema merged commit 808cfe7 into bbottema:master Oct 24, 2023
@mhw
Copy link

mhw commented Nov 13, 2023

Just as a data point, some Outlook message samples I'm working with have X500 addresses where the organisation part is not ExchangeLabs. That is the address values start with /o={some other value}/ou=Exchange Administrative Group. I don't know how widespread this is, and I'd guess it depends on the way that Active Directory is being used. But it does mean that the prefix match in this PR would not match in their case.

@bbottema
Copy link
Owner

bbottema commented Dec 3, 2023

Thanks @mhw, I've added a bit more comprehensive heuristic to test for X500 addresses. Released in 1.10.2.

@bbottema
Copy link
Owner

bbottema commented Dec 3, 2023

I've modified the heuristic for detecting X500 addresses in 1.10.2. Perhaps you can verify your case still works.

@bbottema bbottema added this to the 1.10.2 milestone Dec 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants