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

[FEATURE] Security Support #116

Open
gaoran10 opened this issue Jun 24, 2020 · 7 comments · May be fixed by #897 or #898
Open

[FEATURE] Security Support #116

gaoran10 opened this issue Jun 24, 2020 · 7 comments · May be fixed by #897 or #898
Labels

Comments

@gaoran10
Copy link
Collaborator

gaoran10 commented Jun 24, 2020

We could support security in steps.

@gaoran10 gaoran10 added the type/feature Indicates new functionality label Jun 24, 2020
@gaoran10 gaoran10 changed the title [BUG] Security Support [FEATURE] Security Support Jun 24, 2020
@davidlanouette
Copy link
Contributor

I'll take a look at this one.

@zongtanghu
Copy link

It's yours.Thanks for your contributing.

@davidlanouette davidlanouette removed their assignment Aug 19, 2020
@davidlanouette
Copy link
Contributor

Hi all.

Due to personal issues, I'm not going to be able to get this done in any reasonable time. I'm unassigning myself from it, so someone can pick it up if they would like.

But, I will keep working on it, as I can, so if nobody else picks it up, I'll get it done sometime.

@frankjkelly
Copy link

Can someone give it little background about what's missing here e.g. amqp+ssl support or other AuthN/AuthZ/Encryption? Thanks! CC: @gaoran10 @zongtanghu

@sijie
Copy link
Member

sijie commented Sep 2, 2020

SSL and AuthN/AuthZ support.

@gaoran10
Copy link
Collaborator Author

gaoran10 commented Sep 11, 2020

@frankjkelly Hi, currently, the AoP is lacking in any form of security support.

In the method receiveConnectionStartOk and receiveConnectionSecureOk of the class AmqpConnection, we take a walk around for the auth, if we want to make some security process, we could refer to the qpid-broker-j. Thanks.

@frankjkelly
Copy link

Great - thank you for the clarification!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants