validation of 'keywords' field is too loose

[karenetheridge]

https://metacpan.org/pod/CPAN::Meta::Spec#keywords - "Keywords must not include whitespace." However, CPAN::Meta::Validator only validates the 'keywords' entry as { list => { value => &string } }. 'string' is too lax here.

The other meta validation packages also contain the same issue: Test::CPAN::Meta::Version, Test::CPAN::Meta::YAML::Version, Test::CPAN::Meta::JSON::Version.

Here's an example in the wild that should fail a validation test: https://metacpan.org/source/TOBYINK/Type-Tiny-0.038/META.json#L8

[dagolden]

Good catch. We should get a fix for that in the validator.

On Thu, Feb 27, 2014 at 6:20 PM, Karen Etheridge
notifications@github.comwrote:

https://metacpan.org/pod/CPAN::Meta::Spec#keywords - "Keywords must not
include whitespace." However, CPAN::Meta::Validator only validates the
'keywords' entry as { list => { value => &string } }. 'string' is too lax
here.

The other meta validation packages also contain the same issue:
Test::CPAN::Meta::Version, Test::CPAN::Meta::YAML::Version,
Test::CPAN::Meta::JSON::Version.

Here's an example in the wild that should fail a validation test:
https://metacpan.org/source/TOBYINK/Type-Tiny-0.038/META.json#L8

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/48
.

David Golden xdg@xdg.me
Take back your inbox! → http://www.bunchmail.com/
Twitter/IRC: @xdg