Fixup cc4c84e Upgraded snmp4j to another provider to avoid flaky tests

apache · May 5, 2023 · 0454f1f · 0454f1f
1 parent 438709d
commit 0454f1f
Show file tree

Hide file tree

Showing 8 changed files with 432 additions and 354 deletions.
diff --git a/docs/modules/ROOT/pages/reference/extensions/snmp.adoc b/docs/modules/ROOT/pages/reference/extensions/snmp.adoc
@@ -39,3 +39,17 @@ Please refer to the above link for usage and configuration details.
 ifeval::[{doc-show-user-guide-link} == true]
 Check the xref:user-guide/index.adoc[User guide] for more information about writing Camel Quarkus applications.
 endif::[]
+
+[id="extensions-snmp-camel-quarkus-limitations"]
+== Camel Quarkus limitations
+
+This extension uses `org.snmp4j:snmp4j` as an SNMP protocol implementation.
+This is different from Camel 3.18.x using `org.apache.servicemix.bundles.snmp4j:org.apache.servicemix.bundles`.
+
+The motivation for this change is twofold:
+
+1. `org.snmp4j:snmp4j` brings more stability and fixes many vulnerabilities.
+2. Camel switches to `org.snmp4j.snmp4j` in version 4 anyway, so Camel Quarkus users can enjoy the same benefits a bit earlier.
+
+This change has no impact on configuration of the SNMP component.
+
diff --git a/...yment/src/main/java/org/apache/camel/quarkus/component/snmp/deployment/SnmpProcessor.java b/...yment/src/main/java/org/apache/camel/quarkus/component/snmp/deployment/SnmpProcessor.java
@@ -22,7 +22,7 @@
 import io.quarkus.deployment.builditem.FeatureBuildItem;
 import io.quarkus.deployment.pkg.steps.NativeBuild;
 import org.apache.camel.component.snmp.SnmpComponent;
-import org.apache.camel.quarkus.component.snm.graal.SnmpRecorder;
+import org.apache.camel.quarkus.component.snmp.SnmpRecorder;
 import org.apache.camel.quarkus.core.JvmOnlyRecorder;
 import org.apache.camel.quarkus.core.deployment.spi.CamelBeanBuildItem;
 import org.jboss.logging.Logger;

diff --git a/extensions-jvm/snmp/runtime/src/main/doc/limitations.adoc b/extensions-jvm/snmp/runtime/src/main/doc/limitations.adoc
@@ -0,0 +1,9 @@
+This extension uses `org.snmp4j:snmp4j` as an SNMP protocol implementation.
+This is different from Camel 3.18.x using `org.apache.servicemix.bundles.snmp4j:org.apache.servicemix.bundles`.
+
+The motivation for this change is twofold:
+
+1. `org.snmp4j:snmp4j` brings more stability and fixes many vulnerabilities.
+2. Camel switches to `org.snmp4j.snmp4j` in version 4 anyway, so Camel Quarkus users can enjoy the same benefits a bit earlier.
+
+This change has no impact on configuration of the SNMP component.
diff --git a/...snmp/runtime/src/main/java/org/apache/camel/quarkus/component/snm/graal/SnmpRecorder.java b/...snmp/runtime/src/main/java/org/apache/camel/quarkus/component/snm/graal/SnmpRecorder.java