{"action":"created","issue":{"url":"https://api.github.com/repos/netty/netty/issues/6730","repository_url":"https://api.github.com/repos/netty/netty","labels_url":"https://api.github.com/repos/netty/netty/issues/6730/labels{/name}","comments_url":"https://api.github.com/repos/netty/netty/issues/6730/comments","events_url":"https://api.github.com/repos/netty/netty/issues/6730/events","html_url":"https://github.com/netty/netty/issues/6730","id":227668751,"number":6730,"title":"Cookie decoder stopped throwing exception on malformed cookie name","user":{"login":"alobodzki","id":28387622,"avatar_url":"https://avatars2.githubusercontent.com/u/28387622?v=3","gravatar_id":"","url":"https://api.github.com/users/alobodzki","html_url":"https://github.com/alobodzki","followers_url":"https://api.github.com/users/alobodzki/followers","following_url":"https://api.github.com/users/alobodzki/following{/other_user}","gists_url":"https://api.github.com/users/alobodzki/gists{/gist_id}","starred_url":"https://api.github.com/users/alobodzki/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alobodzki/subscriptions","organizations_url":"https://api.github.com/users/alobodzki/orgs","repos_url":"https://api.github.com/users/alobodzki/repos","events_url":"https://api.github.com/users/alobodzki/events{/privacy}","received_events_url":"https://api.github.com/users/alobodzki/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2017-05-10T12:50:16Z","updated_at":"2017-05-10T14:45:43Z","closed_at":null,"body":"We want to switch from 4.0 to 4.1, but with 4.1 cookie decoding changed and on illegal characters cookie decoding procedure doesn't throw exception anymore. The piece of code that was throwing exception was in `io.netty.handler.codec.http.cookie.DefaultCookie#DefaultCookie`.\r\n\r\n### Expected behavior\r\nWhen a cookie contains illegal characters in name/value cookie decoder could, based on some parameter, throw an exception, so the server could handle it.\r\n### Actual behavior\r\nThere are two modes in CookieDecoder: LAX or STRICT. LAX allowes illegal characters, STRICT swallows malformed cookie. In both cases there is no feedback.\r\n### Steps to reproduce\r\nTests in `io.netty.handler.codec.http.cookie.ServerCookieDecoderTest` are enough to highlight this situation, specially `io.netty.handler.codec.http.cookie.ServerCookieDecoderTest#testRejectCookieValueWithSemicolon`\r\n### Netty version\r\nNetty 4.1.11.Final"},"comment":{"url":"https://api.github.com/repos/netty/netty/issues/comments/300505305","html_url":"https://github.com/netty/netty/issues/6730#issuecomment-300505305","issue_url":"https://api.github.com/repos/netty/netty/issues/6730","id":300505305,"user":{"login":"alobodzki","id":28387622,"avatar_url":"https://avatars2.githubusercontent.com/u/28387622?v=3","gravatar_id":"","url":"https://api.github.com/users/alobodzki","html_url":"https://github.com/alobodzki","followers_url":"https://api.github.com/users/alobodzki/followers","following_url":"https://api.github.com/users/alobodzki/following{/other_user}","gists_url":"https://api.github.com/users/alobodzki/gists{/gist_id}","starred_url":"https://api.github.com/users/alobodzki/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alobodzki/subscriptions","organizations_url":"https://api.github.com/users/alobodzki/orgs","repos_url":"https://api.github.com/users/alobodzki/repos","events_url":"https://api.github.com/users/alobodzki/events{/privacy}","received_events_url":"https://api.github.com/users/alobodzki/received_events","type":"User","site_admin":false},"created_at":"2017-05-10T14:45:43Z","updated_at":"2017-05-10T14:45:43Z","body":"Our app is doing proxying, we don't know which cookie is required and which one is optional."}}
{"actor":{"display_login":"alobodzki"}}