caddyhttp: Fix logging on wildcard sites when SkipUnmappedHosts is true

[francislavoie]

Fixes https://caddy.community/t/caddy-site-access-log-empty/33237, an edgecase bug when a wildcard site block is used alongside a catch-all block of the same listener. We weren't checking logger names with wildcards in shouldLogRequest so it would fall through to the return !s.Logs.SkipUnmappedHosts case and prevent logging when it should have been logged.

Repro config:

http://*.localhost:8881 {
    log

    @foo host foo.localhost
    handle @foo {
        respond "foo"
    }

    handle {
        log_skip
        respond "Nope" 404
    }
}

:8881 {
    respond "catch-all"
}

curl -v foo.localhost:8881 should log, and curl -v bar.localhost:8881 should not log (because of log_skip). Before, neither logged.

Assistance Disclosure

None