activemq
activemq copied to clipboard
[AMQ-9244] Add JWT authentication plugin
Generally speaking this looks more like a proof-of-concept (i.e. based on this blog post) rather than a feature which is ready to use. I'm not sure it makes sense to merge it at this point especially with no tests to validate the functionality and to mitigate future regressions.
That's the starting point. The intention is not to merge right now. I wanted to share here as few users want to try. I will work on this PR after the releases plan.
@jbonofre, understood. That wasn't clear from your comment on the Jira where you said,
It doesn't break anything, just add a new plugin, so I don't see problem to merge.
Thanks for the clarification.
I went ahead and converted this to a draft PR. Generally speaking we should be using draft PRs if something is not ready to merge. This way it makes it clear it's still a work in progress. Marking things like this draft PRs in the future should hopefully help with the confusion about the state.