[jira] [Updated] (ARTEMIS-4710) Keep pem-keystore dependencies as is; make it really optional

2024-04-02 Thread Robbie Gemmell (Jira)


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

Robbie Gemmell updated ARTEMIS-4710:

Description: 
Previously we excluded bcprov-jdk15on and bcpkix-jdk15on dependencies normally 
used by pem-keystore, and replaced them with bcprov-jdk18on and bcpkix-jdk18on 
as already managed and used elsewhere in the build, thus keeping things 
consistent on a single gav for the bc modules.

The latest 2.3.0 version of pem-keystore (used since ARTEMIS-4599) was actually 
changed to use bcprov-jdk18on and bcpkix-jdk18on in 
https://github.com/ctron/pem-keystore/commit/1263b207da0adf4a7d2dc61d6a9a19e7bba97731.
 As such we can now simplify the dependency handling by removing the prior 
exclusion+replacement handling for those deps.

Additionally, by avoiding a global import on PemKeyStoreProvider, and only 
referencing it upon use, those who wish would be able to exclude the dependency 
when the functionality is not used.

  was:
The latest version already uses bcprov-jdk18on.

Avoid global imports to be able to exclude dependency when not used.


> Keep pem-keystore dependencies as is; make it really optional
> -
>
> Key: ARTEMIS-4710
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4710
> Project: ActiveMQ Artemis
>  Issue Type: Improvement
>Affects Versions: 2.33.0
>Reporter: Alexey Markevich
>Priority: Minor
>
> Previously we excluded bcprov-jdk15on and bcpkix-jdk15on dependencies 
> normally used by pem-keystore, and replaced them with bcprov-jdk18on and 
> bcpkix-jdk18on as already managed and used elsewhere in the build, thus 
> keeping things consistent on a single gav for the bc modules.
> The latest 2.3.0 version of pem-keystore (used since ARTEMIS-4599) was 
> actually changed to use bcprov-jdk18on and bcpkix-jdk18on in 
> https://github.com/ctron/pem-keystore/commit/1263b207da0adf4a7d2dc61d6a9a19e7bba97731.
>  As such we can now simplify the dependency handling by removing the prior 
> exclusion+replacement handling for those deps.
> Additionally, by avoiding a global import on PemKeyStoreProvider, and only 
> referencing it upon use, those who wish would be able to exclude the 
> dependency when the functionality is not used.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (ARTEMIS-4710) Keep pem-keystore dependencies as is; make it really optional

2024-04-02 Thread Alexey Markevich (Jira)


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

Alexey Markevich updated ARTEMIS-4710:
--
Summary: Keep pem-keystore dependencies as is; make it really optional  
(was: Keep pem-keystore dependencies as is; make it actually optional)

> Keep pem-keystore dependencies as is; make it really optional
> -
>
> Key: ARTEMIS-4710
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4710
> Project: ActiveMQ Artemis
>  Issue Type: Improvement
>Affects Versions: 2.33.0
>Reporter: Alexey Markevich
>Priority: Minor
>
> The latest version already uses bcprov-jdk18on.
> Avoid global imports to be able to exclude dependency when not used.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)