{"action":"closed","issue":{"url":"https://api.github.com/repos/swagger-api/swagger-socket/issues/22","labels_url":"https://api.github.com/repos/swagger-api/swagger-socket/issues/22/labels{/name}","comments_url":"https://api.github.com/repos/swagger-api/swagger-socket/issues/22/comments","events_url":"https://api.github.com/repos/swagger-api/swagger-socket/issues/22/events","html_url":"https://github.com/swagger-api/swagger-socket/issues/22","id":55889790,"number":22,"title":"The headers building code in Response is inconsistent","user":{"login":"elakito","id":1558648,"avatar_url":"https://avatars.githubusercontent.com/u/1558648?v=3","gravatar_id":"","url":"https://api.github.com/users/elakito","html_url":"https://github.com/elakito","followers_url":"https://api.github.com/users/elakito/followers","following_url":"https://api.github.com/users/elakito/following{/other_user}","gists_url":"https://api.github.com/users/elakito/gists{/gist_id}","starred_url":"https://api.github.com/users/elakito/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/elakito/subscriptions","organizations_url":"https://api.github.com/users/elakito/orgs","repos_url":"https://api.github.com/users/elakito/repos","events_url":"https://api.github.com/users/elakito/events{/privacy}","received_events_url":"https://api.github.com/users/elakito/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/swagger-api/swagger-socket/labels/2.0","name":"2.0","color":"e102d8"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-29T11:52:18Z","updated_at":"2015-02-04T23:42:28Z","closed_at":"2015-02-04T23:42:28Z","body":"Currently, SwaggerSocketProtocolInterceptor creates a Response by adding headers as\r\n        Map<String, String> headers = res.headers();\r\n        for (String s : headers.keySet()) {\r\n            builder.header(new Header(s, headers.get(s)));\r\n        }\r\n\r\nBut the Response itself does not use the headers (not read from the builder when it is instantiated).\r\n\r\nSo there is an inconsistency in the code. I don't think including the above response headers in the swagger-socket response makes sense. But it probably makes sense to include some headers in the response. (e.g., Content-Type). We could also add an option to not include any headers.\r\n"}}