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

Fixes multiple issues #55

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

elarivie
Copy link

This pull request contains many fixes

Temp file issues:

  • Use os library to identify the temp folder
  • Randomize temp file name to avoid race condition

fix #17
fix #24
fix #42

Max Buffer size issue:

  • Increased the max buffer size to infinity

fix #44

Other unreported issues which are fix:

No description for response section

  • Added it to the template

Not possible to embed image with data-uri

  • Well now it is possible (and its really useful)
    To embed an image use the following syntax:

    ![](data:image/gif;base64,…)
    

where … is the image data encoded in base64.
The syntax also works with other file format (png, jpg, etc.)

@elarivie elarivie changed the title Randomize temp file name to avoid race condition Fixes multiple issues Feb 19, 2016
hg-jt and others added 3 commits March 29, 2016 07:23
* elarivie/master:
  Set maxbuffer length to Infinity to avoid stdout maxBuffer exceeded fix danielgtaylor#44
  Removed hardcoded path to tmp, now uses os library to get it.
  Add support for embeded image with data uri
  Added missing description of response to the template
  Randomize temp file name to avoid race condition fix danielgtaylor#17 fix danielgtaylor#24 fix danielgtaylor#42
* hg-jt/fix/57-image-handling:
  Fix for danielgtaylor#57: filePath is now passed to resolveImages.
@maurobender
Copy link

👍

@Chosko
Copy link

Chosko commented Aug 5, 2016

Any chance to see this merged?

@mansona
Copy link

mansona commented Aug 25, 2016

Any Movement on this?

@nilskp
Copy link

nilskp commented Nov 3, 2016

@danielgtaylor Any chance of seeing this merged? Previewing in Atom fails on every other change.

@arkty
Copy link

arkty commented Jan 10, 2017

@danielgtaylor any status for this MR? Seems it can fix annoying "/tmp/atom.apib" issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
7 participants