separate out cors constants

coding-cpp · Oct 10, 2024 · 6e6ea57 · 6e6ea57
1 parent a752213
commit 6e6ea57
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 27 deletions.
diff --git a/include/expresso/constants/cors.h b/include/expresso/constants/cors.h
@@ -0,0 +1,31 @@
+#pragma once
+
+#include <set>
+#include <string>
+
+namespace expresso {
+
+namespace constants {
+
+namespace cors {
+
+const std::set<std::string> headers = {"accept",
+                                       "access-control-allow-credentials",
+                                       "access-control-allow-headers",
+                                       "access-control-allow-methods",
+                                       "access-control-allow-origin",
+                                       "access-control-expose-headers",
+                                       "access-control-max-age",
+                                       "authorization",
+                                       "content-type",
+                                       "origin",
+                                       "user-agent",
+                                       "x-requested-with"};
+
+const std::string forbidden = "Forbidden";
+
+} // namespace cors
+
+} // namespace constants
+
+} // namespace expresso
diff --git a/include/expresso/middleware/cors.h b/include/expresso/middleware/cors.h
@@ -29,9 +29,6 @@ class Cors : public Middleware {
 
   bool use(expresso::core::Request &req,
            expresso::core::Response &res) override;
-
-  static const std::string FORBIDDEN;
-  static const std::set<std::string> HEADERS;
 };
 
 } // namespace middleware

diff --git a/src/middleware/cors.cpp b/src/middleware/cors.cpp
@@ -1,29 +1,9 @@
+#include <expresso/constants/cors.h>
 #include <expresso/middleware/cors.h>
 
-namespace expresso::middleware {
-
-const std::set<std::string> Cors::HEADERS = {
-    "accept",
-    "access-control-allow-credentials",
-    "access-control-allow-headers",
-    "access-control-allow-methods",
-    "access-control-allow-origin",
-    "access-control-expose-headers",
-    "access-control-max-age",
-    "authorization",
-    "content-type",
-    "origin",
-    "user-agent",
-    "x-requested-with",
-};
-
-const std::string Cors::FORBIDDEN = "Forbidden";
-
-} // namespace expresso::middleware
-
 expresso::middleware::Cors::Cors()
     : credentials(false), allowAllOrigins(false), allowedHeaders("") {
-  for (std::string _header : this->HEADERS) {
+  for (std::string _header : expresso::constants::cors::headers) {
     this->headers.insert(_header);
   }
 
@@ -87,7 +67,7 @@ bool expresso::middleware::Cors::use(expresso::core::Request &req,
   if (requestOrigin == "") {
     res.set("access-control-allow-origin", "null");
     res.status(expresso::enums::STATUS_CODE::FORBIDDEN)
-        .send(expresso::middleware::Cors::FORBIDDEN);
+        .send(expresso::constants::cors::forbidden);
     return false;
   }
 
@@ -103,7 +83,7 @@ bool expresso::middleware::Cors::use(expresso::core::Request &req,
   if (!isOriginPresent) {
     res.set("access-control-allow-origin", "null");
     res.status(expresso::enums::STATUS_CODE::FORBIDDEN)
-        .send(expresso::middleware::Cors::FORBIDDEN);
+        .send(expresso::constants::cors::forbidden);
     return false;
   }