Skip to content

feat: Implement production mode license verification in VaadinServlet#23864

Merged
mcollovati merged 11 commits into23.6from
v23-license-check
Mar 18, 2026
Merged

feat: Implement production mode license verification in VaadinServlet#23864
mcollovati merged 11 commits into23.6from
v23-license-check

Conversation

@TatuLund
Copy link
Contributor

Added license verification for production and development modes.

@github-actions
Copy link

github-actions bot commented Mar 13, 2026

Test Results

  984 files  ±0    984 suites  ±0   43m 6s ⏱️ + 1m 59s
6 335 tests +1  6 285 ✅ +1  50 💤 ±0  0 ❌ ±0 
6 651 runs  +4  6 593 ✅ +4  58 💤 ±0  0 ❌ ±0 

Results for commit 3fffa70. ± Comparison against base commit eb6d0a9.

♻️ This comment has been updated with latest results.

@mcollovati
Copy link
Collaborator

There's already a license check-in VaadinServlet. I think we should have validation in a single place.

Removed license validation logic from VaadinServlet.
@TatuLund
Copy link
Contributor Author

There's already a license check-in VaadinServlet. I think we should have validation in a single place.

Fixed.

Removed license verification logic from VaadinServletService.
Add license verification in production mode for VaadinServlet.
@mcollovati mcollovati enabled auto-merge (squash) March 18, 2026 09:41
@mcollovati mcollovati changed the title feat: Implement license verification in VaadinServletService feat: Implement production mode license verification in VaadinServlet Mar 18, 2026
@sonarqubecloud
Copy link

@mcollovati mcollovati merged commit 9cf3b9d into 23.6 Mar 18, 2026
34 of 46 checks passed
@mcollovati mcollovati deleted the v23-license-check branch March 18, 2026 10:06
vaadin-bot pushed a commit that referenced this pull request Mar 18, 2026
…#23864)

Add runtime license verification for production mode.
License check failure is only logged, it does not prevent the application from running.
vaadin-bot added a commit that referenced this pull request Mar 18, 2026
…#23864) (#23929)

Add runtime license verification for production mode.
License check failure is only logged, it does not prevent the application from running.

Co-authored-by: Tatu Lund <tatu@vaadin.com>
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.

4 participants