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

wallabag can't handle plain text #444

Closed
lightonflux opened this issue Feb 12, 2014 · 17 comments
Closed

wallabag can't handle plain text #444

lightonflux opened this issue Feb 12, 2014 · 17 comments

Comments

@lightonflux
Copy link

wallabag can't handle plain text

E.g. http://ibiblio.org/pub/linux/docs/ldpResearch/ldp-historic/LinuxNews.03A

@tcitworld tcitworld added Bug and removed Bug labels Feb 12, 2014
@tcitworld
Copy link
Member

This may be a little more difficult than it seems. We have to check the text/plain http header before starting to get all the content.

@nicosomb nicosomb modified the milestones: 1.5.0, 1.6.0 Feb 12, 2014
@mariroz
Copy link
Contributor

mariroz commented Feb 21, 2014

this is next challenge, related to db structure expansion. Adding of "mime_type" field will allow wallabag to handle also other formats, for example we will be able to store pdf. This is good direction of growth, on my opinion. I hope, will be able to to test @nicosomb's install script next week and will try to think about accurate and secure db changes.

@nicosomb
Copy link
Member

@mariroz you want to add a new column in entries table? To store mimetype? Good idea. But, I'm "chilly" to database modification ... wallabag is not well structured, so upgrades are quite sensitive.

@nicosomb
Copy link
Member

nicosomb commented Apr 6, 2014

@mariroz do you think this feature is possible for wallabag 1.x?

@mariroz
Copy link
Contributor

mariroz commented Apr 7, 2014

I hope, that yes :)

@nicosomb nicosomb modified the milestones: 1.8.0, 1.7.0 Apr 7, 2014
@tcitworld
Copy link
Member

If we modify database structure, could do it once and for all, like adding an is_administrator column in users, a comments column in entries, etc ?

@mariroz
Copy link
Contributor

mariroz commented Apr 7, 2014

@tcitworld, yes, of course we will need more columns.
This is complex task: should be done together with wallabag upgrade (and maybe install) procedure modification.

@tcitworld
Copy link
Member

I'll open a new issue specifically for this.

@tcitworld tcitworld modified the milestones: 2.0, 1.8.0 Jun 7, 2014
@tcitworld
Copy link
Member

Going for v2.0.

@nicosomb
Copy link
Member

I just tried the URL in the 1st post and this one for example (http://www.oracle.com/technetwork/java/readme-2-149793.txt) with wallabag v2. And it works.

Is this issue still mandatory?

@tcitworld
Copy link
Member

And I heard someone saved pdfs. So I guess we're good, with MIME Type now stored.

@nicosomb
Copy link
Member

mimetype is not yet stored.
About PDF, it's not the document content.

@tcitworld
Copy link
Member

No, but the thing is : the url is saved, with the title.

@nicosomb
Copy link
Member

yes 👍

about mimetype, I don't know if it's easy. Because you can't get the URL "mimetype", it doesn't exist ...

@tcitworld
Copy link
Member

Of course you can, it's the Content-Type http header. ;-)

EDIT : https://stackoverflow.com/questions/2610713/get-mime-type-of-external-file-using-curl-and-php

@nicosomb
Copy link
Member

oh yes, you're right.

@j0k3r
Copy link
Member

j0k3r commented Sep 16, 2015

Fixed in graby.

@j0k3r j0k3r closed this as completed Sep 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants