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

Allow tabs as separators between attributes #45

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

Conversation

FliegendeWurst
Copy link

@FliegendeWurst FliegendeWurst commented Jan 6, 2017

An example of this can be seen in the updated test :)

I am actually not sure whether this should be allowed or not..

@seanmonstar
Copy link
Member

I somehow didn't notice this pull request, I'm so sorry!

Is there a situation you find where mime types include tabs? Quickly looking in the specs, it doesn't seem to mention tabs...

@FliegendeWurst
Copy link
Author

I remember Yahoo returning some in their mail headers.

@seanmonstar
Copy link
Member

I'd love to see an example, if you have one.

@FliegendeWurst
Copy link
Author

Sadly I dont have an example anymore :(

@rustonaut
Copy link
Contributor

rustonaut commented Oct 24, 2017

Mime in Mail can actually have tabs in practice[1] (and "\r\n " / "\r\n\t" sequences if the content type is split over multiple lines...).

For mail " " is basically equivalent to "\t", so if a mail program does insert WSP before or after after the ";" having "\t" is totally possible, just a rare choice.

So supporting it is a good idea for robustness.

[1]: and I think in the standard too, just that you have to cross read more than one standard to be sure :=(

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.

3 participants