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

Claus Ibsen commented on CAMEL-4549:
------------------------------------

Thanks for working on such an improvement.

If you are familiar with running maven, then you may want to ensure the source 
code follow our standard layout, which you can check using a maven goal named 
checkstyle. You can read about building the source with checkstyle here
http://camel.apache.org/building.html

Also you have a static initializer to add BouncyCastle as a provider. I wonder 
how well that works with OSGi and IBM JDKs etc. Also static initializers tend 
to have problems in trouble shooting as any exceptions thrown may not be logged 
etc.

You can implement org.apache.camel.Service in the DataFormat, and in the start 
method, add the provider. Then have a static field with a state so you know you 
only add it once. And maybe check that its not already added etc. As some other 
library may have added it etc.
                
> Add support for PGP to camel-crypto
> -----------------------------------
>
>                 Key: CAMEL-4549
>                 URL: https://issues.apache.org/jira/browse/CAMEL-4549
>             Project: Camel
>          Issue Type: Improvement
>          Components:  camel-crypto
>            Reporter: Adam
>             Fix For: 2.9.0
>
>         Attachments: camel-crypt-add-pgp.patch, pubring.gpg, secring.gpg
>
>
> This is an enhancement which add upon the camel-crypto component.  This patch 
> has the following changes:
> - Add PGPDataFormat
> - Introduce the use of bouncycastle, it was previously only a "test" resource 
> Notes:
>  - The camel-crypto "component" did not exist in Jira, could you please add 
> it.
>  - Can you please update the bouncycastle-version from 1.40 to 1.46 in the 
> parent pom.  I temporarily hard coded it into the pom of this project.
>  - The diff doesn't appear to allow binaries, I have a sample pgp keyring in 
> my project, I will upload them as separate files
> One this patch is approved, please let me know and I will go and update the 
> wiki docmentation.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to