1 Commits

Author SHA1 Message Date
Sebastian a25569d6f2 refactor: migrate to scoped execution contexts
Signed-off-by: Sebastian Krupinski <krupinski01@gmail.com>
2026-07-27 00:52:28 -04:00
3 changed files with 15 additions and 15 deletions
+1 -1
View File
@@ -10,7 +10,7 @@ jobs:
runs-on: ubuntu-latest runs-on: ubuntu-latest
services: services:
mongo: mongo:
image: mongo:8 image: mongo:7
options: >- options: >-
--health-cmd "mongosh --quiet --eval \"db.adminCommand('ping')\"" --health-cmd "mongosh --quiet --eval \"db.adminCommand('ping')\""
--health-interval 5s --health-interval 5s
+6 -6
View File
@@ -5,7 +5,7 @@ declare(strict_types=1);
namespace KTXM\AuthenticationProviderTotp\Controllers; namespace KTXM\AuthenticationProviderTotp\Controllers;
use KTXC\Http\Response\JsonResponse; use KTXC\Http\Response\JsonResponse;
use KTXC\SessionIdentity; use KTXC\Context\IdentityContextInterface;
use KTXF\Controller\ControllerAbstract; use KTXF\Controller\ControllerAbstract;
use KTXF\Routing\Attributes\AuthenticatedRoute; use KTXF\Routing\Attributes\AuthenticatedRoute;
use KTXM\AuthenticationProviderTotp\Services\EnrollmentService; use KTXM\AuthenticationProviderTotp\Services\EnrollmentService;
@@ -23,7 +23,7 @@ use KTXM\AuthenticationProviderTotp\Services\EnrollmentService;
class EnrollmentController extends ControllerAbstract class EnrollmentController extends ControllerAbstract
{ {
public function __construct( public function __construct(
private readonly SessionIdentity $userIdentity, private readonly IdentityContextInterface $identityContext,
private readonly EnrollmentService $enrollmentService, private readonly EnrollmentService $enrollmentService,
) {} ) {}
@@ -33,7 +33,7 @@ class EnrollmentController extends ControllerAbstract
#[AuthenticatedRoute('/totp/status', name: 'totp.status', methods: ['GET'])] #[AuthenticatedRoute('/totp/status', name: 'totp.status', methods: ['GET'])]
public function status(): JsonResponse public function status(): JsonResponse
{ {
$userId = $this->userIdentity->identifier(); $userId = $this->identityContext->identifier();
if (!$userId) { if (!$userId) {
return new JsonResponse( return new JsonResponse(
['error' => 'User not authenticated', 'error_code' => 'unauthorized'], ['error' => 'User not authenticated', 'error_code' => 'unauthorized'],
@@ -56,7 +56,7 @@ class EnrollmentController extends ControllerAbstract
#[AuthenticatedRoute('/totp/enroll', name: 'totp.enroll.begin', methods: ['POST'])] #[AuthenticatedRoute('/totp/enroll', name: 'totp.enroll.begin', methods: ['POST'])]
public function beginEnrollment(string $accountName = ''): JsonResponse public function beginEnrollment(string $accountName = ''): JsonResponse
{ {
$userId = $this->userIdentity->identifier(); $userId = $this->identityContext->identifier();
if (!$userId) { if (!$userId) {
return new JsonResponse( return new JsonResponse(
['error' => 'User not authenticated', 'error_code' => 'unauthorized'], ['error' => 'User not authenticated', 'error_code' => 'unauthorized'],
@@ -95,7 +95,7 @@ class EnrollmentController extends ControllerAbstract
#[AuthenticatedRoute('/totp/enroll/verify', name: 'totp.enroll.verify', methods: ['POST'])] #[AuthenticatedRoute('/totp/enroll/verify', name: 'totp.enroll.verify', methods: ['POST'])]
public function completeEnrollment(string $code): JsonResponse public function completeEnrollment(string $code): JsonResponse
{ {
$userId = $this->userIdentity->identifier(); $userId = $this->identityContext->identifier();
if (!$userId) { if (!$userId) {
return new JsonResponse( return new JsonResponse(
['error' => 'User not authenticated', 'error_code' => 'unauthorized'], ['error' => 'User not authenticated', 'error_code' => 'unauthorized'],
@@ -131,7 +131,7 @@ class EnrollmentController extends ControllerAbstract
#[AuthenticatedRoute('/totp/enroll/remove', name: 'totp.enroll.remove', methods: ['POST'])] #[AuthenticatedRoute('/totp/enroll/remove', name: 'totp.enroll.remove', methods: ['POST'])]
public function removeEnrollment(string $code): JsonResponse public function removeEnrollment(string $code): JsonResponse
{ {
$userId = $this->userIdentity->identifier(); $userId = $this->identityContext->identifier();
if (!$userId) { if (!$userId) {
return new JsonResponse( return new JsonResponse(
['error' => 'User not authenticated', 'error_code' => 'unauthorized'], ['error' => 'User not authenticated', 'error_code' => 'unauthorized'],
+8 -8
View File
@@ -4,7 +4,7 @@ declare(strict_types=1);
namespace KTXM\AuthenticationProviderTotp\Services; namespace KTXM\AuthenticationProviderTotp\Services;
use KTXC\SessionTenant; use KTXC\Context\TenantContextInterface;
use KTXM\AuthenticationProviderTotp\Stores\EnrollmentStore; use KTXM\AuthenticationProviderTotp\Stores\EnrollmentStore;
/** /**
@@ -26,7 +26,7 @@ class EnrollmentService
private const SECRET_LENGTH = 20; // 160 bits as recommended by RFC 4226 private const SECRET_LENGTH = 20; // 160 bits as recommended by RFC 4226
public function __construct( public function __construct(
private readonly SessionTenant $tenant, private readonly TenantContextInterface $tenantContext,
private readonly EnrollmentStore $enrollmentStore, private readonly EnrollmentStore $enrollmentStore,
) {} ) {}
@@ -35,7 +35,7 @@ class EnrollmentService
*/ */
public function isEnrolled(string $userId): bool public function isEnrolled(string $userId): bool
{ {
$tenantId = $this->tenant->identifier(); $tenantId = $this->tenantContext->identifier();
if (!$tenantId) { if (!$tenantId) {
return false; return false;
} }
@@ -53,7 +53,7 @@ class EnrollmentService
*/ */
public function beginEnrollment(string $userId, array $config = []): array public function beginEnrollment(string $userId, array $config = []): array
{ {
$tenantId = $this->tenant->identifier(); $tenantId = $this->tenantContext->identifier();
if (!$tenantId) { if (!$tenantId) {
return [ return [
'success' => false, 'success' => false,
@@ -108,7 +108,7 @@ class EnrollmentService
*/ */
public function completeEnrollment(string $userId, string $code): array public function completeEnrollment(string $userId, string $code): array
{ {
$tenantId = $this->tenant->identifier(); $tenantId = $this->tenantContext->identifier();
if (!$tenantId) { if (!$tenantId) {
return [ return [
'success' => false, 'success' => false,
@@ -150,7 +150,7 @@ class EnrollmentService
*/ */
public function removeEnrollment(string $userId): array public function removeEnrollment(string $userId): array
{ {
$tenantId = $this->tenant->identifier(); $tenantId = $this->tenantContext->identifier();
if (!$tenantId) { if (!$tenantId) {
return [ return [
'success' => false, 'success' => false,
@@ -181,7 +181,7 @@ class EnrollmentService
*/ */
public function verifyCode(string $userId, string $code): array public function verifyCode(string $userId, string $code): array
{ {
$tenantId = $this->tenant->identifier(); $tenantId = $this->tenantContext->identifier();
if (!$tenantId) { if (!$tenantId) {
return [ return [
'success' => false, 'success' => false,
@@ -225,7 +225,7 @@ class EnrollmentService
*/ */
public function getEnrollmentSecret(string $userId): ?string public function getEnrollmentSecret(string $userId): ?string
{ {
$tenantId = $this->tenant->identifier(); $tenantId = $this->tenantContext->identifier();
if (!$tenantId) { if (!$tenantId) {
return null; return null;
} }