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 undefined attachment name when headers use "filename*=" format #301

Closed
wants to merge 4 commits into from

Conversation

JulienChavee
Copy link

@JulienChavee JulienChavee commented Oct 16, 2022

Hello,

A little PR as a first step to fix attachment names being set as "undefined" if the header uses the format "filename*=".
I've also added some unit tests to cover the change I made (and updated to phpunit version).

Feel free to review it and drop any comments :)

@Webklex
Copy link
Owner

Webklex commented Jan 2, 2023

Hi @JulienChavee ,
thanks a lot for your pull request. Unfortunately I wasn't able to merge your pull request. However I would like to acknowledge your work and your suggestions which indeed influenced my changes. Please feel free to simply push a " " or "." - I'll happily merge it so you'll get listed as a contributor.

Best regards and a happy new year,

@JulienChavee
Copy link
Author

Hello @Webklex

Thank you for taking time to review the pull requests, and I am glad that you were able to merge it on your side :)
There will be other opportunities to contribute in the future :)

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.

2 participants