weizhouapache commented on pull request #4531:
URL: https://github.com/apache/cloudstack/pull/4531#issuecomment-743175540


   > @ustcweizhou I think for the new novnc console, opening new window/session 
should close the previous on.
   > @davidjumani can you review ?
   
   @rhtyd 
   there are 3 options
   (1) reject new session, and keep old session.
   (2) accept new session, and kick out old session
   (3) keep both sessions
   
   The current behavior (reject the first new session and accept the second new 
session) seems to be bug.
   
   we have (3) in our production but the implementation is different from 
master.
   we think (1) is better than (2).
   
   @davidjumani @DaanHoogland @PaulAngus @andrijapanicsb your opinion ?
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to