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

Fix errors when attachment name contains non-ASCII characters #76

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SebDieBln
Copy link

With a file whose name contains non-ASCII characters, errors where shown when

  • attaching the file
  • viewing the attached file

This was due to the name being processed as an ASCII string and is fixed by this PR.

@rjollos rjollos self-assigned this May 11, 2018
@SebDieBln SebDieBln force-pushed the AttachmentNameUtf8 branch 2 times, most recently from 24e4a16 to 5289bc3 Compare September 22, 2019 17:42
@rjollos
Copy link
Member

rjollos commented Apr 24, 2020

@SebDieBln Let me know if you are interested in having commit access.

@SebDieBln
Copy link
Author

@rjollos It has been a while but I finally found the time to work on this project again (preparing to upgrade our Trac vom 1.0.9 to 1.4.x). And yes, I feel honoured and would appreciate commit access!

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

2 participants