[jira] Created: (CAMEL-1093) camel-jetty - Exchange failures should not be returned as 200

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (CAMEL-1093) camel-jetty - Exchange failures should not be returned as 200

JIRA jira@apache.org
camel-jetty - Exchange failures should not be returned as 200
-------------------------------------------------------------

                 Key: CAMEL-1093
                 URL: https://issues.apache.org/activemq/browse/CAMEL-1093
             Project: Apache Camel
          Issue Type: Bug
          Components: camel-jetty
    Affects Versions: 1.5.0
            Reporter: Claus Ibsen
             Fix For: 1.5.1, 2.0.0


The code below:

{code}
    protected RouteBuilder createRouteBuilder() throws Exception {
        return new RouteBuilder() {
            public void configure() throws Exception {
                errorHandler(noErrorHandler());
                from("jetty:http://localhost:8080/myapp/myservice").process(new MyBookService());
            }
        };
    }

    public class MyBookService implements Processor {
        public void process(Exchange exchange) throws Exception {
            throw new IllegalArgumentException("Damm");
        }
    }
{code}

Will return http response code 200. We should let end users easily set what response code they want and in case of route failures we should probably return 500 and return the stracktrace in the body

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (CAMEL-1093) camel-jetty - Exchange failures should not be returned as 200

JIRA jira@apache.org

    [ https://issues.apache.org/activemq/browse/CAMEL-1093?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=47408#action_47408 ]

Claus Ibsen commented on CAMEL-1093:
------------------------------------

HttpBinding should test for failures

> camel-jetty - Exchange failures should not be returned as 200
> -------------------------------------------------------------
>
>                 Key: CAMEL-1093
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-1093
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-jetty
>    Affects Versions: 1.5.0
>            Reporter: Claus Ibsen
>             Fix For: 1.5.1, 2.0.0
>
>
> The code below:
> {code}
>     protected RouteBuilder createRouteBuilder() throws Exception {
>         return new RouteBuilder() {
>             public void configure() throws Exception {
>                 errorHandler(noErrorHandler());
>                 from("jetty:http://localhost:8080/myapp/myservice").process(new MyBookService());
>             }
>         };
>     }
>     public class MyBookService implements Processor {
>         public void process(Exchange exchange) throws Exception {
>             throw new IllegalArgumentException("Damm");
>         }
>     }
> {code}
> Will return http response code 200. We should let end users easily set what response code they want and in case of route failures we should probably return 500 and return the stracktrace in the body

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Assigned: (CAMEL-1093) camel-jetty - Exchange failures should not be returned as 200

JIRA jira@apache.org
In reply to this post by JIRA jira@apache.org

     [ https://issues.apache.org/activemq/browse/CAMEL-1093?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Claus Ibsen reassigned CAMEL-1093:
----------------------------------

    Assignee: Claus Ibsen

> camel-jetty - Exchange failures should not be returned as 200
> -------------------------------------------------------------
>
>                 Key: CAMEL-1093
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-1093
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-jetty
>    Affects Versions: 1.5.0
>            Reporter: Claus Ibsen
>            Assignee: Claus Ibsen
>             Fix For: 1.5.1, 2.0.0
>
>
> The code below:
> {code}
>     protected RouteBuilder createRouteBuilder() throws Exception {
>         return new RouteBuilder() {
>             public void configure() throws Exception {
>                 errorHandler(noErrorHandler());
>                 from("jetty:http://localhost:8080/myapp/myservice").process(new MyBookService());
>             }
>         };
>     }
>     public class MyBookService implements Processor {
>         public void process(Exchange exchange) throws Exception {
>             throw new IllegalArgumentException("Damm");
>         }
>     }
> {code}
> Will return http response code 200. We should let end users easily set what response code they want and in case of route failures we should probably return 500 and return the stracktrace in the body

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Resolved: (CAMEL-1093) camel-jetty - Exchange failures should not be returned as 200

JIRA jira@apache.org
In reply to this post by JIRA jira@apache.org

     [ https://issues.apache.org/activemq/browse/CAMEL-1093?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Claus Ibsen resolved CAMEL-1093.
--------------------------------

    Resolution: Fixed

> camel-jetty - Exchange failures should not be returned as 200
> -------------------------------------------------------------
>
>                 Key: CAMEL-1093
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-1093
>             Project: Apache Camel
>          Issue Type: Bug
>          Components: camel-jetty
>    Affects Versions: 1.5.0
>            Reporter: Claus Ibsen
>            Assignee: Claus Ibsen
>             Fix For: 1.5.1, 2.0.0
>
>
> The code below:
> {code}
>     protected RouteBuilder createRouteBuilder() throws Exception {
>         return new RouteBuilder() {
>             public void configure() throws Exception {
>                 errorHandler(noErrorHandler());
>                 from("jetty:http://localhost:8080/myapp/myservice").process(new MyBookService());
>             }
>         };
>     }
>     public class MyBookService implements Processor {
>         public void process(Exchange exchange) throws Exception {
>             throw new IllegalArgumentException("Damm");
>         }
>     }
> {code}
> Will return http response code 200. We should let end users easily set what response code they want and in case of route failures we should probably return 500 and return the stracktrace in the body

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.