[Pki-devel] [PATCH] 422 Added login page for TPS UI.

[Endi Sukma Dewata]

On 3/11/2014 12:35 PM, Christina Fu wrote:
> * I asked if window.crypto.logout stuff works for IE as well (we are
> required to support IE, as I understand it)?
>   - I did a quick search and it seems like IE does not support it, but
> you can do the following:
>    document.execCommand('ClearAuthenticationCache');
>     If the research is going to take a long time, then feel free to file
> a separate ticket to take care of it later.  Otherwise, please make sure
> IE is supported.

I opened this ticket for IE support because we might need some browser 
detection mechanism and proper tests:
https://fedorahosted.org/pki/ticket/903

On 3/11/2014 8:27 PM, John Magne wrote:
> What cfu said applies.
>
> As for the window.crypto.logout, yes that won't work in IE. Might
> also want to check to see if window.cryto exists before calling it?

I added a code to check window.cryto.

> Also sounds like to audit log logout as cfu suggested would we not
> need some support on the back end? At this point it might be
> reasonable to pursue logging out the user from the server side.

Ideally the client would logout from the server, but sometimes the 
client would just close the connection without logging out. I added a 
possible way to detect such case in the following ticket:
https://fedorahosted.org/pki/ticket/902

> Othewise if tested to work this seems nice enough to ACK for now.

This patch has been tested on Firefox. Pushed to master. Thanks all.

-- 
Endi S. Dewata