MSSQL uuid column showing as length 16 rather than 36

[dariusj18]

Describe the Bug

On an MSSQL database, directus is showing a length of 16 (and limiting the input field to 16 characters) for fields set with a type of UUID.

To Reproduce

Add a new uuid field to a table

Directus Version

10.10.5

Hosting Strategy

Self-Hosted (Docker Image)

[dariusj18]

This appears to be because MSSQL is reporting the max_length as 16 in the sys.columns table. I assume because it is stored internally as a binary(16) rather than a char(36).

Should be as easy to fix as adding

if (rawColumn.data_type === 'uniqueidentifier') {
    return 36;
}

to the mssql.ts dialect's parseMaxLength() function

[dariusj18]

Another thing to consider is that if a schema export contains the max_length of 16 directus will also need to know that it should ahve been 36 and not try and make changes to preexisting columns when that difference is detected.