* Update plugins.go

* Add extraInfo to eventHook

* Add extraInfo to eventHook

* Update run.go

* Update run.go

* Update run.go

Related to #1660 and #1667

* Browse: Show symbolic links and targets type properly
 * gofmt
Signed-off-by: Jonas Östanbäck <jonas.ostanback@gmail.com>

* Move symbolic link check in to isSymlinkTargetDir
Signed-off-by: Jonas Östanbäck <jonas.ostanback@gmail.com>

* Revert template change and show sym link folders as normal folders

* browse: Updated icons including symlink indicators

* Add grpc plugin directive

* Update plugin.go

Removed whitespace line

* Update plugin.go

Vendor all dependencies

The vendor/ folder was created with the help of @FiloSottile's gvt and
vendorcheck.

Any dependencies of Caddy plugins outside this repo are not vendored.

We do not remove any unused, vendored packages because vendorcheck -u
only checks using the current build configuration; i.e. packages that
may be imported by files toggled by build tags of other systems.

CI tests have been updated to ignore the vendor/ folder. When Go 1.9 is
released, a few of the go commands should be revised to again use ./...
as it will ignore the vendor folder by default.

* de-duplicates code for 'not' ops and replicates 'not' op for ends_with

* fixes incorrect test expectations

* Add not_starts_with condition.

This adds the opposite of the starts_with condition, to check if a
given string does not start with another string.

* Correct white space problems

* Add uri policy test cases
 * Add function definition
 * Add uri hashing policy
 * Refactor and extract hostByHashing and use in IP and URI policy
 * Rename to URIHash
Signed-off-by: Jonas Östanbäck <jonas.ostanback@gmail.com>

Fixes #1674

* Correct proxy policy documentation
Signed-off-by: Jonas Östanbäck <jonas.ostanback@gmail.com>

* Change first's select() wording
Signed-off-by: Jonas Östanbäck <jonas.ostanback@gmail.com>

Signed-off-by: Jonas Östanbäck <jonas.ostanback@gmail.com>

* Add check for maximum number of arguments to log directive
 * Add failing test case
Signed-off-by: Jonas Östanbäck <jonas.ostanback@gmail.com>

* Change else ifs into switch
Signed-off-by: Jonas Östanbäck <jonas.ostanback@gmail.com>

* Refactor
Signed-off-by: Jonas Östanbäck <jonas.ostanback@gmail.com>

* Typo
Signed-off-by: Jonas Östanbäck <jonas.ostanback@gmail.com>

* proxy: added 'health_check_port' to upstream

* proxy: `net.JoinHostPort` instead of `fmt.Printf` for upstream checks

* proxy: changing health_check_port type (int->string)

adding tests for invalid port config

log: allow additional prefix/suffix with predefined format

Signed-off-by: Tw <tw19881113@gmail.com>

fix issue #1652
Signed-off-by: Tw <tw19881113@gmail.com>

Signed-off-by: Thibault Meyer <meyer.thibault@gmail.com>

Introduce `limits` middleware

Previously, the `Include` override used with the markdown plugin did not
provide the optional `args` parameter. This made it impossible to pass
arguments to a template used with that plugin.

1. Replace original `maxrequestbody` directive.
2. Add request header limit.

fix issue #1587
Signed-off-by: Tw <tw19881113@gmail.com>

We now use the release program to assist in deploying Caddy (it is much
more integrated and automated): https://github.com/caddyserver/releaser

This older automation code can still be found in this gist:
https://gist.github.com/mholt/cb7285f4950cb93f23be0aa6050fb043

Signed-off-by: Tw <tw19881113@gmail.com>

Signed-off-by: Tw <tw19881113@gmail.com>

httpserver: More path matching tests

Fixes #1645

This reverts commit e0ed7093.

* httpserver/all: Clean up and standardize request URL handling

The HTTP server now always creates a context value on the request which
is a copy of the request's URL struct. It should not be modified by
middlewares, but it is safe to get the value out of the request and make
changes to it locally-scoped. Thus, the value in the context always
stores the original request URL information as it was received. Any
rewrites that happen will be to the request's URL field directly.

The HTTP server no longer cleans /sanitizes the request URL. It made too
many strong assumptions and ended up making a lot of middleware more
complicated, including upstream proxying (and fastcgi). To alleviate
this complexity, we no longer change the request URL. Middlewares are
responsible to access the disk safely by using http.Dir or, if not
actually opening files, they can use httpserver.SafePath().

I'm hoping this will address issues with #1624, #1584, #1582, and others.

* staticfiles: Fix test on Windows

@abiosoft: I still can't figure out exactly what this is for. 😅

* Use (potentially) changed URL for browse redirects, as before

* Use filepath.ToSlash, clean up a couple proxy test cases

* Oops, fix variable name