Workaround: again, set a max limit of 16_384 open files.

trikko · Apr 27, 2024 · a6c711e · a6c711e
1 parent d8d138e
commit a6c711e
Showing 1 changed file with 7 additions and 11 deletions.
diff --git a/source/serverino/main.d b/source/serverino/main.d
@@ -129,19 +129,15 @@ template ServerinoLoop(Modules...)
       // NOTE: Waiting for https://github.com/dlang/phobos/pull/8990
       version(Posix)
       {
-         version(OSX) { /* This won't work on macOS */}
-         else
-         {
-            import core.sys.posix.sys.resource : rlimit, setrlimit, getrlimit, RLIMIT_NOFILE;
-            rlimit rlim;
-            getrlimit(RLIMIT_NOFILE, &rlim);
+         import core.sys.posix.sys.resource : rlimit, setrlimit, getrlimit, RLIMIT_NOFILE;
+         rlimit rlim;
+         getrlimit(RLIMIT_NOFILE, &rlim);
 
-            // 16k seems to be a good value that balances performance and resources
-            // until the issue is fixed
-            rlim.rlim_cur = max(rlim.rlim_max, 16_384);
+         // 16k seems to be a good value that balances performance and resources
+         // until the issue is fixed
+         rlim.rlim_cur = min(rlim.rlim_max, 16_384);
 
-            setrlimit(RLIMIT_NOFILE, &rlim);
-         }
+         setrlimit(RLIMIT_NOFILE, &rlim);
       }
 
       // Enable terminal colors on older windows