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
adds the notion of a connection deadline to User #3580
Conversation
This will be used mainly by CustomClientAuthentication implementations to indicate that the user connection should be disconnected at some point in future - like when a certificate or token expires Signed-off-by: R.I.Pienaar <rip@devco.net>
@kozlovic draft PR to discuss what we spoke about, seemed easier than anticipated so I think I must be missing something, let me know your thoughts please? |
@@ -83,6 +84,14 @@ func (u *User) clone() *User { | |||
clone := &User{} | |||
*clone = *u | |||
clone.Permissions = u.Permissions.clone() | |||
|
|||
if len(u.AllowedConnectionTypes) > 0 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dont know if this is needed, just noticed it might not be cloning this map so added it, not related to the PR really
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this is good to do.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is good. Make sure it covers all the cases you want and then you could make it a formal PR against dev branch.
@@ -83,6 +84,14 @@ func (u *User) clone() *User { | |||
clone := &User{} | |||
*clone = *u | |||
clone.Permissions = u.Permissions.clone() | |||
|
|||
if len(u.AllowedConnectionTypes) > 0 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this is good to do.
OK, will finish it tomorrow thanks |
Actually it does what I want right now, so I am happy to merge unless you have anythign else @kozlovic |
This will be used mainly by CustomClientAuthentication implementations to indicate that the user connection should be disconnected at some point in future - like when a certificate or token expires
Signed-off-by: R.I.Pienaar rip@devco.net
/cc @nats-io/core