Implemented new changes

Shambhavikumari commented 4 years ago

oAuth-token-based-Authentication before subscribing to a message already available in the broker.

pivotal-issuemaster commented 4 years ago

@Shambhavikumari Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

michaelklishin commented 4 years ago

rabbitmq-users thread with some background.

michaelklishin commented 4 years ago

Instead of adding reverts or manual reverts to this PR create a new branch with only the relevant changes, and submit that with any dependencies necessary (this PR references modules that do not exist anywhere) some additional information requested above:

How do other RabbitMQ users benefit from this change?
How does it fit into the existing authN/authZ framework?
if it adds additional authorization checks to a certain operation, why can't that be achieved using rabbitmq_auth_backend_http, a custom HTTP[S]-based service, and rabbitmq_auth_backend_cache to mitigate the overhead? Do we really need to modify this plugin and the MQTT one?
How do we test it?

rabbitmq / rabbitmq-web-mqtt

Implemented new changes #63