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

Integrate all previous PRs #44

Closed
wants to merge 10 commits into from
Closed

Integrate all previous PRs #44

wants to merge 10 commits into from

Conversation

k3a
Copy link

@k3a k3a commented Aug 26, 2023

I integrated all open PRs into my fork:

Some of them required manual merging but the resulting code should be correct, hopefully.

I am providing this as a PR to the original repo first.
The library is quite nice but the original repo seems to be unmaintained for more than 3 years and the library is largely broken without those PRs.

If there is no response by the original author in a week, I will attempt to make my repo to be some kind of 'official fork', set up CI for it and handle any future PRs.

drewtempelmeyer and others added 9 commits October 23, 2022 17:11
Fixes the issue for:
```
unknown encoding: BASE64
```
where we initially check whether part is the attachments.
Rather doing that in the last step, otherwise attachments with mimetype
of text/html would not be extracted out of the eml file
@k3a k3a force-pushed the master branch 5 times, most recently from f7e3687 to 15c86d4 Compare September 2, 2023 19:29
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.

None yet

4 participants