Skip to content

Should support HEAD request as well #279

Closed

Description

@javenwang

if (req.method !== 'GET') {
return goNext();
}

Activity

  1. evenstensberg commented on Mar 5, 2018

    @evenstensberg
    Member

    Could you elaborate a bit more in depth about this, maybe the problem you are trying to solve as well as how you'd think it should look like?

  2. shellscape commented on Mar 5, 2018

    @shellscape
    Contributor

    Closing as invalid, since the issue template was removed.

  3. michaek commented on Jul 26, 2018

    @michaek

    It seems to me the original reporter is saying that HEAD requests should be handled as GET requests, just adding a req.method !== 'HEAD' to the conditional. It seems to me this wouldn't be harmful, but it may be nice to respond with empty content for a HEAD request. Currently I'm putting a another middleware in front of this to handle HEAD requests.

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

    No labels
    No labels

    Type

    No type

    Fields

    Priority

    None yet

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions