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

Improve error handling #223

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

Improve error handling #223

wants to merge 18 commits into from

Conversation

drallan
Copy link

@drallan drallan commented Jan 7, 2020

No description provided.

daleallan and others added 18 commits May 10, 2016 11:46
Add check for Bytes in attachments before assigning.
Add checks for subject, date & size on the message object before atte…
Check for $address->host before assigning it
Bring up to date with upstream
We were receiving a lot of errors with mandarin strings,
This fix will change unknown characters to `?` and prevent `iconv()` throwing an error

Signed-off-by: Marnu Lombard <[email protected]>
Instead of using the passed in encoding which could contain `//TRANSLIT` or `//IGNORE`,
Use the detected encoding of the string that `imap_mime_header_decode()` supplies us.

Signed-off-by: Marnu Lombard <[email protected]>
Pull in upstream changes
Convert encoding of strings before running through `iconv()`
allow nested attachments in attached emails of email
 - replace any characters that are not letters
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.

5 participants