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

Send heardbeat from ui to back #4417

Closed
freddidierRTE opened this issue Apr 14, 2023 · 0 comments · Fixed by #4487
Closed

Send heardbeat from ui to back #4417

freddidierRTE opened this issue Apr 14, 2023 · 0 comments · Fixed by #4487
Assignees
Labels
Enhancement New feature

Comments

@freddidierRTE
Copy link
Contributor

In certain circumstances , the network is cut between the browser and the cards-consultation services but the connection does not close on the back , this situation has been observe in production for a duration of 1 hour . So the user is considered connected where as it is not the case

Solution :
Send every X minutes an HTTP heartbeat request from the UI to the back . The endpoint shall cards/CardSubscriptionHeartBeat

If the back did not receive the heartbeat for Y minutes , it closes the connection

By default X = 60s
and Y = 150 s

Implementation :

  • Add a last connection date in services/cards-consultation/src/main/java/org/opfab/cards/consultation/services/CardSubscription.java
  • Update it when receiving hearbeat
  • Add a new Thread in services/cards-consultation/src/main/java/org/opfab/cards/consultation/services/CardSubscriptionService.java
@freddidierRTE freddidierRTE added the Enhancement New feature label Apr 14, 2023
@freddidierRTE freddidierRTE moved this to 🏗 In progress in OperatorFabric Release 3.15.0 May 9, 2023
ClementBouvierN added a commit that referenced this issue May 11, 2023
ClementBouvierN added a commit that referenced this issue May 11, 2023
ClementBouvierN added a commit that referenced this issue May 11, 2023
ClementBouvierN added a commit that referenced this issue May 11, 2023
@ClementBouvierN ClementBouvierN linked a pull request May 11, 2023 that will close this issue
@ClementBouvierN ClementBouvierN moved this from 🏗 In progress to 👀 In review in OperatorFabric Release 3.15.0 May 11, 2023
ClementBouvierN added a commit that referenced this issue May 23, 2023
ClementBouvierN added a commit that referenced this issue May 24, 2023
freddidierRTE added a commit that referenced this issue May 24, 2023
freddidierRTE added a commit that referenced this issue May 24, 2023
freddidierRTE pushed a commit that referenced this issue May 25, 2023
freddidierRTE added a commit that referenced this issue May 25, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in OperatorFabric Release 3.15.0 May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants