Skip to content

https: https.Server should be more careful with passed options #13584

Closed
@mscdex

Description

@mscdex
  • Version: all
  • Platform: n/a
  • Subsystem: https

Currently the https.Server constructor has two issues:

  • It does not check that the user-provided opts is truthy before trying to access properties on the value
  • It mutates the user-provided opts if certain properties on it are false-y

Metadata

Metadata

Assignees

No one assigned

    Labels

    good first issueIssues that are suitable for first-time contributors.httpsIssues or PRs related to the https subsystem.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions