search

Lepozepo / cloudinary

MIT License
94 stars 42 forks source link

Error when uploading (server side) #44

Closed arnoldgamboa closed 8 years ago

arnoldgamboa commented 8 years ago

Hi,

I'm getting this error from the server side while uploading the image:

Exception while invoking method 'c.sign' ReferenceError: check is not defined

Any idea?

Lepozepo commented 8 years ago

Did you update to 1.2? Maybe I need to check for the existence of check before running it, I thought it was part of the core. Look for meteor's check package and add it if you need it to work ASAP, if not I'll patch it up tonight if I get a chance.

arnoldgamboa commented 8 years ago

Yes, it's 1.2. So, this might be due to the upgrade?

arnoldgamboa commented 8 years ago

I am able to make it work by adding "check" and "random" packages. Thanks.

Lepozepo commented 8 years ago

Excellent, that will affect all your meteor methods though, make sure to use check in every method you have now. This is good for security anyway if you're planning on taking your app to production someday.

arnoldgamboa commented 8 years ago

Oh, I never thought it would. Hmm.. might not be a good way to do it since my codebase is huge already. I'll just wait for your update, then?

On Thu, Sep 24, 2015 at 6:55 AM, Marcelo Reyna notifications@github.com wrote:

Excellent, that will affect all your meteor methods though, make sure to use check in every method you have now. This is good for security anyway if you're planning on taking your app to production someday.

— Reply to this email directly or view it on GitHub https://github.com/Lepozepo/cloudinary/issues/44#issuecomment-142754822.

Lepozepo commented 8 years ago

Sounds good

Marcelo Reyna

On Wed, Sep 23, 2015 at 3:57 PM, Arnold Gamboa notifications@github.com wrote:

Oh, I never thought it would. Hmm.. might not be a good way to do it since my codebase is huge already. I'll just wait for your update, then? On Thu, Sep 24, 2015 at 6:55 AM, Marcelo Reyna notifications@github.com wrote:

Excellent, that will affect all your meteor methods though, make sure to use check in every method you have now. This is good for security anyway if you're planning on taking your app to production someday.

— Reply to this email directly or view it on GitHub https://github.com/Lepozepo/cloudinary/issues/44#issuecomment-142754822.

Reply to this email directly or view it on GitHub: https://github.com/Lepozepo/cloudinary/issues/44#issuecomment-142755048

Lepozepo commented 8 years ago

The code is checking for check now ^_^