Update RequestHandlersImpl.h (espressif#6179)

mathertel · web-flow · commit 9f08cf4767d4 · 2022-01-31T13:09:04.000+02:00
With LittleFS the `fs.exists(path)` returns true also on folders. A `isDirectory()` call is required to set _isFile to false on directories.
This enables serving all files from a folder like : `server-&gt;serveStatic("/", LittleFS, "/", cacheHeader.c_str());
        File f = fs.open(path);
        _isFile = (f &amp;&amp; (! f.isDirectory()));
diff --git a/libraries/WebServer/src/detail/RequestHandlersImpl.h b/libraries/WebServer/src/detail/RequestHandlersImpl.h
@@ -68,7 +68,8 @@ class StaticRequestHandler : public RequestHandler {
     , _path(path)
     , _cache_header(cache_header)
     {
-        _isFile = fs.exists(path);
+        File f = fs.open(path);
+        _isFile = (f && (! f.isDirectory()));
         log_v("StaticRequestHandler: path=%s uri=%s isFile=%d, cache_header=%s\r\n", path, uri, _isFile, cache_header ? cache_header : ""); // issue 5506 - cache_header can be nullptr
         _baseUriLength = _uri.length();
     }

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,8 @@ class StaticRequestHandler : public RequestHandler {`
`68`	`68`	`, _path(path)`
`69`	`69`	`, _cache_header(cache_header)`
`70`	`70`	`{`
`71`		`- _isFile = fs.exists(path);`
	`71`	`+ File f = fs.open(path);`
	`72`	`+ _isFile = (f && (! f.isDirectory()));`
`72`	`73`	`log_v("StaticRequestHandler: path=%s uri=%s isFile=%d, cache_header=%s\r\n", path, uri, _isFile, cache_header ? cache_header : ""); // issue 5506 - cache_header can be nullptr`
`73`	`74`	`_baseUriLength = _uri.length();`
`74`	`75`	`}`