Skip to content

Conversation

knoobie
Copy link
Contributor

@knoobie knoobie commented Sep 17, 2025

Description

Might be worth to double check what you are really testing at the moment. There is also a 8.1 available.

Type of change

  • Bugfix
  • Feature

Checklist

  • I have read the contribution guide: https://vaadin.com/docs/latest/guide/contributing/overview/
  • I have added a description following the guideline.
  • The issue is created in the corresponding repository and I have referenced it.
  • I have added tests to ensure my change is effective and works as intended.
  • New and existing tests are passing locally with my change.
  • I have performed self-review and corrected misspellings.

Additional for Feature type of change

  • Enhancement / new feature was discussed in a corresponding GitHub issue and Acceptance Criteria were created.

@CLAassistant
Copy link

CLAassistant commented Sep 17, 2025

CLA assistant check
All committers have signed the CLA.

@ZheSun88
Copy link
Contributor

hi @knoobie , thanks for the heads up.

After testing the EAP 8.0, we actually have found some issue with this version, which was cause by the version conflicts on jackson dependencies. we will provide an workaround for this support..

in EAP 8.1, there is no issue with Vaadin 24.9.0 application.

I will modify the PR later with some more info.

@ZheSun88 ZheSun88 enabled auto-merge (squash) September 18, 2025 08:35
- Open Liberty 23
- RedHat JBoss EAP 8.0 beta
- RedHat JBoss EAP 8.1
- To work with RedHat JBoss EAP 8.0, you will need to add the following content to the `jboss-deployment-structure.xml` placed under `WEB-INF` folder
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe worth mentioning that the workaround is needed only for development mode. Production builds do not need this.

Copy link
Contributor

Choose a reason for hiding this comment

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

updated the release note..
main branch is for V25, i think we need revisit all these servlet for testing.

@ZheSun88 ZheSun88 merged commit af32a05 into vaadin:main Sep 18, 2025
3 checks passed
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.

4 participants