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

Portlet not responding after the first Heartbeat request #166

Closed
mehdi-vaadin opened this issue Dec 20, 2019 · 0 comments · Fixed by #167
Closed

Portlet not responding after the first Heartbeat request #166

mehdi-vaadin opened this issue Dec 20, 2019 · 0 comments · Fixed by #167
Assignees
Labels
bug Something isn't working
Milestone

Comments

@mehdi-vaadin
Copy link
Contributor

After the first Heartbeat request is sent (by default 5 minutes after the portlet is loaded), the portlet doesn't respond anymore. For example, if a button on the portlet is clicked the loading indicator is shown and stayed there forever like the following.

vaadin-portlet-loading

To reproduce it, click on a button in the portlet after 5 minutes (heartbeatInterval) has passed since the portlet is loaded.

The following errors are shown in the browser console log:

Received message with server id 6 but expected 5. Postponing handling until the missing message(s) have been received
Gave up waiting for message 5 from the server

One other noticeable thing that seems to be the cause of this bug is that the heartbeat request has a JSON response which can be seen in the Network section of Chrome Developer Tools. While normally heartbeat requests don't have any response. The JSON response is like the following.

for(;;);[{"syncId":3,"clientId":2,"timings":[1631,1168]}]

In fact, the heartbeat request is not handled by HeartbeatHandler. It's handled by either PortletBootstrapHandler or PortletUidlRequestHandler because their canHandleRequest method returns true for a heartbeat request.

Versions:

  • Vaadin Portlet 1.0.0.beta2
  • Flow 2.2.0.alpha8
  • Pluto 3.1.0
  • Java 8
  • Chrome 79
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant