Lepozepo / cloudinary

MIT License
94 stars 42 forks source link

Security: checking all arguments #24

Closed workflow closed 9 years ago

workflow commented 9 years ago

Hi @Lepozepo,

thx for this package!

With audit-argument-checks enabled, your package is throwing the following:

Exception while invoking method 'cloudinary_upload' Error: Did not check() all arguments during call to 'cloudinary_upload'

Could you include a check() in all methods?

Lepozepo commented 9 years ago

Definitely! I'll patch it this weekend hopefully.

Lepozepo commented 9 years ago

I've added checks for the new version but there are a lot of breaking changes so be sure to read the docs before updating.

workflow commented 9 years ago

Awesome, thx!