[3.x] Incorrect uuid value for column webauthn_credentials.aaguid

[codebarista]

PHP & Platform

8.3.10 - Alpine Linux v3.20 x64

Database

MariaDB 10.11.8

Laravel version

11.23.4

Have you done this?

• I have checked my logs and I'm sure is a bug in this package.
• I can reproduce this bug in isolation (vanilla Laravel install)
• I can suggest a workaround as a Pull Request

Expectation

When I register a passkey with Google Chrome Profile, I expect that it is stored to the database.

Description

Instead MariaDB sometimes throws a validation exception for aaguid. It works for one host, but not for another (always the same aaguid). So far only in latest Chrome with Chrome Profile for the passkey. The workaround would be to set the aaguid column type to char(36) for MariaDB.

Reproduction

\Laragear\WebAuthn\Models\WebAuthnCredential::create([
    'authenticatable_id' => 1,
    'authenticatable_type' => 'App\Models\User',
    'id' => 'lYcEEbf7tlNrVSLB9m-g_eptRRG7LMOuqNf3PvWLSsM',
    'user_id' => '66d3243c17184e60b90dffedce626d9e',
    'alias' => '',
    'counter' => 0,
    'rp_id' => 'codebarista.test',
    'origin' => 'https://codebarista.test',
    'transports' => '',
    'aaguid' => 'adce0002-35bc-c60a-648b-0b25f1f05503', // original incorrect uuid value
    'public_key' => 'eyJpdiI6IlV5TWpObG1LMS91ODJSelhNNXlkQmc9P...',
    'attestation_format' => 'none',
    'updated_at' => '2024-09-12 17:34:34',
    'created_at' => '2024-09-12 17:34:34',
]);

Stack trace & logs

SQLSTATE[22007]: Invalid datetime format: 1292 Incorrect uuid value: 'adce0002-35bc-c60a-648b-0b25f1f05503' for column `webauthn_credentials`.`aaguid` at row 1 (Connection: mariadb, SQL: insert into `webauthn_credentials`...)

[DarkGhostHunter]

Bring a PR that uses that column type when the connection is mariadb. Otherwise, you’ll have to wait until my LBP is gone (no eta). Italo Baeza C.El 12-09-2024, a la(s) 15:20, Ralf ***@***.***> escribió: Assigned #96 to @DarkGhostHunter. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were assigned.Message ID: ***@***.***>

[codebarista]

Get well soon. 🙂

[DarkGhostHunter]

By the way, can you set the raw attribute of the aaguid as 32 characters before saving?That could work as a workaround.Italo Baeza C.El 12-09-2024, a la(s) 16:53, Ralf ***@***.***> escribió: Get well soon. 🙂 —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>