Skip to content
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

use NATS #8118

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

use NATS #8118

wants to merge 387 commits into from

Conversation

williamstein
Copy link
Contributor

@williamstein williamstein commented Jan 13, 2025

This is purely some proofs of concept and likely will never be merged.

This works and is serious.

@williamstein williamstein changed the title NATS.io -- proof of concept tests rewrite to use NATS for communication Jan 28, 2025
@williamstein williamstein changed the title rewrite to use NATS for communication use NATS Jan 28, 2025
- building a robust foundation :-)
…than the very restricted subject segment names
- we use the sha1 hash of the key in nats and keep track of original key
  as part of the value
- pros and cons:
  - pro: a lot of probably bugs and subtle ugly code with dangerous
    conventions doesn't have to get written, and it would basically do
    the same thing
  - con: harder to observer what is going on (e.g., impossible with nats
    cli)...  but we can certainly build tooling
- *not* loving this code, but at least less code duplication
…ccount only dstream via more auth rules

- we don't have an application yet for account only auth streams...
- it passes tests but doesn't seem to work for patches yet...
…eletes properly yet" (this is a bad approach)

This reverts commit b5b1f25.
- maybe i'm the first person ever to use auth callout from typescript.
- this is enough to be able to do figure out anything :-)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant