You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Carsten Ziegeler (JIRA)" <ji...@apache.org> on 2015/12/28 15:25:49 UTC
[jira] [Commented] (SLING-5230) Remove
getAdministrativeResourceResolver() usage from org.apache.sling.event.dea
[ https://issues.apache.org/jira/browse/SLING-5230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15072752#comment-15072752 ]
Carsten Ziegeler commented on SLING-5230:
-----------------------------------------
The following patch can be applied once SLING-5355 is solved.
This bundle needs one service user which is able to read/write/delete for the following paths (default config):
/var/eventing/distribution
### Eclipse Workspace Patch 1.0
#P org.apache.sling.event.dea
Index: src/main/java/org/apache/sling/event/dea/impl/DistributedEventReceiver.java
===================================================================
--- src/main/java/org/apache/sling/event/dea/impl/DistributedEventReceiver.java (Revision 1721876)
+++ src/main/java/org/apache/sling/event/dea/impl/DistributedEventReceiver.java (Arbeitskopie)
@@ -136,7 +136,7 @@
DistributedEventReceiver.this.serviceRegistration = reg;
try {
- writerResolver = resourceResolverFactory.getAdministrativeResourceResolver(null);
+ writerResolver = resourceResolverFactory.getServiceResourceResolver(null);
ResourceUtil.getOrCreateResource(writerResolver,
ownRootPath,
DistributedEventAdminImpl.RESOURCE_TYPE_FOLDER,
@@ -296,7 +296,7 @@
this.logger.debug("Checking for old instance trees for distributed events.");
ResourceResolver resolver = null;
try {
- resolver = this.resourceResolverFactory.getAdministrativeResourceResolver(null);
+ resolver = this.resourceResolverFactory.getServiceResourceResolver(null);
final Resource baseResource = resolver.getResource(this.rootPath);
// sanity check - should never be null
@@ -332,7 +332,7 @@
ResourceResolver resolver = null;
try {
- resolver = this.resourceResolverFactory.getAdministrativeResourceResolver(null);
+ resolver = this.resourceResolverFactory.getServiceResourceResolver(null);
final ResourceHelper.BatchResourceRemover brr = ResourceHelper.getBatchResourceRemover(50);
final Resource baseResource = resolver.getResource(this.ownRootPath);
Index: pom.xml
===================================================================
--- pom.xml (Revision 1721876)
+++ pom.xml (Arbeitskopie)
@@ -108,7 +108,7 @@
<dependency>
<groupId>org.apache.sling</groupId>
<artifactId>org.apache.sling.api</artifactId>
- <version>2.4.0</version>
+ <version>2.5.0</version>
<scope>provided</scope>
</dependency>
<!-- Testing -->
Index: src/main/java/org/apache/sling/event/dea/impl/DistributedEventSender.java
===================================================================
--- src/main/java/org/apache/sling/event/dea/impl/DistributedEventSender.java (Revision 1721876)
+++ src/main/java/org/apache/sling/event/dea/impl/DistributedEventSender.java (Arbeitskopie)
@@ -190,7 +190,7 @@
if ( path != null && path.length() > 0 && this.running ) {
ResourceResolver resolver = null;
try {
- resolver = this.resourceResolverFactory.getAdministrativeResourceResolver(null);
+ resolver = this.resourceResolverFactory.getServiceResourceResolver(null);
final Resource eventResource = resolver.getResource(path);
if (eventResource == null) {
this.logger.warn("runInBackground : resource not found at "+path);
> Remove getAdministrativeResourceResolver() usage from org.apache.sling.event.dea
> --------------------------------------------------------------------------------
>
> Key: SLING-5230
> URL: https://issues.apache.org/jira/browse/SLING-5230
> Project: Sling
> Issue Type: Improvement
> Components: Extensions
> Reporter: Antonio Sanso
>
> Counted 4 occurrences
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)