fix: upgrade cbauth for thread-safe authentication startup - #70
Conversation
lucee@5 ColdBox ^8.0.0 Test Results123 tests 123 ✅ 8s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
lucee@6 ColdBox ^8.0.0 Test Results123 tests 123 ✅ 5s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
lucee@5 ColdBox ^7.0.0 Test Results123 tests 123 ✅ 7s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
lucee@6 ColdBox be Test Results123 tests 123 ✅ 4s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
boxlang-cfml@1 ColdBox ^7.0.0 Test Results123 tests 123 ✅ 7s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
boxlang-cfml@be ColdBox 8 Test Results123 tests 123 ✅ 7s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
boxlang-cfml@1 ColdBox ^8.0.0 Test Results123 tests 123 ✅ 7s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
boxlang-cfml@1 ColdBox be Test Results123 tests 123 ✅ 7s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
boxlang-cfml@1 ColdBox 8 Test Results123 tests 123 ✅ 7s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
lucee@6 ColdBox ^7.0.0 Test Results123 tests 123 ✅ 5s ⏱️ Results for commit 393c0c2. |
lucee@6 ColdBox ^7.0.0 Test Results123 tests 123 ✅ 6s ⏱️ Results for commit 393c0c2. |
adobe@2023 ColdBox ^7.0.0 Test Results123 tests 123 ✅ 6s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
adobe@2023 ColdBox ^8.0.0 Test Results123 tests 123 ✅ 5s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
adobe@2023 ColdBox be Test Results123 tests 123 ✅ 4s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
adobe@2025 ColdBox ^7.0.0 Test Results123 tests 123 ✅ 6s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
adobe@2025 ColdBox ^8.0.0 Test Results123 tests 123 ✅ 6s ⏱️ Results for commit 393c0c2. ♻️ This comment has been updated with latest results. |
2 participants
Concurrent first requests can receive cbauth's authentication singleton before its storage dependencies are injected, causing
isLoggedIn()to fail when accessing session storage. cbauth 7 fixes this by marking the servicethreadsafe, but cbSecurity's^6.0.0dependency excludes that fix.Update the dependency to
^7.0.0. Add integration coverage for the resolved mapping's thread-safe publication and login/logout through cbSecurity's real cbauth provider. The cbauth 6.1.0-to-7.0.0 source comparison shows that the only authentication-service change is thethreadsafeannotation.Upstream fix: coldbox-modules/cbauth#25
Validation:
tests.specs.integration.CBSecuritySpec: 3 passed, 0 failures/errors on BoxLang 1.17.4, using the harness/tests/runner.bxm.box run-script formatandgit diff --checkpassed.