[jira] [Commented] (SLING-6419) Remove loginAdministrative() usage from jcr.resource - JcrSystemUserValidator
[ https://issues.apache.org/jira/browse/SLING-6419?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16255412#comment-16255412 ] ASF GitHub Bot commented on SLING-6419: --- karlpauls closed pull request #1: SLING-6419: Switch JcrSystemUserValidator to a service user URL: https://github.com/apache/sling-org-apache-sling-jcr-resource/pull/1 This is an automated message from the Apache Git Service. To respond to the message, please log on 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 > Remove loginAdministrative() usage from jcr.resource - JcrSystemUserValidator > - > > Key: SLING-6419 > URL: https://issues.apache.org/jira/browse/SLING-6419 > Project: Sling > Issue Type: Improvement > Components: JCR >Reporter: Antonio Sanso >Assignee: Karl Pauls > Fix For: JCR Resource 3.0.8 > > > Remove getAdministrativeResourceResolver() and loginAdministrative() usage > from jcr.resource -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (SLING-6419) Remove loginAdministrative() usage from jcr.resource - JcrSystemUserValidator
[ https://issues.apache.org/jira/browse/SLING-6419?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16252072#comment-16252072 ] Carsten Ziegeler commented on SLING-6419: - [~karlpauls] Yeah, why not > Remove loginAdministrative() usage from jcr.resource - JcrSystemUserValidator > - > > Key: SLING-6419 > URL: https://issues.apache.org/jira/browse/SLING-6419 > Project: Sling > Issue Type: Improvement > Components: JCR >Reporter: Antonio Sanso >Assignee: Karl Pauls > Fix For: JCR Resource 3.0.8 > > > Remove getAdministrativeResourceResolver() and loginAdministrative() usage > from jcr.resource -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (SLING-6419) Remove loginAdministrative() usage from jcr.resource - JcrSystemUserValidator
[ https://issues.apache.org/jira/browse/SLING-6419?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16251673#comment-16251673 ] Karl Pauls commented on SLING-6419: --- [~cziegeler], how about shortcutting the recursion using a threadlocal like in the above PR? I tested it and it seems to work like it should... > Remove loginAdministrative() usage from jcr.resource - JcrSystemUserValidator > - > > Key: SLING-6419 > URL: https://issues.apache.org/jira/browse/SLING-6419 > Project: Sling > Issue Type: Improvement > Components: JCR >Reporter: Antonio Sanso >Assignee: Karl Pauls > Fix For: JCR Resource 3.0.8 > > > Remove getAdministrativeResourceResolver() and loginAdministrative() usage > from jcr.resource -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (SLING-6419) Remove loginAdministrative() usage from jcr.resource - JcrSystemUserValidator
[ https://issues.apache.org/jira/browse/SLING-6419?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16251669#comment-16251669 ] ASF GitHub Bot commented on SLING-6419: --- karlpauls opened a new pull request #1: SLING-6419: Switch JcrSystemUserValidator to a service user URL: https://github.com/apache/sling-org-apache-sling-jcr-resource/pull/1 preventing endless recursion using a thread local. This is an automated message from the Apache Git Service. To respond to the message, please log on 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 > Remove loginAdministrative() usage from jcr.resource - JcrSystemUserValidator > - > > Key: SLING-6419 > URL: https://issues.apache.org/jira/browse/SLING-6419 > Project: Sling > Issue Type: Improvement > Components: JCR >Reporter: Antonio Sanso >Assignee: Karl Pauls > Fix For: JCR Resource 3.0.8 > > > Remove getAdministrativeResourceResolver() and loginAdministrative() usage > from jcr.resource -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (SLING-6419) Remove loginAdministrative() usage from jcr.resource - JcrSystemUserValidator
[ https://issues.apache.org/jira/browse/SLING-6419?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15761403#comment-15761403 ] Carsten Ziegeler commented on SLING-6419: - As outlined in SLING-6403 switching JCrSystemUserValidator to a service user ends in an endless recursion. Maybe there is a better option > Remove loginAdministrative() usage from jcr.resource - JcrSystemUserValidator > - > > Key: SLING-6419 > URL: https://issues.apache.org/jira/browse/SLING-6419 > Project: Sling > Issue Type: Improvement > Components: JCR >Reporter: Antonio Sanso > > Remove getAdministrativeResourceResolver() and loginAdministrative() usage > from jcr.resource -- This message was sent by Atlassian JIRA (v6.3.4#6332)