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

Support for NGS #63

Open
jirenius opened this issue Jan 27, 2019 · 4 comments
Open

Support for NGS #63

jirenius opened this issue Jan 27, 2019 · 4 comments
Labels
enhancement New feature or request

Comments

@jirenius
Copy link
Collaborator

Issue

Add support to connect Resgate to NGS.

@jirenius jirenius added the enhancement New feature or request label Jan 27, 2019
@autodidaddict
Copy link

It looks like there's a solution to this but this branch hasn't been merged to master. I was about to start doing some experimentation with resgate and noticed that it didn't seem to support NATS 2.0 authentication (a requirement for my production scenario).

Is this blocked on something @jirenius ?

@ColinSullivan1
Copy link

Perhaps it was blocked on the official release of NATS 2.0- NGS uses the same mechanism. The branch looks ready...

BTW, not required, but an example unit test can be found here.

@jirenius
Copy link
Collaborator Author

jirenius commented Aug 6, 2019

This branch wasn't merge because it didn't fully support NGS. One feature was still missing: the ability to know if a message might have been lost within a super-cluster (eg. through something like a super-cluster node disconnect event).

But since support for authentication credentials is a stand-alone feature, I will have it merged to develop right away.

@autodidaddict , I can mail you new binaries (which architecture?) with authentication support if needed, or post them for you on https://forum.resgate.io .

Thanks @ColinSullivan1 ! That helps when creating the unit tests for it.

@jirenius
Copy link
Collaborator Author

jirenius commented Aug 6, 2019

Created separate issue #108 for the credentials, and pulled it to develop in #109

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants