itteco / meteor-oembed

Add oEmbed support to your Meteor.js app
https://iframely.com
MIT License
13 stars 8 forks source link

Exception while invoking method 'iframely.oembed' Error: Did not check() all arguments during call to 'iframely.oembed' #2

Open shadowzick opened 9 years ago

shadowzick commented 9 years ago

Hi, I tried using this package and i'm currently having this error while trying to use {{>oembed}} in a loop. Do you have any idea why? Thanks.

nleush commented 9 years ago

Looks lie server side method does not receive url argument in "iframely.oembed": function(url) {

Template context, where {{>oembed}} placed should contain url attribute. Maybe it doesn't.

If your url attribute is optional try use following check in widget template to prevent exception:

{{#if url}}
    {{>oembed}}
{{/if}}
shadowzick commented 9 years ago

thank you for the response! I actually already have that line of code. i'm wondering why it is still occurring.

nleush commented 9 years ago

What other packages do you use? What is meteor version?

shadowzick commented 9 years ago

Here are the packages: accounts-facebook
accounts-password
acemtp:meta-extractor
aldeed:collection2
audit-argument-checks
browser-policy
dburles:collection-helpers
fastclick
fortawesome:fontawesome
iframely:oembed
insecure
iron:router
jquery
less
manuelschoebel:ms-seo
matteodem:easy-security
meteor-platform
meteorhacks:fast-render
natestrauser:animate-css
raix:handlebar-helpers
underscore
zimme:iron-router-active

The meteor version was Meteor 1.1.0.2

nleush commented 9 years ago

I'm not sure but searching shows there are some problems with audit-argument-checks:

https://github.com/aldeed/meteor-autoform/issues/678 https://github.com/matteodem/meteor-easy-security/issues/12

Maybe it adds additional arguments to server method. Don't know how it works actually.

shadowzick commented 9 years ago

is audit-argument-checks a dependency of meteor? I tried removing it and now it works.

nleush commented 9 years ago

Its internal package http://docs.meteor.com/#/full/auditargumentchecks for common security checking.

I've tested meteor-oembed with audit-argument-checks - it works fine.

Maybe some other package add additional arguments. But audit-argument-checks catches it.

If you want to know what new arguments passed to that method you can add:

console.log(arguments);

to body of "iframely.oembed": function(url) {. Its in file packages/meteor-oembed/lib/server/oembed.js

shadowzick commented 9 years ago

okay let me check. Thanks @nleush :)

iparamonau commented 9 years ago

Shall we close this issue now?