Fix: file opening path and directory browsing path of filesystem

middleware/filesystem/README.md

@@ -44,7 +44,7 @@ Then create a Fiber app with `app := fiber.New()`.
 ```go
 // Provide a minimal config
 app.Use(filesystem.New(filesystem.Config{
-	Root: http.Dir("./assets")
+	Root: http.Dir("./assets"),
 }))
 
 // Or extend your config for customization
@@ -96,7 +96,7 @@ func main() {
 	// `http://<server>/static/static/image.png`.
 	app.Use("/static", filesystem.New(filesystem.Config{
 		Root: http.FS(embedDirStatic),
-		PathPrefix: "static"
+		PathPrefix: "static",
 		Browse: true,
 	}))
 

middleware/filesystem/filesystem.go

@@ -131,6 +131,9 @@ func New(config ...Config) fiber.Handler {
 			stat os.FileInfo
 		)
 
+		if len(path) > 1 {
+			path = strings.TrimSuffix(path, "/")
+		}
 		file, err = cfg.Root.Open(path)
 		if err != nil && os.IsNotExist(err) && cfg.NotFoundFile != "" {
 			file, err = cfg.Root.Open(cfg.NotFoundFile)

middleware/filesystem/filesystem_test.go

@@ -89,6 +89,12 @@ func Test_FileSystem(t *testing.T) {
 			statusCode:  200,
 			contentType: "text/html",
 		},
+		{
+			name:        "Should list the directory contents",
+			url:         "/dir/img/",
+			statusCode:  200,
+			contentType: "text/html",
+		},
 		{
 			name:        "Should be returns status 200",
 			url:         "/dir/img/fiber.png",

middleware/filesystem/utils.go

@@ -40,7 +40,7 @@ func dirList(c *fiber.Ctx, f http.File) error {
 	fmt.Fprint(c, "<ul>")
 
 	if len(basePathEscaped) > 1 {
-		parentPathEscaped := html.EscapeString(c.Path() + "/..")
+		parentPathEscaped := html.EscapeString(strings.TrimSuffix(c.Path(), "/") + "/..")
 		fmt.Fprintf(c, `<li><a href="%s" class="dir">..</a></li>`, parentPathEscaped)
 	}