Skip to content

fix: Regression on PHP 8.x for connection check#49

Merged
ralflang merged 1 commit into
FRAMEWORK_6_0from
chore/fix-ci-findings
Jul 21, 2026
Merged

fix: Regression on PHP 8.x for connection check#49
ralflang merged 1 commit into
FRAMEWORK_6_0from
chore/fix-ci-findings

Conversation

@ralflang

Copy link
Copy Markdown
Member

No description provided.

@what-the-diff

what-the-diff Bot commented Jul 21, 2026

Copy link
Copy Markdown

PR Summary

  • Enhanced Stability in 'Logout' Function
    The team has improved the logout process by adding safeguards to handle situations where the logout function is initiated before a user has fully connected, which had previously resulted in warnings on newer versions of the software.

  • Improved Testing for Connectivity Scenarios
    A new suite of tests named BaseLogoutNullConnectionTest has been implemented. These tests actively check that our logout process doesn't produce warnings in cases where a user is authenticated (logged in) yet not connected. This addition enhances our software's resilience against potential issues in user lifecycle management.

@github-actions

Copy link
Copy Markdown

🔍 CI Results

Overall: ❌ 6/10 lanes failed

TL;DR: ❌ Quality issues.

Summary by PHP Version

PHP dev stable
8.1
8.2
8.3
8.4
8.5

Quality Metrics

  • PHPUnit: did not run (10 lanes exited non-zero with no test output) ❌
  • PHPStan: No errors found ✅ — 28 advisories at level 6
  • PHP-CS-Fixer: 251 files checked, no issues ✅
❌ Failed Lanes

php8.1-dev

  • PHPUnit: 0 failures, 0 errors

php8.1-stable

  • PHPUnit: 0 failures, 0 errors

php8.2-dev

  • PHPUnit: 0 failures, 0 errors

php8.2-stable

  • PHPUnit: 0 failures, 0 errors

php8.3-dev

  • PHPUnit: 0 failures, 0 errors

php8.3-stable

  • PHPUnit: 0 failures, 0 errors

CI powered by horde-componentsView full results

@ralflang
ralflang merged commit b06e358 into FRAMEWORK_6_0 Jul 21, 2026
1 check failed
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.

1 participant