Skip to content

Release 2.53#5878

Merged
kevinpapst merged 36 commits into
mainfrom
release-2.53
Apr 10, 2026
Merged

Release 2.53#5878
kevinpapst merged 36 commits into
mainfrom
release-2.53

Conversation

@kevinpapst

@kevinpapst kevinpapst commented Mar 19, 2026

Copy link
Copy Markdown
Member

Description

  • Removed remaining usages of toast messages
  • Bump GitHub action versions for Node 24
  • Fetch PHP extension requirements with composer file for Doctor screen
  • Better handling in case of deactivated opcache_status() function (e.g. shared hosting)
  • New configuration to cleanup RelayState if it exceeds 80 character, - fixes SAML2: Relay state is too long. #5752
  • Sanitize RelayState before redirect - special thanks to @morimori-dev 👍
  • Do not allow " in certain user attributes + escaping - special thanks to @wooseokdotkim 👍
  • Do not copy rates upon timesheet restart into the clone - special thanks to @wooseokdotkim 👍
  • API allowed to update disabled user preferences- special thanks to @udaypali 👍
  • Disable access to deprecated api password hashes via Twig SecurityPolicy - special thanks to @hetworkshard 👍

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I verified that my code applies to the guidelines (composer code-check)
  • I updated the documentation (see here)
  • I agree that this code is used in Kimai (see license)

@codecov

codecov Bot commented Mar 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 75.94937% with 38 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.79%. Comparing base (cad9f58) to head (400ecc8).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/Voter/ActivityVoter.php 47.05% 9 Missing ⚠️
src/Twig/ContractExtensions.php 42.85% 8 Missing ⚠️
src/Voter/ProjectVoter.php 57.14% 6 Missing ⚠️
src/Controller/Auth/SamlController.php 0.00% 4 Missing ⚠️
src/API/TimesheetController.php 83.33% 3 Missing ⚠️
src/Controller/DoctorController.php 80.00% 3 Missing ⚠️
src/Configuration/SamlConfiguration.php 33.33% 2 Missing ⚠️
src/API/ActivityController.php 83.33% 1 Missing ⚠️
src/API/ProjectController.php 83.33% 1 Missing ⚠️
src/Form/Type/WeekDaysType.php 75.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #5878      +/-   ##
============================================
- Coverage     87.84%   87.79%   -0.06%     
- Complexity     9730     9791      +61     
============================================
  Files           870      873       +3     
  Lines         31276    31374      +98     
============================================
+ Hits          27475    27544      +69     
- Misses         3801     3830      +29     
Files with missing lines Coverage Δ
src/API/UserController.php 90.00% <100.00%> (+0.52%) ⬆️
src/Configuration/SystemConfiguration.php 89.59% <ø> (ø)
src/DependencyInjection/AppExtension.php 73.78% <100.00%> (ø)
src/DependencyInjection/Configuration.php 97.91% <100.00%> (+<0.01%) ⬆️
src/Entity/Customer.php 100.00% <ø> (ø)
src/Entity/User.php 94.11% <ø> (ø)
...rc/Export/Package/CellFormatter/ArrayFormatter.php 100.00% <100.00%> (ø)
...xport/Spreadsheet/CellFormatter/ArrayFormatter.php 100.00% <100.00%> (ø)
src/Form/Extension/SelectWithApiDataExtension.php 86.04% <ø> (ø)
src/Form/Type/MailType.php 100.00% <100.00%> (ø)
... and 22 more

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kevinpapst kevinpapst marked this pull request as ready for review April 8, 2026 19:29
@kevinpapst kevinpapst merged commit 999d820 into main Apr 10, 2026
8 checks passed
@kevinpapst kevinpapst deleted the release-2.53 branch April 10, 2026 16:09
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.

SAML2: Relay state is too long.

2 participants