Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Image Plugin] Fix MIME type and encoding #6

Merged
merged 3 commits into from
Jun 20, 2014

Conversation

FlorianFieber
Copy link
Contributor

Allows to actually display images from clipboard and images with faulty extensions in clients strictly adhering to the claimed MIME type.

1. The file extension might not be the correct MIME type, and is empty
   for images extracted from the clipboard
2. The image is converted to a JPEG, anyway
@Ri0n
Copy link
Member

Ri0n commented Jun 19, 2014

I believe we should somehow consider http://tools.ietf.org/html/rfc3548#section-4

@FlorianFieber
Copy link
Contributor Author

I don't think so: http://tools.ietf.org/html/rfc2397#section-4

After digging a bit deeper, https://tools.ietf.org/html/rfc4648#section-5 suggest encoding '=', but then again rfc2397 doesn't specify to use base64url, and I see no conflict as described in https://tools.ietf.org/html/rfc3986#section-2.2

@Ri0n
Copy link
Member

Ri0n commented Jun 19, 2014

ok. agreed.

wadealer added a commit that referenced this pull request Jun 20, 2014
[Image Plugin] Fix MIME type and encoding
@wadealer wadealer merged commit 3051cbf into psi-im:master Jun 20, 2014
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants