Yeah :)

I think we should pass the value as uint8_t and the return ptr (cookievalue) should be that as well.

A strncmp may fail if we have cookie names with \0 chars in the payload:

some\0name=blah would be printed for "some", but not for somename or some\0name (don't think there is a way to express non-printable chars in the cookiename log format currently, is there?)

So a memcmp with explicit lenghts is a better option I think.