Skip to content

Conat -- replacing NATS with socketio + sqlite3 + valkey + nodejs #8346

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 162 commits into
base: master
Choose a base branch
from

Conversation

williamstein
Copy link
Contributor

@williamstein williamstein commented May 16, 2025

Screenshot 2025-05-20 at 9 07 04 AM

- did a bunch of benchmarks and tests, since I'm not going down the
  memory leak nightmare rabbit whole later!
- it was very easy and we might as well, as it suddenly provides a
  very, very powerful feature that we might find useful somewhere
…g and as explained is a bad approach anyways (switch to using conat pub/sub!)
(since node 18 is already end of life and I do all dev with node 20
anyways)
diff --git a/.github/workflows/make-and-test.yml b/.github/workflows/make-and-test.yml
index a59eb60..88d31ea 100644
--- a/.github/workflows/make-and-test.yml
+++ b/.github/workflows/make-and-test.yml
@@ -65,7 +65,8 @@ jobs:
       matrix:
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
         node-version:
-          - "18.17.1"
+          - "18"
+          - "20"
         pg-version:
           - "13.12"
           - "16"
…course

- since it is a DOS vulnerability, and we don't need it anyways.
- need to limit total memory usage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant