Skip to content

Content type not set correctly for .badRequest case #345

Open

Description

@jerbeers-personal

If I have a test that needs to send a body with a .badRequest response, the content-type header is not set correctly. The headers() method on HttpResponse should evaluate the body of .badRequest the same way it does for .ok. It's a bit complicated by the fact that the associated data for .ok is not optional and .badRequest is optional, but if badRequest does have a body, it should be handled the same way.

Activity

  1. Vkt0r commented on Oct 24, 2018

    @Vkt0r
    Member

    You can always use the HttpResponse.raw and provide your own Headers. See the following example:

    let statusCode = 400
    let reasonPhrase = "Bad Request"
    let headers = ["Content-Type": "application/json"]
    
    let response =  HttpResponse.raw(statusCode, reasonPhrase , headers,  { writer in
        // jsonData represent your JSON string to return converted to `Data` 
        // or any other implementing `HttpResponseBodyWriter`
        try writer.write(jsonData)
    })
  2. jerbeers-personal commented on Oct 24, 2018

    @jerbeers-personal
    Author

    Yeah, I used that as a workaround, but it still seems like the code should automatically handle the header correctly, so I filed the issue. Thanks!

  3. pinned this issue on Apr 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions