fix(serverHandler): enable dir slash for directory only

Author: marjune163

src/serverHandler/responseData.go

@@ -353,7 +353,7 @@ func (h *aliasHandler) getResponseData(r *http.Request) (data *responseData, fsP
 		status = getStatusByErr(_statErr)
 	}
 
-	needDirSlashRedirect := h.forceDirSlash > 0 && prefixReqPath[len(prefixReqPath)-1] != '/' && !shouldServeAsContent(file, item)
+	needDirSlashRedirect := h.forceDirSlash > 0 && prefixReqPath[len(prefixReqPath)-1] != '/' && item != nil && item.IsDir()
 
 	indexFile, indexItem, _statIdxErr := h.statIndexFile(rawReqPath, reqFsPath, item, authSuccess && !needDirSlashRedirect)
 	if _statIdxErr != nil {
@@ -390,6 +390,11 @@ func (h *aliasHandler) getResponseData(r *http.Request) (data *responseData, fsP
 		errs = append(errs, _dereferenceErrs...)
 	}
 
+	// update `needDirSlashRedirect` for dangling intermediate alias directory
+	if !needDirSlashRedirect && h.forceDirSlash > 0 && len(subItems) > 0 && prefixReqPath[len(prefixReqPath)-1] != '/' {
+		needDirSlashRedirect = true
+	}
+
 	subItems = h.FilterItems(subItems)
 	rawSortBy, sortState := sortInfos(subItems, rawQuery, h.defaultSort)
 

test/case/041.force.dir.slash.bash

@@ -30,4 +30,6 @@ assert $(curl_head_status 'http://127.0.0.1:3003/foo/bar/lorem/ipsum/a') '301'
 	fail "incorrect redirect location"
 assert $(curl_head_status 'http://127.0.0.1:3003/foo/bar/lorem/ipsum/a/') '200'
 
+assert $(curl_head_status 'http://127.0.0.1:3003/foo/bar/lorem/ipsum/a/not-exist') '404'
+
 jobs -p | xargs kill &> /dev/null

test/case/041.force.dir.slash.status.code.bash

@@ -30,4 +30,6 @@ assert $(curl_head_status 'http://127.0.0.1:3003/foo/bar/lorem/ipsum/a') '302'
 	fail "incorrect redirect location"
 assert $(curl_head_status 'http://127.0.0.1:3003/foo/bar/lorem/ipsum/a/') '200'
 
+assert $(curl_head_status 'http://127.0.0.1:3003/foo/bar/lorem/ipsum/a/not-exist') '404'
+
 jobs -p | xargs kill &> /dev/null