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
[ADDED] Allowed connection type "in process" #5395
Conversation
With the JWT library update, there is now a `jwt.ConnectionTypeInProcess` connection type that can be added to the list of allowed connection types when defining an user. If the list is not empty and does not have this type, an "in process" connection will not be allowed. Conversely, it is now possible to limit a user to be used only for in process connections (or several types based on the content of the list). Possible solution for #5387 Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
@derekcollison This would be a possible solution to issue #5387. If you think this is reasonable, I will make the JWT PR, have it merged, and then update this PR with the new JWT lib dep and open this for formal review. |
Will take a look. |
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.
LGTM - Let's proceed.
Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
@derekcollison I have updated the PR with the go.mod change to point to newly released JWT library that adds the "in process" connection type and updated the test to cover JWT test. It is ready for review now. |
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.
LGTM
With the JWT library update, there is now a
jwt.ConnectionTypeInProcess
connection type that can be added to the list of allowed connection types when defining an user. If the list is not empty and does not have this type, an "in process" connection will not be allowed.Conversely, it is now possible to limit a user to be used only for in process connections (or several types based on the content of the list).
Possible solution for #5387
Signed-off-by: Ivan Kozlovic ivan@synadia.com