Skip to content
New issue

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

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

Already on GitHub? # to your account

slow "time to first byte" on logged in state #682

Open
wants to merge 2 commits into
base: ssr
Choose a base branch
from
Open

slow "time to first byte" on logged in state #682

wants to merge 2 commits into from

Conversation

AirBorne04
Copy link

When the user is logged in the browser measure for time to first byte is very slow (1.5s). I am testing with a remote mongodb which might contribute to the issue. In that situation also responses for js-files, assets and css all suffer the same initial delay. My debugging brought me to the conclusion that the handleOnAllRoutes has some side effect that is causing this. I do not see a reason why handleOnAllRoutes would need to run on every request and especially for static files / assets?
Therefore i wrapped the call to handleOnAllRoutes into the picker route when we are anticipating an html rendering to occur.

AirBorne04 added 2 commits August 17, 2016 16:12
When the user is logged in the browser measure for time to first byte is very slow (1.5s). I am testing with a remote mongodb which might contribute to the issue. In that situation also responses for js-files, assets and css all suffer the same initial delay. My debugging brought me to the conclusion that the handleOnAllRoutes has some side effect that is causing this. I do not see a reason why handleOnAllRoutes would need to run on every request and especially for static files / assets?
Therefore i wrapped the call to handleOnAllRoutes into the picker route when we are anticipating an html rendering to occur.
travis build failed due to trailing whitespace
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant