Got rid of serveStatic() attempt: it's difficult, I don't need it and it isn't used in the standard WebServer examples

Author: jackjansen

src/ESPWebServer.cpp

@@ -94,9 +94,7 @@ void ESPWebServer::on(const String &uri, HTTPMethod method, THandlerFunction fn,
 }
 
 void ESPWebServer::serveStatic(const char* uri, fs::FS& fs, const char* path, const char* cache_header) {
-  // TODO: add * or something to path so serveStatic("/dir/",...) will serve all content from that directory
-  ESPWebServerStaticNode *node = new ESPWebServerStaticNode(this, std::string(path), fs, path, cache_header);
-  _server.registerNode(node);
+  HTTPS_LOGE("serveStatic() not implemented");
 }
 
 void ESPWebServer::onNotFound(THandlerFunction fn) {
@@ -328,33 +326,3 @@ ESPWebServerNode::ESPWebServerNode(
 ESPWebServerNode::~ESPWebServerNode() {
 
 }
-
-ESPWebServerStaticNode::ESPWebServerStaticNode(
-  ESPWebServer *server,
-  const std::string &path,
-  FS& fs,
-  const char *filePath,
-  const char *cache_header)
-: ResourceNode(path, "GET", &(ESPWebServerStaticNode::_handlerWrapper), ""),
-  _fs(fs),
-  _filePath(filePath),
-  _cache_header(cache_header),
-  _isFile(false)
-{
-  _isFile = _fs.exists(filePath);
-}
-
-void ESPWebServerStaticNode::_handler(httpsserver::HTTPRequest *req, httpsserver::HTTPResponse *res) {
-  HTTPS_LOGE("static not yet implemented path=%s filePath=%s\n", _path.c_str(), _filePath);
-}
-
-void ESPWebServerStaticNode::_handlerWrapper(httpsserver::HTTPRequest *req, httpsserver::HTTPResponse *res) {
-  ESPWebServerStaticNode *node = (ESPWebServerStaticNode*)req->getResolvedNode();
-  node->_handler(req, res);
-}
-
-
-ESPWebServerStaticNode::~ESPWebServerStaticNode() {
-
-}
-

src/ESPWebServer.hpp

@@ -154,25 +154,4 @@ class ESPWebServerNode : public httpsserver::ResourceNode {
   const THandlerFunction _wrappedUploadHandler;
 };
 
-class ESPWebServerStaticNode : public httpsserver::ResourceNode {
-public:
-  ESPWebServerStaticNode(
-    ESPWebServer *server,
-    const std::string &path,
-    FS& fs,
-    const char *filePath,
-    const char *cache_header);
-  virtual ~ESPWebServerStaticNode();
-
-protected:
-  static void _handlerWrapper(httpsserver::HTTPRequest *req, httpsserver::HTTPResponse *res);
-  void _handler(httpsserver::HTTPRequest *req, httpsserver::HTTPResponse *res);
-  friend class ESPWebServer;
-  ESPWebServer *_wrapper;
-  FS& _fs;
-  const char *_filePath;
-  const char *_cache_header;
-  bool _isFile;
-};
-
 #endif //ESPWEBSERVER_H