camel-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Claus Ibsen (Commented) (JIRA)" <>
Subject [jira] [Commented] (CAMEL-4549) Add support for PGP to camel-crypto
Date Mon, 17 Oct 2011 06:20:11 GMT


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

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:
>             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:!default.jspa
For more information on JIRA, see:


View raw message