[REVIEW] IGNITE-12344 Remote listener of IgniteMessaging has to run with appropriate SecurityContext

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[REVIEW] IGNITE-12344 Remote listener of IgniteMessaging has to run with appropriate SecurityContext

Denis Garus
Hi, Igniters!


I've prepared the PR[1] for the ticket[2], could somebody do a review?


Thanks.



   1. https://github.com/apache/ignite/pull/7338
   2. https://issues.apache.org/jira/browse/IGNITE-12344
Reply | Threaded
Open this post in threaded view
|

Re: [REVIEW] IGNITE-12344 Remote listener of IgniteMessaging has to run with appropriate SecurityContext

Nikita Amelchev
Denis,

I have pre-reviewed your changes, LGTM.

Could some security maintainer take a look?

вт, 11 февр. 2020 г. в 20:37, Denis Garus <[hidden email]>:

>
> Hi, Igniters!
>
>
> I've prepared the PR[1] for the ticket[2], could somebody do a review?
>
>
> Thanks.
>
>
>
>    1. https://github.com/apache/ignite/pull/7338
>    2. https://issues.apache.org/jira/browse/IGNITE-12344



--
Best wishes,
Amelchev Nikita
Reply | Threaded
Open this post in threaded view
|

Re: [REVIEW] IGNITE-12344 Remote listener of IgniteMessaging has to run with appropriate SecurityContext

Denis Garus
Hello, Alexei!

Thank you for the review!
I`ve fixed your comments, take a look, please.



пн, 17 февр. 2020 г. в 12:53, Nikita Amelchev <[hidden email]>:

> Denis,
>
> I have pre-reviewed your changes, LGTM.
>
> Could some security maintainer take a look?
>
> вт, 11 февр. 2020 г. в 20:37, Denis Garus <[hidden email]>:
> >
> > Hi, Igniters!
> >
> >
> > I've prepared the PR[1] for the ticket[2], could somebody do a review?
> >
> >
> > Thanks.
> >
> >
> >
> >    1. https://github.com/apache/ignite/pull/7338
> >    2. https://issues.apache.org/jira/browse/IGNITE-12344
>
>
>
> --
> Best wishes,
> Amelchev Nikita
>