[ 
https://issues.apache.org/jira/browse/THRIFT-2470?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13971858#comment-13971858
 ] 

ASF GitHub Bot commented on THRIFT-2470:
----------------------------------------

Github user Jens-G commented on the pull request:

    https://github.com/apache/thrift/pull/100#issuecomment-40643771
  
    Thanks for the patch. I'm not so sure if introducing a number of 
dependencies plus another new test project by a coup de main is such a great 
idea, especially if we compare all those changes to the net size of the patch 
itself. Did you run the tests on all relevant platforms or only with Visual 
Studio? I'm just asking, because that's important.


> THttpHandler swallows exceptions from processor
> -----------------------------------------------
>
>                 Key: THRIFT-2470
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2470
>             Project: Thrift
>          Issue Type: Bug
>          Components: C# - Library
>    Affects Versions: 0.9.1
>            Reporter: Adam Connelly
>            Assignee: Jens Geyer
>
> The THttpHandler.ProcessRequest() method currently swallows any exceptions by 
> the processor, and writes them out to Console.Error. This makes it very 
> difficult to figure out what's going on if an exception is thrown in anything 
> other than a console application.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to