Skip to content

Adds support for fetching cached images from an HTTP(S) source#8

Open
brandonhudson wants to merge 1 commit into
jbeuckm:masterfrom
brandonhudson:master
Open

Adds support for fetching cached images from an HTTP(S) source#8
brandonhudson wants to merge 1 commit into
jbeuckm:masterfrom
brandonhudson:master

Conversation

@brandonhudson
Copy link
Copy Markdown

Adds support for fetching cached images from an external source (like AWS S3) that provide the cached image via a url. The current implementation fails with a 500 error because the image is not located on the filesystem. fs.readFile... only supports fetching files located on the filesystem (reference).

Comment thread services/ImageFormats.js

let buffer = null;

# If the url is an HTTP url, get the image from the external source
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not a valid js comment. Try // or /* .. */

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