Kiko Beats

Hey, @n1ru4l I understand your use case but I prefer to don't add a new environment variable it could be a thing we can avoid. What we can do to...

@n1ru4l not seeing that is an issue since query parameters are ignored by S3 files. e.g., https://s.microlink.io/kdsy_4QyLo6Kh61e8SOtMcxCtWg_.png?foo=bar&platform=darwin

@yafp not really – most of the time I just get the JSON version of the payload, skipping the download, so I'm not treating with this kind of problems

@brunobg happy so ship it as major version; keep it as a separate project is also comprehensible 👍

Could be a problem related to how the url is interpreted since the `-` can be mixed with flags parser 🤔

@DiegoFleitas can you add the example to README? 🙂

I think flags need to be passed following this way: https://github.com/microlinkhq/metascraper/blob/master/packages/metascraper-media-provider/src/get-media/provider/generic.js#L15 and of course, `skip_download` is a not necessary thing using `.getInfo`

I didn't face with this since I do regular npm install on production and the version tends to be the latest – but yea, PRs are very welcome enhancement it...

@JaysQubeXon That's a really good catch, thanks for that! Please go ahead with the PR, happy to merge is asap 🙂

Can you make a PR?