Closed LucasBrazi06 closed 4 years ago
Issue opened since 2 weeks! Please have a look!
Preliminary work:
The CORS issue is not related, it's another issue.
Tested dev locally and it works now. To be tested in QA but need someone who charges his car.
I do not have changed anything except the socket.io upgrade. And the code to refresh charts seems to do the right thing.
I do not have changed anything except the socket.io upgrade. And the code to refresh charts seems to do the right thing.
Ok, let's assume it was a false positive then :-)
We still have the issue in prod (tested today)
Check all the screens (eg. when I activate auto refresh in the logs it's not working either)
@fcoutellier Can you please do some testing?
I do not have changed anything and during all my last week tests through mitmproxy, it's been working fine.
@jerome-benoit Serge reported me that during a test with Seb 2 opened web browser were opened and only one of them was connected to socket IO and refreshing the charging curve automatically.
@jerome-benoit https://github.com/jerome-benoit Serge reported me that during a test with Seb 2 opened web browser were opened and only one of them was connected to socket IO and refreshing the charging curve automatically.
Two opened windows or tabs of the same browser? Because it’s not the same in the processus parenting.
And two windows, one for two different browser, are working fine?
-- Jérôme Benoit aka fraggle Piment Noir - https://piment-noir.org OpenPGP Key ID : 27B535D3 Key fingerprint : B799 BBF6 8EC8 911B B8D7 CDBC C3B1 92C6 27B5 35D3
@jerome-benoit https://github.com/jerome-benoit Serge reported me that during a test with Seb 2 opened web browser were opened and only one of them was connected to socket IO and refreshing the charging curve automatically. Two opened windows or tabs of the same browser? Because it’s not the same in the processus parenting. And two windows, one for two different browser, are working fine?
Two differents Chrome browsers
We have the issue reported by Frank @SAP reception desk, also not working in the giant iPad. You could have a look at it this afternoon.
god-prio ;-)
The bug reproducibility is still not evident even given all the usage patterns listed in that issue, so goes its resolution. On the Giant iPad, it seems 100% reproducible. So appointment will be taken with the IT in order to work on it with a mouse and a keyboard.
I think I've narrowed down that issue with the debug logs:
The socket.io initialisation is only done after a login phase because of the need to authorise it with the Jwt token. But when you already have a Jwt token on your browser, the login function is not called again and the socket.io initialisation is not done. I will try to hook it in the dashboard init if there's already a Jwt token referenced in the browser.
I think I've narrowed down that issue with the debug logs:
The socket.io initialisation is only done after a login phase because of the need to authorise it with the Jwt token. But when you already have a Jwt token on your browser, the login function is not called again and the socket.io initialisation is not done. I will try to hook it in the dashboard init if there's already a Jwt token.
Excellent ! Let's see if it'll work out after this fix.
The socket.io initialisation is only done after a login phase because of the need to authorise it with the Jwt token. But when you already have a Jwt token on your browser, the login function is not called again and the socket.io initialisation is not done. I will try to hook it in the dashboard init if there's already a Jwt token.
Excellent ! Let's see if it'll work out after this fix.
So far so good: always one socket.io connection per browser tab to refresh each tab, socket.io reconnect successfully on a per browser tab basis whatever is the number of browsers and tabs opened.
I still think that at some point to avoid to overload the rest services because of socket.io, the socket.io server will need to be split out but that's not before an important usage of the dashboard.
So far so good: always one socket.io connection per browser tab to refresh each tab, socket.io reconnect successfully on a per browser tab basis whatever is the number of browsers and tabs opened.
I still think that at some point to avoid to overload the rest services because of socket.io, the socket.io server will need to be split out but that not before an important usage of the dashboard.
Ok thank you. Yes we'll split it when we'll have million users :-)
The issue is back on production SCP landscape only. It works everywhere else.
And now, it works correctly again on the production landscape.
Next time, maybe try to restart the REST application to see if it fix it or not.
And now, it works correctly again on the production landscape.
Next time, maybe try to restart the REST application to see if it fix it or not.
Weird anyway !
I close this issue and will reopen a new one later one if necessary
Socket IO is active in the prod. Check what would be wrong with it. Thanks!