[jira] [Commented] (CAMEL-12672) Renovate Zipkin (brave) support

2018-07-20 Thread Adrian Cole (JIRA)


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

Adrian Cole commented on CAMEL-12672:
-

to answer possibly a different question: why not just use OpenTracing? Well 
that can be used, with brave-opentracing, though the experience will be less 
efficient and impossible to take advantage of abstractions created by the 
zipkin community. For example, the runtime efficiency is worse due to factors 
including layers of wrapping and likely a future need to do extra context 
tracking unless some bugs are solved in opentracing's api design. Next, 
consistency is difficult as [data and sampling policy 
tools|https://github.com/openzipkin/brave/tree/master/instrumentation/http] 
tools, tools with stable apis, can't be used. OpenTracing data has poor data 
quality evidenced by complaints about things like nested json attributes. Even 
if you don't believe there are any problems, it would be intentionally not 
moving back into supporting what is by far the most popular java tracing 
library.

 

If you choose for whatever reason to keep the zipkin integration as-is, and 
continue to not help it move forward, I would recommend removing it entirely. 
This way, the openzipkin community can create their own in another repo that is 
more friendly towards them. A rotting library is worse than no library, 
especially a rotting tracing library.

> Renovate Zipkin (brave) support
> ---
>
> Key: CAMEL-12672
> URL: https://issues.apache.org/jira/browse/CAMEL-12672
> Project: Camel
>  Issue Type: Improvement
>  Components: camel-zipkin
>Reporter: Adrian Cole
>Priority: Major
>
> With some hard effort, we've managed to maintain the original form of the 
> camel-zipkin integration. However, this integration's design is considerably 
> out of date, especially considering the work done in camel-opentracing. 
> camel-zipkin should be completely rewritten in order to improve experience 
> (and stop volunteers from having support questions in zipkin channel).
>  
> I've been told folks don't know what to do, yet I think it is somewhat 
> simple. The work that was done in camel-opentracing.. do that in 
> camel-zipkin. Beyond that is polishing.
> For example, the brave v4+ api is similar enough to opentracing to mostly 
> find/replace. This is a good start and can fix some considerable modeling 
> bugs in camel-zipkin around messaging. Once that is complete, we can move to 
> being more idiomatic and taking advantage of brave's features.
> For example, `brave.Tracing` is injected by many things in spring. People use 
> it to configure advanced propagation things not possible in the normal api. 
> Similarly, `brave.http.HttpTracing` sets and verifies http-scoped 
> configuration so that when downstream systems are sensitive, data is 
> collected in ways not proprietary to camel. One such example is X-Ray. 
> Another example is that usually the data in opentracing is a lot bigger and 
> so more expensive for sites. Using tools like this helps keep costs down.
> While the above steps are "going beyond" again, a relatively straight-forward 
> port of camel-opentracing would be a great way to re-enable the very large 
> OpenZipkin community and leverage the work you've done.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (CAMEL-12672) Renovate Zipkin (brave) support

2018-07-20 Thread Adrian Cole (JIRA)
Adrian Cole created CAMEL-12672:
---

 Summary: Renovate Zipkin (brave) support
 Key: CAMEL-12672
 URL: https://issues.apache.org/jira/browse/CAMEL-12672
 Project: Camel
  Issue Type: Improvement
  Components: camel-zipkin
Reporter: Adrian Cole


With some hard effort, we've managed to maintain the original form of the 
camel-zipkin integration. However, this integration's design is considerably 
out of date, especially considering the work done in camel-opentracing. 
camel-zipkin should be completely rewritten in order to improve experience (and 
stop volunteers from having support questions in zipkin channel).

 

I've been told folks don't know what to do, yet I think it is somewhat simple. 
The work that was done in camel-opentracing.. do that in camel-zipkin. Beyond 
that is polishing.

For example, the brave v4+ api is similar enough to opentracing to mostly 
find/replace. This is a good start and can fix some considerable modeling bugs 
in camel-zipkin around messaging. Once that is complete, we can move to being 
more idiomatic and taking advantage of brave's features.

For example, `brave.Tracing` is injected by many things in spring. People use 
it to configure advanced propagation things not possible in the normal api. 
Similarly, `brave.http.HttpTracing` sets and verifies http-scoped configuration 
so that when downstream systems are sensitive, data is collected in ways not 
proprietary to camel. One such example is X-Ray. Another example is that 
usually the data in opentracing is a lot bigger and so more expensive for 
sites. Using tools like this helps keep costs down.

While the above steps are "going beyond" again, a relatively straight-forward 
port of camel-opentracing would be a great way to re-enable the very large 
OpenZipkin community and leverage the work you've done.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12195) camel redeliveries does not work with hystrix

2018-07-20 Thread Mark Egan (JIRA)


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

Mark Egan commented on CAMEL-12195:
---

[~davsclaus]: I share [~rahmnate] and [~rodrigoramalho]'s confusion and concern 
regarding the capabilities of retries in with a circuit breaker enabled route.  
From my understanding, retry and circuit breaker are two approaches that go 
well together.  A good description of this is found on [Rahul Rajat Singh's 
DZone article "Understanding Retry Pattern With Exponential Back-Off and 
Circuit Breaker Pattern" 
|https://dzone.com/articles/understanding-retry-pattern-with-exponential-back].

If we're truly in a position where these are incompatible approaches, perhaps a 
request for that feature within Hystrix is in order.  Perhaps retries can be 
incorporated into the Circuit Breaker implementation such that you can 
configure retries similar to and along side of the circuit breaker parameters.

> camel redeliveries does not work with hystrix
> -
>
> Key: CAMEL-12195
> URL: https://issues.apache.org/jira/browse/CAMEL-12195
> Project: Camel
>  Issue Type: Bug
>  Components: camel-hystrix
>Reporter: Scott Feldstein
>Assignee: Claus Ibsen
>Priority: Major
> Fix For: 2.21.0
>
>
> Camel hystrix does not work with errorHander and redeliveries.
> I discussed this here -> 
> [https://stackoverflow.com/questions/48396320/camel-exception-redelivery-with-hystrix]
> clone this repo to reproduce the issue
> {code:java}
> $ git clone g...@github.com:scottmf/camel-hystrix-bug.git
> $ cd camel-hystrix-bug
> $ gradlew clean test{code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12642) No support for http4 feature authenticationPreemptive in pollEnrich

2018-07-20 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on CAMEL-12642:


onderson commented on a change in pull request #2432: CAMEL-12642:Fix for http4 
feature authenticationPreemptive in pollEnrich
URL: https://github.com/apache/camel/pull/2432#discussion_r204117717
 
 

 ##
 File path: 
components/camel-http4/src/main/java/org/apache/camel/component/http4/HttpPollingConsumer.java
 ##
 @@ -33,21 +33,33 @@
 import org.apache.http.client.methods.HttpGet;
 import org.apache.http.client.methods.HttpRequestBase;
 import org.apache.http.client.protocol.HttpClientContext;
+import org.apache.http.impl.auth.BasicScheme;
+import org.apache.http.protocol.HttpContext;
 import org.apache.http.util.EntityUtils;
 
 /**
  * A polling HTTP consumer which by default performs a GET
  *
  * @version 
  */
+@SuppressWarnings("deprecation")
 
 Review comment:
   I would not prefer supressing deprecation. Especially while we have java's 6 
month release cycle.. 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> No support for http4 feature authenticationPreemptive in pollEnrich
> ---
>
> Key: CAMEL-12642
> URL: https://issues.apache.org/jira/browse/CAMEL-12642
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-http4
>Affects Versions: 2.23.0
>Reporter: Frank Olschewski
>Assignee: Ramu
>Priority: Major
>
> authenticationPreemptive=true does not work with pollEnrich.
> {code:java}
> .pollEnrich("https4:localhost?authenticationPreemptive=true" +
>"&authUsername=user&authPassword=passwd"){code}
> In Class /org/apache/camel/component/http4/HttpPollingConsumer.java:79 you 
> see the following code:
> {code:java}
> HttpResponse response = httpClient.execute(method, httpClientContext);{code}
> The setup for authenticationPreemptive is missing.
> You can see what to do in 
> org.apache.camel.component.http4.HttpProducer#executeMethod:
> {code:java}
> pprotected HttpResponse executeMethod(HttpUriRequest httpRequest) throws 
> IOException {
> HttpContext localContext = new BasicHttpContext();
> if (getEndpoint().isAuthenticationPreemptive()) {
> BasicScheme basicAuth = new BasicScheme();
> localContext.setAttribute("preemptive-auth", basicAuth);
> }
> if (httpContext != null) {
> localContext = new BasicHttpContext(httpContext);
> }
> return httpClient.execute(httpRequest, localContext);
> }{code}
>  In org.apache.camel.component.http4.HttpPollingConsumer#doReceive 
> getEndpoint().isAuthenticationPreemptive() returns the correct value, so 
> there is nothing against implementing this feature here as well.
> The problem behind the scene ist that the context attribute "preemptive-auth" 
> is missing.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12654) RabbitMQ Headers - Headers with null value are skipped.

2018-07-20 Thread Dmitry Volodin (JIRA)


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

Dmitry Volodin commented on CAMEL-12654:


[~Srivastav], thanks for explanation. Looks like this is useful feature, but it 
must be added as an option (allowNullHeaders or something else) to provide 
backward compatibility.

> RabbitMQ Headers - Headers with null value are skipped.
> ---
>
> Key: CAMEL-12654
> URL: https://issues.apache.org/jira/browse/CAMEL-12654
> Project: Camel
>  Issue Type: Bug
>  Components: camel-rabbitmq
>Affects Versions: 2.21.1
>Reporter: Prakhar
>Priority: Minor
>  Labels: headers, rabbitmq
>
> Reference: Conversation with Claus Ibsen on 
> [Stackoverflow|https://stackoverflow.com/questions/50583749/apache-camel-how-to-setheader-value-as-null]
> org.apache.camel.component.rabbitmq.RabbitMQMessageConverter.buildProperties()
>  skips the headers which have value null. The actual java client for RabbitMQ 
> does not have this check. It accepts a basic Map which can 
> accept null value. 
> [Reference|https://rabbitmq.github.io/rabbitmq-java-client/api/current/com/rabbitmq/client/AMQP.BasicProperties.Builder.html]
>  . Check the definition of headers() method
> *The business scenario where we use it*. On the RabbitMQ, we accept messages 
> from multiple sources. Depending on whether a specific header is null or not, 
> we route these message downstream in our processing pipeline.
> If required, I could provide a working example of rabbitmq java-client to 
> demonstrate that the headers with null value are not skipped



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12654) RabbitMQ Headers - Headers with null value are skipped.

2018-07-20 Thread Prakhar (JIRA)


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

Prakhar commented on CAMEL-12654:
-

Consider this scenario.

System A produces a message (For example Customer submitting his electric meter 
reading) to rabbitmq exchange. This message is passed in sequence through 3 
systems
--> System 1 (performs a check if submitted through the Online interface)
 --> Adds header (Online : /null)
--> System 2 (more checks...)
 --> Adds header (PreviousBalance : /null)
--> System 3 (more chesks)
 --> Adds header (ClearByInvoiceService : /null)
...
...
...
--> System B
    --> Does final processing

System B processes this message. If either of the headers is set, it does a 
microservice lookup to get more details to enrich the base message. This works 
fine until we started reading the message from another system C. System C can 
directly send this message to System B, but it needs to setup headers with a 
null value.

So far we have been using a set of POJO based rabbit-mq consumers and producers 
to read messages and do ETL operations. Now we have started porting many of our 
integrations to apache camel, but we are not able to use camel-rabbitmq 
component, because headers with null values are skipped. It would be good to 
have camel-rabbitmq not to filter out the header with null values like the 
original rabbitmq java-client does.

Another factor which adds to the complexity is, there are multiple systems 
similar to System A, owned by different IT teams (and different programming 
languages). Making changes to System B would mean breaking message structure 
for rest of the parties which is additional cost and effort.

We can actually fork the camel-rabbitmq component and fix the changes. But that 
would mean, we have to maintain this fork for all the versions that we would 
use.

> RabbitMQ Headers - Headers with null value are skipped.
> ---
>
> Key: CAMEL-12654
> URL: https://issues.apache.org/jira/browse/CAMEL-12654
> Project: Camel
>  Issue Type: Bug
>  Components: camel-rabbitmq
>Affects Versions: 2.21.1
>Reporter: Prakhar
>Priority: Minor
>  Labels: headers, rabbitmq
>
> Reference: Conversation with Claus Ibsen on 
> [Stackoverflow|https://stackoverflow.com/questions/50583749/apache-camel-how-to-setheader-value-as-null]
> org.apache.camel.component.rabbitmq.RabbitMQMessageConverter.buildProperties()
>  skips the headers which have value null. The actual java client for RabbitMQ 
> does not have this check. It accepts a basic Map which can 
> accept null value. 
> [Reference|https://rabbitmq.github.io/rabbitmq-java-client/api/current/com/rabbitmq/client/AMQP.BasicProperties.Builder.html]
>  . Check the definition of headers() method
> *The business scenario where we use it*. On the RabbitMQ, we accept messages 
> from multiple sources. Depending on whether a specific header is null or not, 
> we route these message downstream in our processing pipeline.
> If required, I could provide a working example of rabbitmq java-client to 
> demonstrate that the headers with null value are not skipped



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Resolved] (CAMEL-12669) Camel-Nats: add new client options available with client 2.x

2018-07-20 Thread Andrea Cosentino (JIRA)


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

Andrea Cosentino resolved CAMEL-12669.
--
Resolution: Fixed

> Camel-Nats: add new client options available with client 2.x
> 
>
> Key: CAMEL-12669
> URL: https://issues.apache.org/jira/browse/CAMEL-12669
> Project: Camel
>  Issue Type: Improvement
>  Components: camel-nats
>Reporter: Andrea Cosentino
>Assignee: Andrea Cosentino
>Priority: Major
> Fix For: 2.23.0
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12671) camel-iec60870 - Add documentation for missing field

2018-07-20 Thread Jens Reimann (JIRA)


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

Jens Reimann commented on CAMEL-12671:
--

Yes, I can take care of that. However I can only come back to that issue in 
about two weeks. I hope you don't mind.

> camel-iec60870 - Add documentation for missing field
> 
>
> Key: CAMEL-12671
> URL: https://issues.apache.org/jira/browse/CAMEL-12671
> Project: Camel
>  Issue Type: Task
>  Components: documentation
>Reporter: Claus Ibsen
>Assignee: Jens Reimann
>Priority: Major
> Fix For: 2.23.0
>
>
> This field did not have documentation. So I added something
> https://github.com/apache/camel/blob/48539d7156a5d4795f0dc593c6334427fc8450e1/components/camel-iec60870/src/main/java/org/apache/camel/component/iec60870/client/ClientOptions.java#L48
> Jens do you mind taking a look, and add the correct documentation, and also 
> set the correct default value.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread Luca Burgazzoli (JIRA)


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

Luca Burgazzoli updated CAMEL-12649:

Priority: Minor  (was: Major)

> Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API
> ---
>
> Key: CAMEL-12649
> URL: https://issues.apache.org/jira/browse/CAMEL-12649
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-braintree
>Reporter: Russ Johnson
>Assignee: Luca Burgazzoli
>Priority: Minor
> Fix For: 2.23.0
>
>
> With the release of version 2.83.1 of the Braintree Java SDK 
> ([https://github.com/braintree/braintree_java/releases/tag/2.83.1|https://github.com/braintree/braintree_java/releases/tag/2.83.0])
>  the transaction fees, the Transaction Level Fee Report has become available.
> The intention of this ticket is to upgrade the Braintree Java SDK version and 
> expose the Braintree Transaction Level Fee Report in the Component.
> Transaction Level Fees Reports are describe here: 
> [https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
>  but the API is not yet documented



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Assigned] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread Luca Burgazzoli (JIRA)


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

Luca Burgazzoli reassigned CAMEL-12649:
---

Assignee: Luca Burgazzoli

> Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API
> ---
>
> Key: CAMEL-12649
> URL: https://issues.apache.org/jira/browse/CAMEL-12649
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-braintree
>Reporter: Russ Johnson
>Assignee: Luca Burgazzoli
>Priority: Major
> Fix For: 2.23.0
>
>
> With the release of version 2.83.1 of the Braintree Java SDK 
> ([https://github.com/braintree/braintree_java/releases/tag/2.83.1|https://github.com/braintree/braintree_java/releases/tag/2.83.0])
>  the transaction fees, the Transaction Level Fee Report has become available.
> The intention of this ticket is to upgrade the Braintree Java SDK version and 
> expose the Braintree Transaction Level Fee Report in the Component.
> Transaction Level Fees Reports are describe here: 
> [https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
>  but the API is not yet documented



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread Luca Burgazzoli (JIRA)


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

Luca Burgazzoli updated CAMEL-12649:

Fix Version/s: (was: 2.22.0)
   2.23.0

> Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API
> ---
>
> Key: CAMEL-12649
> URL: https://issues.apache.org/jira/browse/CAMEL-12649
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-braintree
>Reporter: Russ Johnson
>Priority: Major
> Fix For: 2.23.0
>
>
> With the release of version 2.83.1 of the Braintree Java SDK 
> ([https://github.com/braintree/braintree_java/releases/tag/2.83.1|https://github.com/braintree/braintree_java/releases/tag/2.83.0])
>  the transaction fees, the Transaction Level Fee Report has become available.
> The intention of this ticket is to upgrade the Braintree Java SDK version and 
> expose the Braintree Transaction Level Fee Report in the Component.
> Transaction Level Fees Reports are describe here: 
> [https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
>  but the API is not yet documented



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Resolved] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread Luca Burgazzoli (JIRA)


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

Luca Burgazzoli resolved CAMEL-12649.
-
Resolution: Fixed

> Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API
> ---
>
> Key: CAMEL-12649
> URL: https://issues.apache.org/jira/browse/CAMEL-12649
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-braintree
>Reporter: Russ Johnson
>Assignee: Luca Burgazzoli
>Priority: Minor
> Fix For: 2.23.0
>
>
> With the release of version 2.83.1 of the Braintree Java SDK 
> ([https://github.com/braintree/braintree_java/releases/tag/2.83.1|https://github.com/braintree/braintree_java/releases/tag/2.83.0])
>  the transaction fees, the Transaction Level Fee Report has become available.
> The intention of this ticket is to upgrade the Braintree Java SDK version and 
> expose the Braintree Transaction Level Fee Report in the Component.
> Transaction Level Fees Reports are describe here: 
> [https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
>  but the API is not yet documented



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread Luca Burgazzoli (JIRA)


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

Luca Burgazzoli commented on CAMEL-12649:
-

thank you

> Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API
> ---
>
> Key: CAMEL-12649
> URL: https://issues.apache.org/jira/browse/CAMEL-12649
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-braintree
>Reporter: Russ Johnson
>Assignee: Luca Burgazzoli
>Priority: Minor
> Fix For: 2.23.0
>
>
> With the release of version 2.83.1 of the Braintree Java SDK 
> ([https://github.com/braintree/braintree_java/releases/tag/2.83.1|https://github.com/braintree/braintree_java/releases/tag/2.83.0])
>  the transaction fees, the Transaction Level Fee Report has become available.
> The intention of this ticket is to upgrade the Braintree Java SDK version and 
> expose the Braintree Transaction Level Fee Report in the Component.
> Transaction Level Fees Reports are describe here: 
> [https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
>  but the API is not yet documented



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on CAMEL-12649:


Github user lburgazzoli closed the pull request at:

https://github.com/apache/camel/pull/2433


> Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API
> ---
>
> Key: CAMEL-12649
> URL: https://issues.apache.org/jira/browse/CAMEL-12649
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-braintree
>Reporter: Russ Johnson
>Priority: Major
> Fix For: 2.22.0
>
>
> With the release of version 2.83.1 of the Braintree Java SDK 
> ([https://github.com/braintree/braintree_java/releases/tag/2.83.1|https://github.com/braintree/braintree_java/releases/tag/2.83.0])
>  the transaction fees, the Transaction Level Fee Report has become available.
> The intention of this ticket is to upgrade the Braintree Java SDK version and 
> expose the Braintree Transaction Level Fee Report in the Component.
> Transaction Level Fees Reports are describe here: 
> [https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
>  but the API is not yet documented



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on CAMEL-12649:


lburgazzoli closed pull request #2433: CAMEL-12649: camel-braintree - Upgraded 
Braintree Java SDK to 2.83.1 and exposed transaction level fees API
URL: https://github.com/apache/camel/pull/2433
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/components/camel-braintree/pom.xml 
b/components/camel-braintree/pom.xml
index 7ba0c32e674..737a743baf7 100644
--- a/components/camel-braintree/pom.xml
+++ b/components/camel-braintree/pom.xml
@@ -225,6 +225,11 @@
   com.braintreegateway.PlanGateway
   
 
+
+  report
+  com.braintreegateway.ReportGateway
+  
+
 
   settlementBatchSummary
   
com.braintreegateway.SettlementBatchSummaryGateway
diff --git a/components/camel-braintree/src/main/docs/braintree-component.adoc 
b/components/camel-braintree/src/main/docs/braintree-component.adoc
index 202274a7552..686053991bc 100644
--- a/components/camel-braintree/src/main/docs/braintree-component.adoc
+++ b/components/camel-braintree/src/main/docs/braintree-component.adoc
@@ -121,6 +121,7 @@ Endpoint prefix can be one of:
 * paymentmethod
 * paymentmethodNonce
 * plan
+* report
 * settlementBatchSummary
 * subscription
 * transaction
@@ -568,7 +569,32 @@ braintree://plan/endpoint
 |all |  |  |List
 |
 
- 
+
+ Endpoint prefix _report_
+
+The following endpoints can be invoked with the prefix *report* as
+follows:
+
+[source,java]
+-
+braintree://plan/report?[options]
+-
+
+[cols="<,<,<,<",options="header",]
+|==
+|Endpoint |Shorthand Alias |Options |Result Body Type
+|transactionLevelFees |  | request 
|com.braintreegateway.Result
+|==
+
+[[Braintree-URIOptionsfortreport]]
+URI Options for _report_
+
+[cols="<,<",options="header",]
+|==
+|Name |Type
+|request |com.braintreegateway.TransactionLevelFeeReportRequest
+|==
+
 
  Endpoint prefix _settlementBatchSummary_
 
diff --git 
a/components/camel-braintree/src/test/java/org/apache/camel/component/braintree/ReportGatewayIntegrationTest.java
 
b/components/camel-braintree/src/test/java/org/apache/camel/component/braintree/ReportGatewayIntegrationTest.java
new file mode 100644
index 000..ace38b42678
--- /dev/null
+++ 
b/components/camel-braintree/src/test/java/org/apache/camel/component/braintree/ReportGatewayIntegrationTest.java
@@ -0,0 +1,79 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.component.braintree;
+
+import java.text.SimpleDateFormat;
+import java.util.Calendar;
+import java.util.List;
+
+import com.braintreegateway.TransactionLevelFeeReport;
+import com.braintreegateway.TransactionLevelFeeReportRequest;
+import com.braintreegateway.TransactionLevelFeeReportRow;
+
+import org.apache.camel.builder.RouteBuilder;
+import org.apache.camel.component.braintree.internal.BraintreeApiCollection;
+import org.apache.camel.component.braintree.internal.ReportGatewayApiMethod;
+import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+/**
+ * Test class for {@link com.braintreegateway.ReportGateway} APIs.
+ */
+public class ReportGatewayIntegrationTest extends AbstractBraintreeTestSupport 
{
+
+private static final Logger LOG = 
LoggerFact

[jira] [Commented] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on CAMEL-12649:


nofuss opened a new pull request #2433: CAMEL-12649: camel-braintree - Upgraded 
Braintree Java SDK to 2.83.1 and exposed transaction level fees API
URL: https://github.com/apache/camel/pull/2433
 
 
   Upgraded Braintree Java SDK to 2.83.1 and exposed transaction level fees API


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API
> ---
>
> Key: CAMEL-12649
> URL: https://issues.apache.org/jira/browse/CAMEL-12649
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-braintree
>Reporter: Russ Johnson
>Priority: Major
> Fix For: 2.22.0
>
>
> With the release of version 2.83.1 of the Braintree Java SDK 
> ([https://github.com/braintree/braintree_java/releases/tag/2.83.1|https://github.com/braintree/braintree_java/releases/tag/2.83.0])
>  the transaction fees, the Transaction Level Fee Report has become available.
> The intention of this ticket is to upgrade the Braintree Java SDK version and 
> expose the Braintree Transaction Level Fee Report in the Component.
> Transaction Level Fees Reports are describe here: 
> [https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
>  but the API is not yet documented



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on CAMEL-12649:


GitHub user nofuss opened a pull request:

https://github.com/apache/camel/pull/2433

CAMEL-12649: camel-braintree - Upgraded Braintree Java SDK to 2.83.1 and 
exposed transaction level fees API

Upgraded Braintree Java SDK to 2.83.1 and exposed transaction level fees API

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/nofuss/camel CAMEL-12649

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/camel/pull/2433.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2433


commit 62803d6f1e91c47c13a540ff422d1b78cdfa9be4
Author: Russ Johnson 
Date:   2018-07-20T07:45:43Z

Upgraded Braintree Java SDK to 2.83.1 and exposed transaction level fees API




> Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API
> ---
>
> Key: CAMEL-12649
> URL: https://issues.apache.org/jira/browse/CAMEL-12649
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-braintree
>Reporter: Russ Johnson
>Priority: Major
> Fix For: 2.22.0
>
>
> With the release of version 2.83.1 of the Braintree Java SDK 
> ([https://github.com/braintree/braintree_java/releases/tag/2.83.1|https://github.com/braintree/braintree_java/releases/tag/2.83.0])
>  the transaction fees, the Transaction Level Fee Report has become available.
> The intention of this ticket is to upgrade the Braintree Java SDK version and 
> expose the Braintree Transaction Level Fee Report in the Component.
> Transaction Level Fees Reports are describe here: 
> [https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
>  but the API is not yet documented



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (CAMEL-12671) camel-iec60870 - Add documentation for missing field

2018-07-20 Thread Claus Ibsen (JIRA)
Claus Ibsen created CAMEL-12671:
---

 Summary: camel-iec60870 - Add documentation for missing field
 Key: CAMEL-12671
 URL: https://issues.apache.org/jira/browse/CAMEL-12671
 Project: Camel
  Issue Type: Task
  Components: documentation
Reporter: Claus Ibsen
Assignee: Jens Reimann
 Fix For: 2.23.0


This field did not have documentation. So I added something
https://github.com/apache/camel/blob/48539d7156a5d4795f0dc593c6334427fc8450e1/components/camel-iec60870/src/main/java/org/apache/camel/component/iec60870/client/ClientOptions.java#L48

Jens do you mind taking a look, and add the correct documentation, and also set 
the correct default value.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-11707) spring boot code generation: there should be a way to exclude some properties from the code generation

2018-07-20 Thread Claus Ibsen (JIRA)


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

Claus Ibsen commented on CAMEL-11707:
-

At first cut I added support for adding @XmlTransient annotation to the field 
then its ignored (for components) as this was needed in some situations for the 
spring-boot documentation.

> spring boot code generation: there should be a way to exclude some properties 
> from the code generation
> --
>
> Key: CAMEL-11707
> URL: https://issues.apache.org/jira/browse/CAMEL-11707
> Project: Camel
>  Issue Type: Improvement
>  Components: tooling
>Reporter: Luca Burgazzoli
>Priority: Major
> Fix For: 2.23.0
>
>
> As today every property is exposed to spring boot auto configuration 
> processor but sometimes you may want to hide some or them as they may be 
> supposed to be used only internally or because you want to write your own 
> code to manage complex configurations.
> As this may also be needed to hide info from the catalog, we may have 
> something like:
> {code}
> @Metadata(key = "ignore", value = "true", labels = "spring-boot,catalog")
> {code}
> Or have specific annotations.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread Russ Johnson (JIRA)


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

Russ Johnson commented on CAMEL-12649:
--

Updated to Braintree Java SDK version 2.83.1 . Version 2.83.0 was released 
without the javadoc jar resource, which is required for building in Camel. 
version 2.83.1 has all required resources. 

> Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API
> ---
>
> Key: CAMEL-12649
> URL: https://issues.apache.org/jira/browse/CAMEL-12649
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-braintree
>Reporter: Russ Johnson
>Priority: Major
> Fix For: 2.22.0
>
>
> With the release of version 2.83.1 of the Braintree Java SDK 
> ([https://github.com/braintree/braintree_java/releases/tag/2.83.1|https://github.com/braintree/braintree_java/releases/tag/2.83.0])
>  the transaction fees, the Transaction Level Fee Report has become available.
> The intention of this ticket is to upgrade the Braintree Java SDK version and 
> expose the Braintree Transaction Level Fee Report in the Component.
> Transaction Level Fees Reports are describe here: 
> [https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
>  but the API is not yet documented



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread Russ Johnson (JIRA)


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

Russ Johnson updated CAMEL-12649:
-
Description: 
With the release of version 2.83.1 of the Braintree Java SDK 
([https://github.com/braintree/braintree_java/releases/tag/2.83.1|https://github.com/braintree/braintree_java/releases/tag/2.83.0])
 the transaction fees, the Transaction Level Fee Report has become available.

The intention of this ticket is to upgrade the Braintree Java SDK version and 
expose the Braintree Transaction Level Fee Report in the Component.

Transaction Level Fees Reports are describe here: 
[https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
 but the API is not yet documented

  was:
With the release of version 2.83.0 of the Braintree Java SDK 
([https://github.com/braintree/braintree_java/releases/tag/2.83.0]) the 
transaction fees, the Transaction Level Fee Report has become available.

The intention of this ticket is to upgrade the Braintree Java SDK version and 
expose the Braintree Transaction Level Fee Report in the Component.

Transaction Level Fees Reports are describe here: 
[https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
 but the API is not yet documented


> Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API
> ---
>
> Key: CAMEL-12649
> URL: https://issues.apache.org/jira/browse/CAMEL-12649
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-braintree
>Reporter: Russ Johnson
>Priority: Major
> Fix For: 2.22.0
>
>
> With the release of version 2.83.1 of the Braintree Java SDK 
> ([https://github.com/braintree/braintree_java/releases/tag/2.83.1|https://github.com/braintree/braintree_java/releases/tag/2.83.0])
>  the transaction fees, the Transaction Level Fee Report has become available.
> The intention of this ticket is to upgrade the Braintree Java SDK version and 
> expose the Braintree Transaction Level Fee Report in the Component.
> Transaction Level Fees Reports are describe here: 
> [https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
>  but the API is not yet documented



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (CAMEL-12649) Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API

2018-07-20 Thread Russ Johnson (JIRA)


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

Russ Johnson updated CAMEL-12649:
-
Summary: Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API  
(was: Upgrade Braintree SDK to 2.83.0 and Expose Transaction Fees API)

> Upgrade Braintree SDK to 2.83.1 and Expose Transaction Fees API
> ---
>
> Key: CAMEL-12649
> URL: https://issues.apache.org/jira/browse/CAMEL-12649
> Project: Camel
>  Issue Type: New Feature
>  Components: camel-braintree
>Reporter: Russ Johnson
>Priority: Major
> Fix For: 2.22.0
>
>
> With the release of version 2.83.0 of the Braintree Java SDK 
> ([https://github.com/braintree/braintree_java/releases/tag/2.83.0]) the 
> transaction fees, the Transaction Level Fee Report has become available.
> The intention of this ticket is to upgrade the Braintree Java SDK version and 
> expose the Braintree Transaction Level Fee Report in the Component.
> Transaction Level Fees Reports are describe here: 
> [https://articles.braintreepayments.com/control-panel/reporting/transaction-level-fee-report]
>  but the API is not yet documented



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)