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.

Christina


On 02/23/2017 01:47 PM, Endi Sukma Dewata wrote:
On 2/22/2017 1:57 PM, Endi Sukma Dewata wrote:
The PKI UI main page has been modified to retrieve access banner
and display it once at the beginning of the SSL connection.

https://fedorahosted.org/pki/ticket/2582

New patch #957-1 attached. It's now using the two REST services added in patch #956-1.



_______________________________________________
Pki-devel mailing list
Pki-devel@redhat.com
https://www.redhat.com/mailman/listinfo/pki-devel