Extension name with more than 20 characters throws error

[deissnerk]

SetExtension() of the v1.0 event context throws an error, if the name of the extension is longer than 20 characters. In the spec this length restriction is a SHOULD NOT restriction. So failing with an error seems to be too strong. Shouldn't it be possible to violate SHOULD requirements? 😉

[n3wscott]

It should be possible. Good point. What version of the sdk are you seeing this in? it is in the latest 2.4.0?

[deissnerk]

We experienced it first with the 2.4.0 release. I think it might have sneaked in with this commit.

[n3wscott]

Yep. ok thanks! I will get a point fix release out in the next couple days.

[deissnerk]

Thank you. If it is only about removing the length check, I can come up with a little PR, too.

[n3wscott]

@deissnerk If you want to write the PR, that would be great, I was thinking of having a global setting that an importer can set the max length to something they wish, but have the default be unlimited (0).