I only have time to play with it. So this review is not based on code reading. I was able to trigger a session timeout and the banner appears again as expected. So from that point of view, as long as the patches don't break existing banner-ignorant clients, ack.
And again, please make sure it works with IE.
On 02/23/2017 01:47 PM, Endi Sukma Dewata wrote:
On 2/22/2017 1:57 PM, Endi Sukma Dewata wrote: