Switch Handler to an interface.

HandlerFunc is much better as a function returning a Handler, rather
than a newtype for the function type itself. This way there is no
confusion creating a type-inferenced variable with HandlerFunc(func(...
and then using a HandlerFunc where a Handler is expected. Much better to
only have one public type.

1 files changed, 3 insertions, 3 deletions

diff --git a/contrib/sharedhost/replacement.go b/contrib/sharedhost/replacement.go
index 1fb2a0d..9267530 100644
--- a/contrib/sharedhost/replacement.go
+++ b/contrib/sharedhost/replacement.go
@@ -19,14 +19,14 @@ import (
 // "users/", "domain.com/~jim/index.gmi" maps to "domain.com/users/jim/index.gmi".
 func ReplaceTilde(replacement string) gus.Middleware {
 	return func(inner gus.Handler) gus.Handler {
-		return func(ctx context.Context, request *gus.Request) *gus.Response {
+		return gus.HandlerFunc(func(ctx context.Context, request *gus.Request) *gus.Response {
 			if len(request.Path) > 1 && request.Path[0] == '/' && request.Path[1] == '~' {
 				request = cloneRequest(request)
 				request.Path = "/" + replacement + request.Path[2:]
 			}
 
-			return inner(ctx, request)
-		}
+			return inner.Handle(ctx, request)
+		})
 	}
 }