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

Not reading arabic characters while fetching from outlook.com #274

Closed
avinash403 opened this issue Jan 16, 2019 · 1 comment
Closed

Not reading arabic characters while fetching from outlook.com #274

avinash403 opened this issue Jan 16, 2019 · 1 comment
Labels
needs investigation This will be tested / debugged or checked out.

Comments

@avinash403
Copy link

Firstly it wasn't fetching mails at all from outlook.com, and was showing

IMAP method imap_search() failed with error: [BADCHARSET (US-ASCII)] The specified charset is not supported.

For a workaround I changed the encoding for cases where this exception comes. Now if email body has arabic characters, it skips them and fetches the rest of the body.

STEPS TO REPRODUCE:
Fetch mail from an outlook account with encoding as US-ASCII and body as below

مقتطفات من: صن تزو. "فن الحرب". كتب أبل. Something in english

After fetching body will only have

Something in english
@Sebbo94BY Sebbo94BY mentioned this issue May 3, 2019
Closed
@Sebbo94BY Sebbo94BY added the needs investigation This will be tested / debugged or checked out. label May 4, 2019
Sebbo94BY pushed a commit that referenced this issue May 11, 2019
Issue #274: Some strings with default charset had UTF-8 characters an…
04e01b9 
…d failed to convert
@Sebbo94BY
Copy link
Collaborator

Fixed with above referenced commit. Will be included in the next release.

@Sebbo94BY Sebbo94BY mentioned this issue May 11, 2019
Merged
Sebbo94BY added a commit that referenced this issue May 11, 2019
@Sebbo94BY
Mainly fixed decoding issues
ab18be7 
- Updated comments, made code a bit more dynamic, used setAttachmentsDir() in constructor
- Added condition for setting attachments dir in constructor
- Issue #232: Added test strings with emojis (smiles) to PHPUnit test
- Issue #232: Added strings to test decodeMimeStr() in PHPUnit test with non-UTF8 server encoding
- Issue #232: Fixed broken UTF-8 decoded strings, when not using UTF-8 as server encoding
- Issue #265: Added string to test iconv() in PHPUnit test
- Issue #274: Some strings with default charset had UTF-8 characters and failed to convert
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs investigation This will be tested / debugged or checked out.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Sebbo94BY @avinash403