camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Babak Vahdat (JIRA)" <>
Subject [jira] [Commented] (CAMEL-11655) Camel-Nagios: Use Encryption enum instead of EncryptionMethod
Date Fri, 18 Aug 2017 18:23:01 GMT


Babak Vahdat commented on CAMEL-11655:

[~ancosen] IMHO this's a breaking change for the upcoming {{2.20.0}} version as the {{encryptionMethod}}
query parameter has been simply removed and replaced with a new {{encryption}} query parameter,
which _would_ break user's code making use of this option. AFAIK a minor release should not
include any breaking changes, right? Shouldn't we better somehow mark {{encryptionMethod}}
as deprecated and encourage users to make use of the new {{encryption}} parameter instead?
Other than that some feedback on your made code changes:

What would be wrong to do:

Instead of the following if / else if:
if (encryption != null) {
  if (Encryption.NONE == encryption) {
  } else if (Encryption.XOR == encryption) {
  } else if (Encryption.TRIPLE_DES == encryption) {
  } else {
    throw new IllegalArgumentException("Unknown encryption method: " + encryption);

As because:
private Encryption encryption = Encryption.NONE;

Also maybe mark the {{NagiosEncryptionMethod}} enum itself as deprecated so we don't forget
to remove it in Camel 3.

> Camel-Nagios: Use Encryption enum instead of EncryptionMethod
> -------------------------------------------------------------
>                 Key: CAMEL-11655
>                 URL:
>             Project: Camel
>          Issue Type: Task
>          Components: camel-nagios
>            Reporter: Andrea Cosentino
>            Assignee: Andrea Cosentino
>             Fix For: 2.20.0

This message was sent by Atlassian JIRA

View raw message