Bug in ZmCsfeAsynchCommand

I simply had to add the following 2 lines in order to make the function correctly behave:
this._fireInvokeEvent(newEx); // at line 161
return;

After some debugging, I noticed I was getting an Exception in that block but it was then not passed to the listener. I noticed that the Exception was thrown but that it was actually overriden in the following block by another exception, since you use a single variable newEx in your code:

In other words, if you get 2 Exceptions, the first one will never get to the application level. FYI, I was getting an Exception because the server was not sending back XML, but the content-type of the response was set to XML. So, if you check the above block, if there was an error parsing the response, variable this._responseSoapDoc will not be defined, and that is were the second exception was occuring.

Anyway, why not stop the execution of this method at the first exception ?

I wil take a look at this problem. It is true that this class is not used in mail client. It is used only in the Admin client, which currently is mostly using SOAP. Most likely this class will go away once we finish switching the Admin client to JSON.