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

Display bin image file as image. #17

Merged
merged 13 commits into from
Jan 31, 2013
Merged

Display bin image file as image. #17

merged 13 commits into from
Jan 31, 2013

Conversation

Natim
Copy link
Contributor

@Natim Natim commented Jan 19, 2013

No description provided.

@sametmax
Copy link
Contributor

sametmax commented Jan 2, 2013

That's very interesting. We didn't think about encrypted images. Currently, I didn't even think it would support it because the maximal file size is so low and we don't bother with checking if it's binary or not.

So:

  • does it work (beside display) ?
  • if yes, do you have an example URL ?
  • is the file size limit (and behavior) sufficent enough ?
  • is there somebody else interested in adding image support ?

Nice idea anyway. I'll prompt the community about this.

@Natim
Copy link
Contributor Author

Natim commented Jan 2, 2013

Here is the example:

http://0bin.net/paste/85a57c33d5239a3e6442a6e1e757d3ea341664bd#0eIjVYLuFA4hfAzK9dbHq82vXz/SrbrCwlH8N5VJ5zs=

I don't know what is the file limit but since it is a TextArea I don't think it is a problem.

@Natim
Copy link
Contributor Author

Natim commented Jan 2, 2013

It could be even better to implement the drag and drop in the textarea as it works with Github.

@sametmax
Copy link
Contributor

sametmax commented Jan 2, 2013

Ok. If we implement your other issue, we would have the mime/type, so we could definitly add this image feature.

It would probably require to seriously clean up the code because it's not modular at all for now: the workflow is kinda one big block. But it seems with it.

Drag and drop as well, but it should be another opened issue.

@scostes
Copy link

scostes commented Jan 2, 2013

Hello,

We are interested too. But TBH, i personnally know natim ;)

In fact, we are looking for a pastebin and imagebin ( http://imagebin.org/ ) to be installed on our own server, in order to not rely on external service like friendpaste ou gist.

Regarding the file size, since we have installed it locally, we can maximize withtout trouble.

Apology for my english.

Regards,

@sametmax
Copy link
Contributor

sametmax commented Jan 2, 2013

Very good. I'm glad you wish to install that on your own server. The
more 0bin there is out there, the more the future of the system will be
reliable.

Le 02/01/2013 13:37, scostes a écrit :

Hello,

We are interested too. But TBH, i personnally know natim ;)

In fact, we are looking for a pastebin and imagebin (
http://imagebin.org/ ) to be installed on our own server, in order to
not rely on external service like friendpaste ou gist.

Regarding the file size, since we have installed it locally, we can
maximize withtout trouble.

Apology for my english.

Regards,


Reply to this email directly or view it on GitHub
#17 (comment).

@scostes
Copy link

scostes commented Jan 2, 2013

I'm not just wishing, i've already done it ;)

I'm looking forward to pip upgrade it and have image paste support

@sametmax
Copy link
Contributor

sametmax commented Jan 2, 2013

\o/

Le 02/01/2013 14:19, scostes a écrit :

I'm not just wishing, i've already done it ;)

I'm looking forward to pip upgrade it and have image paste support


Reply to this email directly or view it on GitHub
#17 (comment).

@Natim
Copy link
Contributor Author

Natim commented Jan 19, 2013

I need to work a little bit more to handle encryption error with some jpg files, but the result so far is quite nice.

@Natim
Copy link
Contributor Author

Natim commented Jan 19, 2013

Actually it seams that bottle cut the request body on too big messages. That's a shame but it seams to be the normal behavior over http the good solution, would be to send it has a file so it would be chunked.

@Natim Natim mentioned this pull request Jan 19, 2013
@Natim Natim mentioned this pull request Jan 19, 2013
@Natim
Copy link
Contributor Author

Natim commented Jan 20, 2013

Don't hesitate to contact me if you need explanation.

@sametmax
Copy link
Contributor

Ok, that's a big piece of work. I'll need some time to review it. I'll give it a try on monday morning.

@sametmax
Copy link
Contributor

I haven't forgot to check the code. It's cloned on my machine, it's just I couldn't find a good time to review it. Sorry for the delay.

@Natim
Copy link
Contributor Author

Natim commented Jan 22, 2013

"Je suis tellement impatient ;)"

@scostes
Copy link

scostes commented Jan 22, 2013

i've seen a demo, looks great from the user side !

@Natim
Copy link
Contributor Author

Natim commented Jan 24, 2013

A full working version of all these PR is available here : https://github.com/novagile/0bin on master.

@sametmax
Copy link
Contributor

Hi man,

Sorry I took so much time to review this.

I checkout your master and some commits here and there to be sure, but I can't make it work.

When I'm on Firefox (18.0.1), uploading the jped file just doesn't work (no error displayed in the Python / JS console).

When I use Chrome, I do save the image successfully, but it spit me back the binarie data instead of dispaying the image.

I'm using Ubuntu, so it could be a mime/type detection issue but I could find it by scanning the code.

@Natim
Copy link
Contributor Author

Natim commented Jan 31, 2013

Are you sure that the js file loaded is up-to-date?

@sametmax
Copy link
Contributor

Ok I suck, I just emptied my cache and it worked perfectly.

@sametmax
Copy link
Contributor

Can you update the PR from #17 to #21 ? I will merge it.

@Natim
Copy link
Contributor Author

Natim commented Jan 31, 2013

Ok done but we need to create a ticket to let the choice for is.gd.

@sametmax
Copy link
Contributor

Here you are #25

@sametmax
Copy link
Contributor

When I click on "merge pull request", it still offer me to merge #17. I'm not sure but I think you can update the original pull request itself t the choosen commit on the PR page.

@Natim
Copy link
Contributor Author

Natim commented Jan 31, 2013

Yes I merge everything I did in the #17 PR

sametmax added a commit that referenced this pull request Jan 31, 2013
Display bin image file as image.
@sametmax sametmax merged commit 0c1bcfd into Tygs:master Jan 31, 2013
@sametmax
Copy link
Contributor

I merged it, but it merged the google url shortener change as well. It's not the end of the world I guess.

@Natim
Copy link
Contributor Author

Natim commented Jan 31, 2013

It is why #25 exists ;)

@sametmax
Copy link
Contributor

Anyway, thanks for the work, Natim. It's nice to have people involved.

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

3 participants