You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Justin Edelson (JIRA)" <ji...@apache.org> on 2010/03/17 15:58:27 UTC

[jira] Created: (SLING-1447) jcr resource events are only created for the default workspace

jcr resource events are only created for the default workspace
--------------------------------------------------------------

                 Key: SLING-1447
                 URL: https://issues.apache.org/jira/browse/SLING-1447
             Project: Sling
          Issue Type: Improvement
            Reporter: Justin Edelson
            Assignee: Justin Edelson
             Fix For: JCR Resource 2.0.8


via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SLING-1447) support for resource paths containing workspace name

Posted by "Carsten Ziegeler (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Carsten Ziegeler updated SLING-1447:
------------------------------------

    Fix Version/s: API 2.1.0
      Component/s: API

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-1447) support for resource paths containing workspace name

Posted by "Carsten Ziegeler (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12869894#action_12869894 ] 

Carsten Ziegeler commented on SLING-1447:
-----------------------------------------

I've removed the workspace authentication info property in revision 946896 (as discussed on the mailing list). Instead of specifying the workspace
during authentication it is now specified during resolution by either using the [workspace]:[path_within_workspace]  notation or setting a request attribute.

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-1447) support for resource paths containing workspace name

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12872242#action_12872242 ] 

Justin Edelson commented on SLING-1447:
---------------------------------------

required changes to the post servlet

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>         Attachments: SLING-1447_post.patch
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SLING-1447) support for resource paths containing workspace name

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Justin Edelson updated SLING-1447:
----------------------------------

        Summary: support for resource paths containing workspace name  (was: jcr resource events are only created for the default workspace)
    Component/s: JCR

changing name to reflect increased scope.

As discussed on the mailing list (http://markmail.org/thread/2xpbrnvfqmcwggyi), the initial implementation was flawed in that it added a new event property containing the workspace. Instead, the resource path should contain the workspace name as:

[workspace]:[path_within_workspace]

In order to do that, this type of resource path must be resolveable and resources returned by the ResourceResolver interface methods must return the correct path values.

Each JcrResourceResolver is associated with a single primary session. This is either the default session or one specified using the user.jcr.workspace auth info property. In the case where a ResourceResolver gets a request for a resource path in a different workspace, it should ask the factory for a new resolver and save that resolver, closing it later. 


> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (SLING-1447) support for resource paths containing workspace name

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12872245#action_12872245 ] 

Justin Edelson commented on SLING-1447:
---------------------------------------

patch to handle the adaptTo(Session.class) business better. also need to change the query method to use adaptTo(Session.class) so the right session is used for querying.

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>         Attachments: SLING-1447_adapttosession.patch, SLING-1447_post.patch
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-1447) support for resource paths containing workspace name

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12874058#action_12874058 ] 

Justin Edelson commented on SLING-1447:
---------------------------------------

applying applyTo() patch in r950103

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>         Attachments: SLING-1447_adapttosession.patch, SLING-1447_post.patch
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-1447) support for resource paths containing workspace name

Posted by "Carsten Ziegeler (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12872306#action_12872306 ] 

Carsten Ziegeler commented on SLING-1447:
-----------------------------------------

The adaptTo patch looks good to me; I haven't looked at the post servlet patch

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>         Attachments: SLING-1447_adapttosession.patch, SLING-1447_post.patch
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Reopened: (SLING-1447) jcr resource events are only created for the default workspace

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Justin Edelson reopened SLING-1447:
-----------------------------------


> jcr resource events are only created for the default workspace
> --------------------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (SLING-1447) support for resource paths containing workspace name

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Justin Edelson updated SLING-1447:
----------------------------------

    Attachment: SLING-1447_adapttosession.patch

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>         Attachments: SLING-1447_adapttosession.patch, SLING-1447_post.patch
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-1447) support for resource paths containing workspace name

Posted by "Carsten Ziegeler (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12869897#action_12869897 ] 

Carsten Ziegeler commented on SLING-1447:
-----------------------------------------

With revision 946900, if a user is not allowed to login into a different workspace than the default and tries to access a resource in such a workspace,
this is treated as a resource not found.

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (SLING-1447) jcr resource events are only created for the default workspace

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Justin Edelson resolved SLING-1447.
-----------------------------------

    Resolution: Fixed

> jcr resource events are only created for the default workspace
> --------------------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-1447) support for resource paths containing workspace name

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12874057#action_12874057 ] 

Justin Edelson commented on SLING-1447:
---------------------------------------

applied patch for post servlets in r950102

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>         Attachments: SLING-1447_adapttosession.patch, SLING-1447_post.patch
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-1447) support for resource paths containing workspace name

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12886511#action_12886511 ] 

Justin Edelson commented on SLING-1447:
---------------------------------------

I'm calling this resolved. Been using this for about a month and it is complete for me.

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>         Attachments: SLING-1447_adapttosession.patch, SLING-1447_post.patch
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-1447) jcr resource events are only created for the default workspace

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12856545#action_12856545 ] 

Justin Edelson commented on SLING-1447:
---------------------------------------

reopening per http://markmail.org/thread/2xpbrnvfqmcwggyi

> jcr resource events are only created for the default workspace
> --------------------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Resolved: (SLING-1447) support for resource paths containing workspace name

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Justin Edelson resolved SLING-1447.
-----------------------------------

    Resolution: Fixed

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>         Attachments: SLING-1447_adapttosession.patch, SLING-1447_post.patch
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SLING-1447) jcr resource events are only created for the default workspace

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12847700#action_12847700 ] 

Justin Edelson commented on SLING-1447:
---------------------------------------

done in r925531.

if resource.resolver.listener.workspaces isn't defined, the default workspace is observed.
otherwise, resource.resolver.listener.workspaces is a list of workspace names to observe. if any of those names are "*", all workspaces are observed.

in the latter case, the event will have a property called workspace containing the workspace name

> jcr resource events are only created for the default workspace
> --------------------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SLING-1447) support for resource paths containing workspace name

Posted by "Justin Edelson (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SLING-1447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Justin Edelson updated SLING-1447:
----------------------------------

    Attachment: SLING-1447_post.patch

> support for resource paths containing workspace name
> ----------------------------------------------------
>
>                 Key: SLING-1447
>                 URL: https://issues.apache.org/jira/browse/SLING-1447
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, JCR
>            Reporter: Justin Edelson
>            Assignee: Justin Edelson
>             Fix For: JCR Resource 2.0.8, API 2.1.0
>
>         Attachments: SLING-1447_post.patch
>
>
> via configuration, it'd be nice to be able to specify multiple workspaces.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.