René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"NPE when @SetsCookie is used but cookie was *not* set by the server","articleBody":"Tested this on version 3.0.\n\nGiven an interface like this:\n\n``` java\n@Rest(converters = { StringHttpMessageConverter.class, FormHttpMessageConverter.class })\npublic interface MyCoolRestInterface {\n    @Post(\"/very/cool?foo={bar}\")\n    @SetsCookie({ \"YUMMY\" })\n    @Accept(MediaType.APPLICATION_FORM_URLENCODED_VALUE)\n    String veryCool(MultiValueMap\u003cString, String\u003e params, boolean bar);\n\n    @Get(\"needs/cookie\")\n    @RequiresCookie(\"YUMMY\")\n    String needsCookie();\n}\n```\n\nWhen the response comes back from the server and YUMMY **_is not**_ set, we get an NPE at the for loop in the generated code (see _RIGHT HERE_ below):\n\n``` java\n@Override\npublic String veryCool(MultiValueMap\u003cString, String\u003e params, boolean bar) {\n    ...\n    String[] requestedCookies = new String[] {\"YUMMY\"};\n    List\u003cString\u003e allCookies = response.getHeaders().get(\"Set-Cookie\");\n    *** RIGHT HERE *** \u003e for (String rawCookie: allCookies) {\n        ...\n}\n```\n\nbecause **_Set-Cookie**_ header is not present, and **allCookies** is **null**. Well, at least in my case it is. This should be better guarded by a null check of the list returned by the _get()_ call.\n","author":{"url":"https://github.com/kutsal","@type":"Person","name":"kutsal"},"datePublished":"2014-01-02T21:07:38.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":3},"url":"https://github.com/855/androidannotations/issues/855"}