Skip to content

Upgrade smack dependency to 4.4.x#22867

Draft
apupier wants to merge 1 commit intoapache:mainfrom
apupier:smackNotUpdated
Draft

Upgrade smack dependency to 4.4.x#22867
apupier wants to merge 1 commit intoapache:mainfrom
apupier:smackNotUpdated

Conversation

@apupier
Copy link
Copy Markdown
Contributor

@apupier apupier commented Apr 30, 2026

Dependabot wasn't able to trigger a PR because one artifact was relocated.

it requires to adapt to few API changes

Description

Target

  • I checked that the commit is targeting the correct branch (Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.
  • I have run mvn clean install -DskipTests locally from root folder and I have committed all auto-generated changes.

Dependabot wasn't able to trigger a PR because one artifact was
relocated.

it requires to adapt to few API changes

Signed-off-by: Aurélien Pupier <apupier@ibm.com>
@github-actions
Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@apupier
Copy link
Copy Markdown
Contributor Author

apupier commented Apr 30, 2026

with main branch and the branch of this PR, the tests are both failing locallly for me

with Semeru:

Caused by: org.jivesoftware.smack.SmackException: javax.net.ssl.SSLHandshakeException: Remote host terminated the handshake
	at org.jivesoftware.smack.tcp.XMPPTCPConnection$PacketReader.parsePackets(XMPPTCPConnection.java:1176)
	at org.jivesoftware.smack.tcp.XMPPTCPConnection$PacketReader.access$1000(XMPPTCPConnection.java:1092)
	at org.jivesoftware.smack.tcp.XMPPTCPConnection$PacketReader$1.run(XMPPTCPConnection.java:1112)
	at java.base/java.lang.Thread.run(Thread.java:1595)

with Temurin:

Caused by: org.jivesoftware.smack.SmackException$SmackWrappedException: javax.net.ssl.SSLHandshakeException: (certificate_unknown) PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
	at org.jivesoftware.smack.AbstractXMPPConnection.setCurrentConnectionExceptionAndNotify(AbstractXMPPConnection.java:686)
	at org.jivesoftware.smack.AbstractXMPPConnection.notifyConnectionError(AbstractXMPPConnection.java:986)
	at org.jivesoftware.smack.tcp.XMPPTCPConnection.access$4100(XMPPTCPConnection.java:131)
	at org.jivesoftware.smack.tcp.XMPPTCPConnection$PacketReader.parsePackets(XMPPTCPConnection.java:1167)
	at org.jivesoftware.smack.tcp.XMPPTCPConnection$PacketReader.access$700(XMPPTCPConnection.java:916)
	at org.jivesoftware.smack.tcp.XMPPTCPConnection$PacketReader$1.run(XMPPTCPConnection.java:939)

@github-actions
Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • components/camel-xmpp
  • parent

POM dependency changes: targeted tests included

Changed properties: smack-version

Modules affected by dependency changes (1)
  • :camel-xmpp

⚠️ Some tests are disabled on GitHub Actions (@DisabledIfSystemProperty(named = "ci.env.name")) and require manual verification:

  • components/camel-xmpp: 6 test(s) disabled on GitHub Actions
All tested modules (8 modules)
  • Camel :: JBang :: MCP
  • Camel :: JBang :: Plugin :: Route Parser
  • Camel :: JBang :: Plugin :: TUI
  • Camel :: JBang :: Plugin :: Validate
  • Camel :: Launcher :: Container
  • Camel :: XMPP
  • Camel :: YAML DSL :: Validator
  • Camel :: YAML DSL :: Validator Maven Plugin

⚙️ View full build and test results

@apupier
Copy link
Copy Markdown
Contributor Author

apupier commented Apr 30, 2026

tests are skipped on GitHub actions https://github.com/apache/camel/actions/runs/25161416455/artifacts/6728737027 :

[camel-xmpp] [INFO] Running org.apache.camel.component.xmpp.XmppDeferredConnectionIT
[camel-xmpp] [WARNING] Tests run: 1, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0.011 s -- in org.apache.camel.component.xmpp.XmppDeferredConnectionIT
[camel-xmpp] [INFO] Running org.apache.camel.component.xmpp.integration.XmppProducerConcurrentIT
[camel-xmpp] [WARNING] Tests run: 2, Failures: 0, Errors: 0, Skipped: 2, Time elapsed: 0.004 s -- in org.apache.camel.component.xmpp.integration.XmppProducerConcurrentIT
[camel-xmpp] [INFO] Running org.apache.camel.component.xmpp.integration.XmppMultiUserChatIT
[camel-xmpp] [WARNING] Tests run: 1, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0 s -- in org.apache.camel.component.xmpp.integration.XmppMultiUserChatIT
[camel-xmpp] [INFO] Running org.apache.camel.component.xmpp.integration.XmppRouteChatIT
[camel-xmpp] [WARNING] Tests run: 1, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0.002 s -- in org.apache.camel.component.xmpp.integration.XmppRouteChatIT
[camel-xmpp] [INFO] Running org.apache.camel.component.xmpp.integration.XmppRouteMultipleProducersSingleConsumerIT
[camel-xmpp] [WARNING] Tests run: 1, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0.001 s -- in org.apache.camel.component.xmpp.integration.XmppRouteMultipleProducersSingleConsumerIT
[camel-xmpp] [INFO] Running org.apache.camel.component.xmpp.integration.XmppRouteIT
[camel-xmpp] [WARNING] Tests run: 1, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0 s -- in org.apache.camel.component.xmpp.integration.XmppRouteIT
[camel-xmpp] [INFO] 
[camel-xmpp] [INFO] Results:
[camel-xmpp] [INFO] 
[camel-xmpp] [INFO] Tests run: 7, Failures: 0, Errors: 0, Skipped: 7

@apupier
Copy link
Copy Markdown
Contributor Author

apupier commented Apr 30, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants