Contrib.social

The PECL UUID implementation can return invalid UUIDs

Open on Drupal.org →
Created on 12 November 2015, over 8 years ago
Updated 31 January 2023, over 1 year ago

Problem/Motivation

On my system (Mac with PHP 5.5.29 from homebrew) the Pecl UUID implementation is used and it produces UUIDs in uppercase, like 86AF9163-9980-4DD2-A7D3-D48515F82551. This is however not valid according to Uuid::isValid() which requires lowercase.

Proposed resolution

strtolower() the result of uuid_create() in Uuid\Pecl?

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Needs work

Version

10.1 ✨

Component
Base  →

Last updated less than a minute ago

Created by

🇸🇪Sweden Arla

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Comment over 1 year ago →
    🇺🇸United States smustgrave

    This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

    If still a valid bug in D10 it will need a test case to show it please

    Thanks!

  • Comment over 1 year ago →
    🇬🇧United Kingdom catch

    I think we can skip tests here, there's not really a bug as such, we're just normalizing something to lowercase. To write test coverage we'd need to know PECL is always including upper case characters when it generates a UUID, but it might not every time so the test could easily have false negatives.

contrib.social Blog FAQ Discussions
Production build 0.69.0 2024