Add max scroll limit in screenshot function to prevent timeout for infinite scroll page

[Aman-Codes]

Description

Currently the screenshot function will scroll the page by 200 units until it reaches the end of the page and then it takes a screenshot of that page. But for webpages with infinite scroll (like Google images page), it would give API timeout error. Hence a reasonable limit should be added on the number of times page is scrolled and also user should be able to specify that as a parameter in the post request. If no parameter is specified then a default value could be used.

Relevant Files

• https://github.com/webevaluator/nodebackend/blob/master/controllers/helper.js
• https://github.com/webevaluator/nodebackend/blob/master/routes/screenshot.js

[Atish03]

Can I work on it ?
And what would be an appropriate distance to scroll in such cases ?

[Aman-Codes]

Sure @Atish03 go ahead.
Maybe scrolling atmax 10 times would be good for a default value.