Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nom cli fails to fetch meta #2

Closed
hemanth opened this issue Nov 21, 2013 · 9 comments
Closed

nom cli fails to fetch meta #2

hemanth opened this issue Nov 21, 2013 · 9 comments

Comments

@hemanth
Copy link

hemanth commented Nov 21, 2013

nom url 'meta[http-equiv="refresh"]' does not fetch anything even though the url has few many meta tags, but nom url 'head' works fine.

@buschtoens
Copy link

Does nom url meta work? I could imagine that the dash in http-equiv causes the bug.

@buschtoens
Copy link

nom url meta does not work aswell. But this is probably an issue of cheerio.

@hemanth
Copy link
Author

hemanth commented Dec 8, 2013

@silvinci does not seem like a cheerio issue, but more of a CLI issue @matthewmueller help!

@buschtoens
Copy link

Can't see a problem in the cli source. Have you tried $.html("meta");?

@hemanth
Copy link
Author

hemanth commented Dec 8, 2013

@silvinci That works in the script, but how do you do the same on the CLI?

nom url head works but nom url head > meta nor nom url meta works...

@buschtoens
Copy link

This is interesting since the selector gets directly passed.

So basically, if it doesn't work in the cli, it should not work in the script. Just can't test it right now.

@hemanth
Copy link
Author

hemanth commented Dec 8, 2013

@silvinci You are right. There seems to be no return values from html() or text() for meta.

Something as simple as :

if(selector === "meta"){
   console.log($(selector));
}

Is doing the trick for me, but this is just a silly hack, need to paw at cheerio and check why html() is failing for this.

@buschtoens
Copy link

So, it's a cheerio issue. Cross-link it here and close? :)

@hemanth
Copy link
Author

hemanth commented Dec 8, 2013

@silvinci Cya there ;)

@hemanth hemanth closed this as completed Dec 8, 2013
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

No branches or pull requests

2 participants