Skip to content

unnecessary explicit defaults #1660

@Enrice

Description

@Enrice

This module implements multiple directives with explicit defaults, which are unnecessary, because nginx does already have feasible defaults, so we should rely on them.
The parameters should be changed to Optional with undef defaults, so they won't be set in the configuration if not set otherwise.

Also we should streamline the types of the parameters. For example If they specify numbers, they should be declared as such.

I could do that and provide a PR if you agree.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions