Skip to content

domain is now required as a string, and null does not pass validation #64

@mshick

Description

@mshick

Hapi's server.state accepts domain: null as an option, and this is a useful setting for me when in active development, because devs all use different local domains.

I think schema validation should be updated, and the logic here adapted to check property presence, not truthiness:

if (settings.hasOwnProperty('domain')) {
  cookieOptions.domain = settings.domain;
}

Metadata

Metadata

Assignees

Labels

featureNew functionality or improvement

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions