On Sat, 9 Sep 2017, Kevin Smith wrote:
Yeah, I see in the code where a check could be added for making sure the cookie value is not too long. I have never submitted a fix to curl before, do I just provide a diff here or open a pull request in Github? Would you want any unit test to show it is working?
Ideally a pull request, as that will also make a lot of extra checks and tests run to make sure the code is reasonably fine. But if you submit a patch here, someone else can push it to github to have the CI tests verify it.
I'd love a test case or two for it, yes. It should just be a matter of sending curl an overly long cookie and make sure curl doesn't save it.
-- / daniel.haxx.se ------------------------------------------------------------------- Unsubscribe: https://cool.haxx.se/list/listinfo/curl-library Etiquette: https://curl.haxx.se/mail/etiquette.html
