Skip to content

Trivial: Reject config block for log_skip #7050

@francislavoie

Description

@francislavoie

These should not successfully parse as a Caddyfile:

log_skip {
    path /api/* /socket/*
}

log_skip foo bar

👆 this is a config based on misunderstanding how matchers work, and should be rejected during unmarshal. If there's a block ({ }), or any arguments at all, then it should return an error immediately.

A correct config would look like this:

@skipped path /api/* /socket/*
log_skip @skipped

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions