dgarus commented on a change in pull request #7137: IGNITE-12282 Access
restriction to the internal package of Ignite
URL: https://github.com/apache/ignite/pull/7137#discussion_r372368543
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/processors/security/IgniteSecurityProcessor.java
##########
@@ -197,21 +201,39 @@ public IgniteSecurityProcessor(GridKernalContext ctx,
GridSecurityProcessor secP
* Updates the package access property to specify the internal Ignite
package.
*/
private void updatePackageAccessProperty() {
- String packAccess = Security.getProperty("package.access");
-
- if (!F.isEmpty(packAccess)) {
- if (!packAccess.contains(IGNITE_INTERNAL_PACKAGE))
- Security.setProperty("package.access", packAccess + ',' +
IGNITE_INTERNAL_PACKAGE);
+ synchronized (SANDBOXED_NODES_COUNTER) {
+ if (SANDBOXED_NODES_COUNTER.incrementAndGet() == 1) {
Review comment:
ok
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
