8000 fix issue #659, compatible attachment description """" parsing. by xiaoquanidea · Pull Request #679 · emersion/go-imap · GitHub
[go: up one dir, main page]
Skip to content

fix issue #659, compatible attachment description """" parsing. #679

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
Open
wants to merge 2 commits into
base: v2
Choose a base branch
from
Open

fix issue #659, compatible attachment description """" parsing. #679

wants to merge 2 commits into from

Conversation

xiaoquanidea
Copy link
@xiaoquanidea xiaoquanidea commented Mar 25, 2025 •
edited by emersion
Loading

Closes #659

@emersion
Copy link
Owner

Not sure I want to merge this workaround TBH. The syntax is completely broken here.

@xiaoquanidea
Copy link
Author
xiaoquanidea commented Mar 26, 2025 •
edited
Loading

Not sure I want to merge this workaround TBH. The syntax is completely broken here.

After observing for several months, I found that there are still quite a few improperly formatted emails. These emails can be correctly parsed by webmail clients and built-in mail applications on computers, but they fail to be parsed using go-imap, causing the program to miss emails, which is quite frustrating for me. If possible, I hope you can find a more elegant solution in line with go-imap’s design.

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.

v2：imapclient fetch message, parse FetchMessageBuffers error
2 participants
@xiaoquanidea @emersion
0