Closed ctavan closed 4 years ago
@marcoscaceres any idea what this "Travis CI - Branch" is this PR seems to be blocked on? I'd expect "Travis CI - Pull Request" to be required, not the branch version, since afaik the branch version doesn't work for pull requests made from forks. Is this some misconfiguration?
Oh, this might be stuck because of the trivisci.com vs travisci.org change... I'll just merge.
The algorithm for creating a blob URL references RFC4122 to generate a UUID. However the example contains an invalid UUID (according to RFC4122):
9115d58c-bcda-ff47-86e5-083e9a215304
This given string is not a valid UUID because it does not contain a meaningful version field (the first character after the second dash).
It could be worthwhile to consider specifying the specific UUID algorithm to be used (Version 4 UUIDs) however that might be an actual change in the spec, not just an editorial change like the one I propose here.
Preview | Diff