Skip to content

Conversation

@mshabarov
Copy link
Contributor

Related-to #21410

@github-actions
Copy link

github-actions bot commented Jul 3, 2025

Test Results

1 237 files  ±  0  1 237 suites  ±0   1h 15m 45s ⏱️ + 3m 55s
8 512 tests ±  0  8 452 ✅ +  1  60 💤 ±0  0 ❌  - 1 
8 919 runs  +104  8 850 ✅ +105  69 💤 ±0  0 ❌  - 1 

Results for commit ab5e012. ± Comparison against base commit cc1d64f.

♻️ This comment has been updated with latest results.

@mshabarov mshabarov changed the title feat!: Deprecate VaadinWebSecurity feat: Deprecate VaadinWebSecurity Jul 4, 2025
@mcollovati mcollovati requested a review from tepi July 21, 2025 11:35
@mcollovati mcollovati removed the request for review from tepi July 29, 2025 04:37
mcollovati
mcollovati previously approved these changes Jul 30, 2025
mcollovati
mcollovati previously approved these changes Aug 6, 2025
@vaadin-bot vaadin-bot added +1.0.0 and removed +0.1.0 labels Aug 6, 2025
@mshabarov mshabarov marked this pull request as ready for review August 6, 2025 13:12
@mshabarov mshabarov enabled auto-merge (squash) August 6, 2025 13:16
@vaadin-bot vaadin-bot added +0.1.0 and removed +1.0.0 labels Aug 6, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 6, 2025

@mshabarov mshabarov merged commit eeacfe5 into main Aug 6, 2025
26 of 27 checks passed
@mshabarov mshabarov deleted the deprecate-web-security branch August 6, 2025 13:28
vaadin-bot pushed a commit that referenced this pull request Aug 6, 2025
Related-to #21410

---------

Co-authored-by: Marco Collovati <[email protected]>
mcollovati added a commit that referenced this pull request Aug 6, 2025
Related-to #21410

---------

Co-authored-by: Mikhail Shabarov <[email protected]>
Co-authored-by: Marco Collovati <[email protected]>
@Legioth Legioth added this to Roadmap Aug 26, 2025
@Legioth Legioth moved this to September 2025 (24.9) in Roadmap Aug 26, 2025
@Legioth Legioth added flow hilla Issues related to Hilla labels Aug 26, 2025
Comment on lines +103 to +105
* @deprecated Use {@link VaadinSecurityConfigurer} instead. It follows the
* Spring's SecurityConfigurer pattern and we recommend use it to
* configure Spring Security with Vaadin:
Copy link
Contributor

@vursen vursen Sep 5, 2025

Choose a reason for hiding this comment

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

The deprecation notice should state the version in which this class will be removed. Otherwise, it's not immediately clear.

Copilot AI pushed a commit that referenced this pull request Oct 1, 2025
Related-to #21410

---------

Co-authored-by: Marco Collovati <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: September 2025 (24.9) - Released

Development

Successfully merging this pull request may close these issues.

7 participants