Skip to content

Fixes #8592. Add camel-quarkus-jsoup extension with native support #8626

Open
jomin7 wants to merge 1 commit intoapache:camel-mainfrom
jomin7:8592-camel-quarkus-jsoup
Open

Fixes #8592. Add camel-quarkus-jsoup extension with native support #8626
jomin7 wants to merge 1 commit intoapache:camel-mainfrom
jomin7:8592-camel-quarkus-jsoup

Conversation

@jomin7
Copy link
Copy Markdown
Contributor

@jomin7 jomin7 commented May 6, 2026

Fixes #8592. Add camel-quarkus-jsoup extension with native support

Introduces Quarkus extension for Apache Camel jsoup component enabling
HTML parsing, sanitization, and text extraction with GraalVM native support.

  • Minimal processor implementation (feature registration only)
  • Comprehensive tests: 10 test cases covering JVM, native, and edge cases
  • Simple language functions: htmlClean(), htmlDecode(), htmlParse()
  • YAML DSL support with route definitions
  • No reflection configuration required (auto-discovered by core)
  • Added to BOM, catalog, and test categories
  • Generated documentation with extension metadata

The extension provides XSS-safe HTML sanitization, text extraction,
and DOM manipulation capabilities for Camel routes.

@jamesnetherton
Copy link
Copy Markdown
Contributor

@jomin7 you'll need to rebase. I had to force push camel-main to get it back into a buildable state.

@jomin7
Copy link
Copy Markdown
Contributor Author

jomin7 commented May 7, 2026

Thanks for the heads up @jamesnetherton I'll rebase my branch onto the updated camel-main and force-push the changes shortly.

@jomin7 jomin7 force-pushed the 8592-camel-quarkus-jsoup branch from eb8bb2e to 96ce915 Compare May 7, 2026 09:06
@jomin7
Copy link
Copy Markdown
Contributor Author

jomin7 commented May 7, 2026

I noticed the CI build failed on the latest push. I'm looking into the logs now and will push a fix shortly.

class JsoupTest {

@Test
public void testHtmlClean() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpick - remove public modifier from test methods.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the feedback. I'll go ahead and remove the public modifiers from the test methods.

@jomin7
Copy link
Copy Markdown
Contributor Author

jomin7 commented May 7, 2026

It appears an issue on the camel main branch, likely a recent change, is causing our current build failure. The integration-tests/grok module is failing to compile due to a missing io.krakens.grok.api.exception.GrokException class.

@apupier
Copy link
Copy Markdown
Contributor

apupier commented May 7, 2026

It appears an issue on the camel main branch, likely a recent change, is causing our current build failure. The integration-tests/grok module is failing to compile due to a missing io.krakens.grok.api.exception.GrokException class.

there was a library change for grok, it should be "only" a matter of changing the package name from io.krakens.grok to io.whatap.grok , maybe adapt some tests apache/camel#23031

@jamesnetherton
Copy link
Copy Markdown
Contributor

@jomin7 I pushed some fixes. Please rebase again.

@jomin7 jomin7 force-pushed the 8592-camel-quarkus-jsoup branch from c9ced77 to 835da1a Compare May 7, 2026 15:24
@jomin7
Copy link
Copy Markdown
Contributor Author

jomin7 commented May 7, 2026

@jomin7 I pushed some fixes. Please rebase again.

@jamesnetherton I have rebased the changes on the updated camel-main branch.

- Added jsoup extension with deployment and runtime modules
- Added integration tests for JVM and native modes
- Updated BOMs and documentation
- Fixed alphabetical ordering in generated BOM files
@jomin7 jomin7 force-pushed the 8592-camel-quarkus-jsoup branch from 835da1a to 5364001 Compare May 7, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants