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.

Sign up for GitHub

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

Jump to bottom

Use message codepage if body code page is not present #57

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Use message codepage if body code page is not present #57

wants to merge 4 commits into from

Conversation

mvz
Copy link
Owner

@mvz mvz commented Dec 16, 2023

This may resolve #49.

  • Fetch PR_MESSAGE_CODEPAGE property
  • Log found encoding info in verbose mode
  • Log if unicode encoding uses non-unicode code page
  • Use message codepage if body code page is not present

@mvz
Fetch PR_MESSAGE_CODEPAGE property
2012d6d
@mvz
Log found encoding info in verbose mode
addbf6c
@mvz
Log if unicode encoding uses non-unicode code page
120065f 
It seems this can happen because the listed codepage is the original
codepage and unicode was used to re-encode the original body text
@mvz
Use message codepage if body code page is not present
63d76e0
@mvz mvz force-pushed the improved-codepage-finding branch 2 times, most recently from 6238938 to 63d76e0 Compare December 16, 2023 11:12
@mvz
Copy link
Owner Author

mvz commented Dec 16, 2023

It does not seem useful to merge this unless it fixes #49 so let's wait and see if we can get a sample for that.

@ubpre ubpre mentioned this pull request Jan 16, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

a converted message have a broken encoding in a message body (have cp1251 in msg )
1 participant
@mvz