Feat/pm resolve

src/env.cc

@@ -858,20 +858,24 @@ Environment::Environment(IsolateData* isolate_data,
       flags_ = flags_ | EnvironmentFlags::kNoCreateInspector;
       permission()->Apply(this, "*", permission::PermissionScope::kInspector);
       if (!options_->allow_child_process) {
-        permission()->Apply(this, "*", permission::PermissionScope::kChildProcess);
+        permission()->Apply(
+            this, "*", permission::PermissionScope::kChildProcess);
       }
       if (!options_->allow_worker_threads) {
-        permission()->Apply(this, "*", permission::PermissionScope::kWorkerThreads);
+        permission()->Apply(
+            this, "*", permission::PermissionScope::kWorkerThreads);
       }
     }
 
     if (!options_->allow_fs_read.empty()) {
-      permission()->Apply(this, options_->allow_fs_read,
+      permission()->Apply(this,
+                          options_->allow_fs_read,
                           permission::PermissionScope::kFileSystemRead);
     }
 
     if (!options_->allow_fs_write.empty()) {
-      permission()->Apply(this, options_->allow_fs_write,
+      permission()->Apply(this,
+                          options_->allow_fs_write,
                           permission::PermissionScope::kFileSystemWrite);
     }
   }

src/permission/child_process_permission.cc

@@ -9,7 +9,8 @@ namespace permission {
 
 // Currently, ChildProcess manage a single state
 // Once denied, it's always denied
-void ChildProcessPermission::Apply(Environment* env, const std::string& allow,
+void ChildProcessPermission::Apply(Environment* env,
+                                   const std::string& allow,
                                    PermissionScope scope) {
   deny_all_ = true;
 }

src/permission/child_process_permission.h

@@ -12,7 +12,9 @@ namespace permission {
 
 class ChildProcessPermission final : public PermissionBase {
  public:
-  void Apply(Environment* env, const std::string& allow, PermissionScope scope) override;
+  void Apply(Environment* env,
+             const std::string& allow,
+             PermissionScope scope) override;
   bool is_granted(PermissionScope perm,
                   const std::string_view& param = "") override;
 

src/permission/fs_permission.cc

@@ -116,7 +116,9 @@ namespace permission {
 
 // allow = '*'
 // allow = '/tmp/,/home/example.js'
-void FSPermission::Apply(Environment* env, const std::string& allow, PermissionScope scope) {
+void FSPermission::Apply(Environment* env,
+                         const std::string& allow,
+                         PermissionScope scope) {
   using std::string_view_literals::operator""sv;
   for (const std::string_view res : SplitString(allow, ","sv)) {
     if (res == "*"sv) {
@@ -129,16 +131,11 @@ void FSPermission::Apply(Environment* env, const std::string& allow, PermissionS
       }
       return;
     }
-    GrantAccess(
-        scope,
-        PathResolve(env, { { res.data(), res.size() } })
-    );
+    GrantAccess(scope, PathResolve(env, {{res.data(), res.size()}}));
   }
 }
 
 void FSPermission::GrantAccess(PermissionScope perm, const std::string& res) {
-  std::cout << "Result: " << res << std::endl;
-
   const std::string path = WildcardIfDir(res);
   if (perm == PermissionScope::kFileSystemRead) {
     granted_in_fs_.Insert(path);

src/permission/fs_permission.h

@@ -15,7 +15,9 @@ namespace permission {
 
 class FSPermission final : public PermissionBase {
  public:
-  void Apply(Environment* env, const std::string& allow, PermissionScope scope) override;
+  void Apply(Environment* env,
+             const std::string& allow,
+             PermissionScope scope) override;
   bool is_granted(PermissionScope perm, const std::string_view& param) override;
 
   struct RadixTree {

src/permission/inspector_permission.cc

@@ -8,7 +8,8 @@ namespace permission {
 
 // Currently, Inspector manage a single state
 // Once denied, it's always denied
-void InspectorPermission::Apply(Environment* env, const std::string& allow,
+void InspectorPermission::Apply(Environment* env,
+                                const std::string& allow,
                                 PermissionScope scope) {
   deny_all_ = true;
 }

src/permission/inspector_permission.h

@@ -12,7 +12,9 @@ namespace permission {
 
 class InspectorPermission final : public PermissionBase {
  public:
-  void Apply(Environment* env, const std::string& allow, PermissionScope scope) override;
+  void Apply(Environment* env,
+             const std::string& allow,
+             PermissionScope scope) override;
   bool is_granted(PermissionScope perm,
                   const std::string_view& param = "") override;
 

src/permission/permission.cc

@@ -130,7 +130,9 @@ void Permission::EnablePermissions() {
   }
 }
 
-void Permission::Apply(Environment* env, const std::string& allow, PermissionScope scope) {
+void Permission::Apply(Environment* env,
+                       const std::string& allow,
+                       PermissionScope scope) {
   auto permission = nodes_.find(scope);
   if (permission != nodes_.end()) {
     permission->second->Apply(env, allow, scope);

src/permission/permission_base.h

@@ -41,7 +41,9 @@ enum class PermissionScope {
 
 class PermissionBase {
  public:
-  virtual void Apply(Environment* env, const std::string& allow, PermissionScope scope) = 0;
+  virtual void Apply(Environment* env,
+                     const std::string& allow,
+                     PermissionScope scope) = 0;
   virtual bool is_granted(PermissionScope perm,
                           const std::string_view& param = "") = 0;
 };

src/permission/worker_permission.cc

@@ -9,7 +9,9 @@ namespace permission {
 
 // Currently, PolicyDenyWorker manage a single state
 // Once denied, it's always denied
-void WorkerPermission::Apply(Environment* env, const std::string& allow, PermissionScope scope) {
+void WorkerPermission::Apply(Environment* env,
+                             const std::string& allow,
+                             PermissionScope scope) {
   deny_all_ = true;
 }
 

src/permission/worker_permission.h

@@ -12,7 +12,9 @@ namespace permission {
 
 class WorkerPermission final : public PermissionBase {
  public:
-  void Apply(Environment* env, const std::string& allow, PermissionScope scope) override;
+  void Apply(Environment* env,
+             const std::string& allow,
+             PermissionScope scope) override;
   bool is_granted(PermissionScope perm,
                   const std::string_view& param = "") override;