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

Maximum call stack size exceeded #19

Closed
danielgindi opened this issue Mar 27, 2016 · 4 comments
Closed

Maximum call stack size exceeded #19

danielgindi opened this issue Mar 27, 2016 · 4 comments

Comments

@danielgindi
Copy link

findAll is recursive, and fails on big pages.

@davidcalhoun
Copy link

Looks like this is still a problem. I'm using cheerio to scrape a YouTube page (https://www.youtube.com/watch?v=6SFNW5F8K9Y). cheerio depends on css-select, which depends on domutils:

.../node_modules/domutils/lib/traversal.js:22
exports.getName = function(elem){
                          ^

RangeError: Maximum call stack size exceeded
    at Object.exports.getName (.../node_modules/domutils/lib/traversal.js:22:27)
    at tag (.../node_modules/css-select/lib/general.js:21:11)
    at findAll (.../node_modules/domutils/lib/querying.js:87:6)
    at findAll (.../node_modules/domutils/lib/querying.js:90:27)
    at findAll (.../node_modules/domutils/lib/querying.js:90:27)
    at findAll (.../node_modules/domutils/lib/querying.js:90:27)
    at findAll (.../node_modules/domutils/lib/querying.js:90:27)
    at findAll (.../node_modules/domutils/lib/querying.js:90:27)
    at findAll (.../node_modules/domutils/lib/querying.js:90:27)
    at findAll (.../node_modules/domutils/lib/querying.js:90:27)

@amit777
Copy link

amit777 commented Apr 30, 2017

Noticed the same error occasionally.

@markadrake
Copy link

Recently ran into this issue - I opened an issue with the wrong repo unfortunately.

lgraubner/sitemap-generator#11

Same issue with the getName in traversal.js line 22.

image

@fb55
Copy link
Owner

fb55 commented Jan 20, 2018

Fixed in #22. There is still some bug that prevents this from being used in css-select, but it's getting there.

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

5 participants