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

[BUG] Email.AttachFromFilename does not dispose stream #23

Closed
asherber opened this issue Mar 30, 2024 · 3 comments
Closed

[BUG] Email.AttachFromFilename does not dispose stream #23

asherber opened this issue Mar 30, 2024 · 3 comments

Comments

@asherber
Copy link

It seems that GitHub won't let me fork this repo, because I already have a fork of lukencode/FluentEmail. But please take a look at lukencode#384 and implement that fix here -- disposing of the file stream after reading the file to attach.

@MarkMenchavez
Copy link

Do you know if anyone is working on this already? if not, I can work on it.

@asherber
Copy link
Author

asherber commented Sep 1, 2024

There's already a suggested fix here. I just can't fork this repo because I already have a fork of the source repo.

@JohnCampionJr
Copy link

Thanks! I'll review and get it in ASAP. You should be able to fork into a differently named repo though.

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

No branches or pull requests

3 participants