Merge pull request #567 from traPtitech/fix/563

✨ add EnableLogger option

interfaces/handler/router.go

@@ -11,11 +11,18 @@ import (
 	"github.com/traPtitech/traPortfolio/usecases/repository"
 )
 
-func Setup(e *echo.Echo, api API) error {
+type Option func(*echo.Echo) error
+
+func Setup(e *echo.Echo, api API, opts ...Option) error {
 	e.HTTPErrorHandler = newHTTPErrorHandler(e)
 	e.Binder = &binderWithValidation{}
 
-	e.Use(middleware.Logger())
+	for _, opt := range opts {
+		if err := opt(e); err != nil {
+			return fmt.Errorf("apply option: %w", err)
+		}
+	}
+
 	e.Use(middleware.Recover())
 
 	apiGroup := e.Group("/api")
@@ -94,3 +101,10 @@ func (b *binderWithValidation) Bind(i interface{}, c echo.Context) error {
 
 	return nil
 }
+
+func WithRequestLogger() Option {
+	return func(e *echo.Echo) error {
+		e.Use(middleware.Logger())
+		return nil
+	}
+}

main.go

@@ -42,7 +42,7 @@ func main() {
 	}
 
 	e := echo.New()
-	if err := handler.Setup(e, api); err != nil {
+	if err := handler.Setup(e, api, handler.WithRequestLogger()); err != nil {
 		log.Fatal(err)
 	}