Skip to content

doc: res.getHeaders return value wrong in docsΒ #12885

Closed

Description

@refack
  • Version: 7
  • Platform: *
  • Subsystem: HTTP

Ref: https://github.com/nodejs/node/pull/12883/files#r115149721

      const headersCopy = res.getHeaders();
      assert.ok(headersCopy instanceof Object);

Fails with AssertionError [ERR_ASSERTION]: false == true
In contradiction to API https://nodejs.org/api/http.html#http_response_getheaders

Activity

  1. refack commented on May 7, 2017

    @refack
    ContributorAuthor

    More concretely
    headersCopy.hasOwnProperty === undefined
    &
    headersCopy.propertyIsEnumerable === undefined

  2. added
    httpIssues and PRs related to the http subsystem.
    testIssues and PRs related to Node.js core tests and test infrastructure.
    on May 7, 2017
  3. removed
    testIssues and PRs related to Node.js core tests and test infrastructure.
    on May 7, 2017
  4. addaleax commented on May 7, 2017

    @addaleax
    Member

    See #12883 (comment) – can please you be precise about what the issue is or what you want to do here? Otherwise this issue is just stating facts.

  5. refack commented on May 7, 2017

    @refack
    ContributorAuthor

    If there is a reason why it has a null prototype, we need to correct the API documentation.
    Else just give it a prototype.

  6. addaleax commented on May 7, 2017

    @addaleax
    Member

    If there is a reason why it has a null prototype

    Yes, because header names can clash with the built-in fields of Object.prototype (for example with __proto__ itself).

    correct the API

    Are you talking about docs?

  7. refack commented on May 7, 2017

    @refack
    ContributorAuthor

    Are you talking about docs?

    Yes

  8. added
    docIssues and PRs related to Node.js documentation.
    on May 7, 2017
  9. mscdex commented on May 7, 2017

    @mscdex
    Contributor

    You can probably just copy the text from querystring.parse().

  10. changed the title [-]http: res.getHeaders return value not an Object[/-] [+]doc: res.getHeaders return value wrong in docs[/+] on May 8, 2017
  11. refack commented on May 10, 2017

    @refack
    ContributorAuthor

    Fixed by: #12887

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

    docIssues and PRs related to Node.js documentation.httpIssues and PRs related to the http subsystem.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions