glukos commented on a change in pull request #7523: IGNITE-12759 Getting a
SecurityContext from GridSecurityProcessor
URL: https://github.com/apache/ignite/pull/7523#discussion_r400358952
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/processors/security/GridSecurityProcessor.java
##########
@@ -77,7 +89,7 @@
public SecuritySubject authenticatedSubject(UUID subjId) throws
IgniteCheckedException;
/**
- * Gets security context.
+ * Gets security context for authenticated nodes and thin clients.
Review comment:
Let's at least extend javadoc of SecurityContext in order to explain its
area of responsibility.
It deserved something more than this:
```
/**
* Security context.
*/
public interface SecurityContext {
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
