Comment on attachment 9220966 [details] [diff] [review]
The else can be removed too. Now a simple if/else and The "return STATE_IS_BROKEN;" is unconditional here.
dd("Exception getting certificate for connection: " + ex.message);
Should at least the "return STATE_IS_BROKEN;" moved to be after the catch block?
Function has no return value at the end otherwise and looks a little "unclean".