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

Huge Number of Spurious Requests Generated #699

Closed
zoffixznet opened this issue Jul 14, 2016 · 5 comments
Closed

Huge Number of Spurious Requests Generated #699

zoffixznet opened this issue Jul 14, 2016 · 5 comments

Comments

@zoffixznet
Copy link
Contributor

There seems to be something that's generating a huge number of reqests when page loads. I get 231 requests generated when visiting https://docs.perl6.org/type/ComplexStr

This is very problematic for users who pay for their bandwidth or users on slow connections, such as users on mobile and wifi, or users in countries with less speedy Internet.

It looks like it's sending a HEAD request for every link on the page? During the day, I browse from a monitored corprorate network. I don't know the details of the data they gather, but I really wouldn't want them to think I'm vising millions of pages per day, instead of working.

This is also a needless load on our volunteer-donated server.

Here's a run on a regular 2G network simulation. 231 requests, half a meg transfer, and it takes half a minute to load:

untitled

@vendethiel
Copy link
Member

vendethiel commented Jul 14, 2016

It seems that the JavaScript website tries to detect broken links for every single requests on the docs:

https://github.com/perl6/doc/blob/master/html/js/main.js#L127

I guess setup_debug_mode shouldn't get called for every request?

@coke
Copy link
Collaborator

coke commented Jul 14, 2016

This needs to be removed. We should not be running this check in everyone's browser on every page load.

@AlexDaniel
Copy link
Member

Related to #697

@gfldex
Copy link
Contributor

gfldex commented Jul 14, 2016

Cache is in place. It's checking once per commit per page. If the commit changes, the cache is wiped.

@gfldex
Copy link
Contributor

gfldex commented Jul 15, 2016

fixed with dcab8da

@gfldex gfldex closed this as completed Jul 15, 2016
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