Re: svn commit: r988161 - in /sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/helper/jcr: JcrNodeResource.java JcrPropertyResource.java

On Mon, Aug 30, 2010 at 8:34 AM, Carsten Ziegeler cziege...@apache.org wrote: ...basically the rule was a non static logger for services and components and static loggers for data objects. Of course we can revisit this and use non static loggers - but whatever we do, we should be consistent

Re: svn commit: r988161 - in /sling/trunk/bundles/jcr/resource/src/main/java/org/apache/sling/jcr/resource/internal/helper/jcr: JcrNodeResource.java JcrPropertyResource.java

Hi, Hmm, was under the impression that we didn't reach consensus. Anyway, I don't like static fields for stuff, which is used by instances. This sounds incorrect and smells like premature optimization ... Alas, I won't fight now ;-) Regards Felix On 30.08.2010 08:34, Carsten Ziegeler wrote:

[VOTE RESULT] Release Commons ClassLoader 1.2.0 and JCR ClassLoader 3.1.2

The vote to release Apache Sling Commons ClassLoader 1.2.0 Apache Sling JCR ClassLoader 3.1.2 passed successfully with four binding votes from Ian Boston, Carsten Ziegeler, Bertrand Delacretaz, and Felix Meschberger. Thanks for voting, I'll upload the artifacts asap. Regards Carsten -- Carsten

[DISCUSS] WebDAV

Hi, Yesterday I fixed SLING-1672 implementing the synthetic resource provisioning for the listChildren method in the ResourceResolver. This works very nicely with both Clemens Wyss's JQeuery based explorer and my whiteboard Milton-based WebDAV bundle. Thus I wonder, whether we should not replace

Re: [DISCUSS] WebDAV

Hi, On Mon, Aug 30, 2010 at 10:28 AM, Felix Meschberger fmesc...@gmail.com wrote: ...I wonder, whether we should not replace the jcr/webdav bundle with the new Milton based WebDAV bundle, which provides full access to the resource tree, while the jcr/webdav bundle just provides access to the

Re: [DISCUSS] WebDAV

Hi, On 30.08.2010 10:33, Bertrand Delacretaz wrote: Hi, On Mon, Aug 30, 2010 at 10:28 AM, Felix Meschberger fmesc...@gmail.com wrote: ...I wonder, whether we should not replace the jcr/webdav bundle with the new Milton based WebDAV bundle, which provides full access to the resource tree,

Re: [DISCUSS] WebDAV

Felix Meschberger wrote Hi, Yesterday I fixed SLING-1672 implementing the synthetic resource provisioning for the listChildren method in the ResourceResolver. This works very nicely with both Clemens Wyss's JQeuery based explorer and my whiteboard Milton-based WebDAV bundle. Thus I

[RESULT] [VOTE] Apache Sling Engine 2.1.0, Apache Sling Adapter 2.0.6, Apache Sling Settings 1.0.0

Hi all, Time to tally. We have got the following votes: +1 Felix Meschberger*, Carsten Ziegeler*, Ian Boston*, Justin Edelson* 0 - -1 - Thus this vote succeeds with 4 binding +1 votes. No other votes have been cast. Thanks for your votes. I will now continue to publish the artifacts.

[jira] Commented: (SLING-1700) contrib/scripting/velocity embeds velocity, causing classloading issue at runtime

[ https://issues.apache.org/jira/browse/SLING-1700?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12904133#action_12904133 ] Olaf Otto commented on SLING-1700: -- Yes, the patch looks good, however it also needs to be

[jira] Updated: (SLING-1700) contrib/scripting/velocity embeds velocity, causing classloading issue at runtime

[ https://issues.apache.org/jira/browse/SLING-1700?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Olaf Otto updated SLING-1700: - Attachment: patch-v2.txt contrib/scripting/velocity embeds velocity, causing classloading issue at

[jira] Updated: (SLING-1700) contrib/scripting/velocity embeds velocity, causing classloading issue at runtime

[ https://issues.apache.org/jira/browse/SLING-1700?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Olaf Otto updated SLING-1700: - Attachment: SLING-1700-ootto.patch Version 2 of the classloading patch for velocity.

[jira] Updated: (SLING-1700) contrib/scripting/velocity embeds velocity, causing classloading issue at runtime

[ https://issues.apache.org/jira/browse/SLING-1700?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Olaf Otto updated SLING-1700: - Attachment: (was: patch-v2.txt) contrib/scripting/velocity embeds velocity, causing classloading

Howto determine whether the user is authenticated/logged in

what is the appropriate way to determine whether a user is authenticated/logged in? Anything more explicit than !request.resourceResolver.getUserID().equals(anonymous) ? How about on the client side (javascript)? Anything else than anonymous != Sling.getSessionInfo.userID ?

[jira] Closed: (SLING-1637) Support DynamicClassLoader interface

[ https://issues.apache.org/jira/browse/SLING-1637?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler closed SLING-1637. --- Support DynamicClassLoader interface

[jira] Closed: (SLING-1434) Inherit from SecureClassLoader (and not URLClassLoader)

[ https://issues.apache.org/jira/browse/SLING-1434?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler closed SLING-1434. --- Inherit from SecureClassLoader (and not URLClassLoader)

[jira] Closed: (SLING-1480) Don't use Workspace#move, use Session#move

[ https://issues.apache.org/jira/browse/SLING-1480?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler closed SLING-1480. --- Don't use Workspace#move, use Session#move --

Re: Howto determine whether the user is authenticated/logged in

Hi, On 30.08.2010 14:05, Clemens Wyss - MySign AG wrote: what is the appropriate way to determine whether a user is authenticated/logged in? Anything more explicit than !request.resourceResolver.getUserID().equals(anonymous) ? On the server side, the correct thing is to check the value of

[jira] Closed: (SLING-1460) Move SlingSettingsService to new Settings bundle

[ https://issues.apache.org/jira/browse/SLING-1460?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1460. Close after release Move SlingSettingsService to new Settings bundle

[jira] Closed: (SLING-1124) RunMode should only provide a configuration printer for the web console

[ https://issues.apache.org/jira/browse/SLING-1124?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1124. Close after release RunMode should only provide a configuration printer for the web console

[jira] Closed: (SLING-1651) Integrate RunMode module into new Settings Module

[ https://issues.apache.org/jira/browse/SLING-1651?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1651. Close after release Integrate RunMode module into new Settings Module

[jira] Closed: (SLING-1462) Add methods to get the sling home to SlingSettingsService

[ https://issues.apache.org/jira/browse/SLING-1462?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1462. Close after release Add methods to get the sling home to SlingSettingsService

RE: Howto determine whether the user is authenticated/logged in

I also recently thought about extending this servlet by adding another property authType which is fed with the value of the HttpServletRequest.getAuthType() method. you would extend SessionInfoProvider? +1, authType is session related and, as you mentioned, more stable than 'anonymous !=

[jira] Closed: (SLING-1167) getParameters(String name) internally uses getParameterMap() causing OutOfMemoryError on file upload

[ https://issues.apache.org/jira/browse/SLING-1167?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1167. Close after release getParameters(String name) internally uses getParameterMap() causing

[jira] Closed: (SLING-1309) Potential concurrent modification exception during request progress tracker dump

[ https://issues.apache.org/jira/browse/SLING-1309?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1309. Close after release Potential concurrent modification exception during request progress

[jira] Closed: (SLING-1427) RequestLoggerService must not be factory component

[ https://issues.apache.org/jira/browse/SLING-1427?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1427. Close after release RequestLoggerService must not be factory component

[jira] Closed: (SLING-1256) Util.fixEncoding(ParameterMap) should ignore case when comparing encodings

[ https://issues.apache.org/jira/browse/SLING-1256?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1256. Close after release Util.fixEncoding(ParameterMap) should ignore case when comparing

[jira] Closed: (SLING-1456) org.apache.sling.api.include.servlet request attribute is set as well when RequestDispatcher#forward is called which is wrong

[ https://issues.apache.org/jira/browse/SLING-1456?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1456. Close after release org.apache.sling.api.include.servlet request attribute is set as well

[jira] Closed: (SLING-633) RequestDispatcherOptions.setReplaceSelectors() doesn't work

[ https://issues.apache.org/jira/browse/SLING-633?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-633. --- Close after release RequestDispatcherOptions.setReplaceSelectors() doesn't work

[jira] Closed: (SLING-821) Remove dependency to JCR

[ https://issues.apache.org/jira/browse/SLING-821?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-821. --- Close after release Remove dependency to JCR Key:

[jira] Closed: (SLING-1003) Integration of 3rd party servlet filters problematic

[ https://issues.apache.org/jira/browse/SLING-1003?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1003. Close after release Integration of 3rd party servlet filters problematic

[jira] Closed: (SLING-1217) Disabling access log in RequestLogger throws NPE

[ https://issues.apache.org/jira/browse/SLING-1217?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1217. Close after release Disabling access log in RequestLogger throws NPE

[jira] Closed: (SLING-1182) Add support to set resource to which login is requested

[ https://issues.apache.org/jira/browse/SLING-1182?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1182. Close after release Add support to set resource to which login is requested

[jira] Closed: (SLING-1183) Log RepositoryException on debug level if obtaining JCR session fails

[ https://issues.apache.org/jira/browse/SLING-1183?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1183. Close after release Log RepositoryException on debug level if obtaining JCR session fails

[jira] Closed: (SLING-1276) Scope dependencies as provided

[ https://issues.apache.org/jira/browse/SLING-1276?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1276. Close after release Scope dependencies as provided --

[jira] Closed: (SLING-1270) Replace Session.logout from SlingMainServlet

[ https://issues.apache.org/jira/browse/SLING-1270?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1270. Close after release Replace Session.logout from SlingMainServlet

[jira] Closed: (SLING-1340) TRACE is supported and probably should not be by default

[ https://issues.apache.org/jira/browse/SLING-1340?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1340. Close after release TRACE is supported and probably should not be by default

[jira] Closed: (SLING-1461) Move request and resource util to Sling API

[ https://issues.apache.org/jira/browse/SLING-1461?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1461. Close after release Move request and resource util to Sling API

[jira] Closed: (SLING-1465) Deprecate RequestLog

[ https://issues.apache.org/jira/browse/SLING-1465?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1465. Close after release Deprecate RequestLog Key:

[jira] Closed: (SLING-1354) Dependency to web console is not optional

[ https://issues.apache.org/jira/browse/SLING-1354?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1354. Close after release Dependency to web console is not optional

[jira] Closed: (SLING-1576) Provide a Sling request listener independent from a specific HTTP service

[ https://issues.apache.org/jira/browse/SLING-1576?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1576. Close after release Provide a Sling request listener independent from a specific HTTP

[jira] Closed: (SLING-1464) Move EngineConstants.SLING_CURRENT_SERVLET_NAME to SlingConstants

[ https://issues.apache.org/jira/browse/SLING-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1464. Close after release Move EngineConstants.SLING_CURRENT_SERVLET_NAME to SlingConstants

[jira] Closed: (SLING-1487) TypeOverwritingResourceWrapper should return null as resource super type

[ https://issues.apache.org/jira/browse/SLING-1487?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1487. Close after release TypeOverwritingResourceWrapper should return null as resource super type

[jira] Closed: (SLING-1694) Instantiate adapter factories lazy

[ https://issues.apache.org/jira/browse/SLING-1694?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1694. Close after release Instantiate adapter factories lazy --

[jira] Closed: (SLING-1195) Make AdapterManager implementation somehow accessible for junit tests

[ https://issues.apache.org/jira/browse/SLING-1195?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-1195. Close after release Make AdapterManager implementation somehow accessible for junit tests

[jira] Created: (SLING-1706) Add authentication type info to the result of the SessionInfoServlet

Add authentication type info to the result of the SessionInfoServlet Key: SLING-1706 URL: https://issues.apache.org/jira/browse/SLING-1706 Project: Sling Issue Type:

[jira] Resolved: (SLING-1706) Add authentication type info to the result of the SessionInfoServlet

[ https://issues.apache.org/jira/browse/SLING-1706?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger resolved SLING-1706. -- Resolution: Fixed Added support for the authType property of the sessionInfo in Rev.

Build failed in Hudson: sling-trunk-1.6 #544

See https://hudson.apache.org/hudson/job/sling-trunk-1.6/544/changes Changes: [fmeschbe] SLING-1706 Add authType property to the sessionInfo result [cziegeler] Use released versions of classloaders -- [...truncated 274 lines...] [INFO]task-segment:

Re: Howto determine whether the user is authenticated/logged in

Ok, done in trunk of the GET servlets for https://issues.apache.org/jira/browse/SLING-1706 Regards Felix On 30.08.2010 14:49, Clemens Wyss wrote: I also recently thought about extending this servlet by adding another property authType which is fed with the value of the

Build failed in Hudson: sling-trunk-1.5 #852

See https://hudson.apache.org/hudson/job/sling-trunk-1.5/852/changes Changes: [fmeschbe] SLING-1706 Add authType property to the sessionInfo result [cziegeler] Use released versions of classloaders -- [...truncated 342 lines...] T E S T S

[jira] Created: (SLING-1707) add a nodetype configuration printer

add a nodetype configuration printer Key: SLING-1707 URL: https://issues.apache.org/jira/browse/SLING-1707 Project: Sling Issue Type: Improvement Components: JCR Reporter: Justin

[jira] Commented: (SLING-1700) contrib/scripting/velocity embeds velocity, causing classloading issue at runtime

[ https://issues.apache.org/jira/browse/SLING-1700?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12904208#action_12904208 ] Olaf Otto commented on SLING-1700: -- I've successfully tested the patch

Build failed in Hudson: sling-trunk-1.5 #853

See https://hudson.apache.org/hudson/job/sling-trunk-1.5/853/changes Changes: [fmeschbe] Use released version of the settings bundle and use provided scope for SCR annotations [fmeschbe] Use released versions -- [...truncated 339 lines...]

Build failed in Hudson: sling-trunk-1.6 #545

See https://hudson.apache.org/hudson/job/sling-trunk-1.6/545/changes Changes: [fmeschbe] Use released version of the settings bundle and use provided scope for SCR annotations [fmeschbe] Use released versions -- [...truncated 271 lines...] [INFO]

[jira] Created: (SLING-1708) contrib/scripting/velocity is missing suitable resourceloader for the JCR

contrib/scripting/velocity is missing suitable resourceloader for the JCR - Key: SLING-1708 URL: https://issues.apache.org/jira/browse/SLING-1708 Project: Sling Issue

[jira] Reopened: (SLING-1705) Proactively set length of jcr:data property for nt:file node resources

[ https://issues.apache.org/jira/browse/SLING-1705?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Justin Edelson reopened SLING-1705: --- This fails for multi-valued properties, demonstrated in the ReferenceTypeHintTest case.

[jira] Resolved: (SLING-1703) Mixin types aren't modified for sub nodes

[ https://issues.apache.org/jira/browse/SLING-1703?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Justin Edelson resolved SLING-1703. --- Assignee: Justin Edelson Resolution: Fixed Mixin types aren't modified for sub nodes

[jira] Resolved: (SLING-1707) add a nodetype configuration printer

[ https://issues.apache.org/jira/browse/SLING-1707?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Justin Edelson resolved SLING-1707. --- Resolution: Fixed add a nodetype configuration printer

[jira] Commented: (SLING-1707) add a nodetype configuration printer

[ https://issues.apache.org/jira/browse/SLING-1707?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12904233#action_12904233 ] Justin Edelson commented on SLING-1707: --- added in r990832 add a nodetype

[jira] Resolved: (SLING-1705) Proactively set length of jcr:data property for nt:file node resources

[ https://issues.apache.org/jira/browse/SLING-1705?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Justin Edelson resolved SLING-1705. --- Resolution: Fixed added multi-valued check in r990833. it might be interesting to set the

[jira] Commented: (SLING-1703) Mixin types aren't modified for sub nodes

[ https://issues.apache.org/jira/browse/SLING-1703?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12904232#action_12904232 ] Justin Edelson commented on SLING-1703: --- fixed in 990831 Mixin types aren't

Build failed in Hudson: sling-trunk-1.6 #546

See https://hudson.apache.org/hudson/job/sling-trunk-1.6/546/changes Changes: [justin] SLING-1707 - committing correct version of NodeType printer [justin] fixing multi-valued problem with SLING-1705 [justin] SLING-1707 - adding a nodetype configuration printer [justin] SLING-1703 - adding

Build failed in Hudson: sling-trunk-1.5 #854

See https://hudson.apache.org/hudson/job/sling-trunk-1.5/854/changes Changes: [justin] SLING-1707 - committing correct version of NodeType printer [justin] fixing multi-valued problem with SLING-1705 [justin] SLING-1707 - adding a nodetype configuration printer [justin] SLING-1703 - adding

Failed sync of Apache Sling Artifacts

Hi, It looks like a sync of Apache Sling artifacts failed today: org/apache/sling/org.apache.sling.adapter/2.0.6/ org/apache/sling/org.apache.sling.engine/2.1.0/ org/apache/sling/org.apache.sling.settings/1.0.0/ org/apache/sling/org.apache.sling.commons.classloader/1.2.0/

[jira] Resolved: (SLING-1700) contrib/scripting/velocity embeds velocity, causing classloading issue at runtime

[ https://issues.apache.org/jira/browse/SLING-1700?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mike Müller resolved SLING-1700. Resolution: Fixed Commitet a modifed version in r990926. Please let me know if you have any

[jira] Resolved: (SLING-1701) contrib/scripting/velocity VelocityTemplatesScriptEngine has multiple issues

[ https://issues.apache.org/jira/browse/SLING-1701?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mike Müller resolved SLING-1701. Resolution: Duplicate contrib/scripting/velocity VelocityTemplatesScriptEngine has multiple

Why should scripts only dispatch GET requests?

Hi While fixing things on the Velocity Scripting bundle the question arised why the scripting bundles like Velocity Scripting should only accept GET requests. Is there a reason why the scripting engines should not handle other requests? best regards mike

[jira] Updated: (SLING-1702) contrib/scripting/velocity VelocityTemplatesScriptEngineFactory needs workaround in case of engine instantiation errors

[ https://issues.apache.org/jira/browse/SLING-1702?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mike Müller updated SLING-1702: --- Priority: Minor (was: Major) contrib/scripting/velocity VelocityTemplatesScriptEngineFactory needs

Build failed in Hudson: sling-trunk-1.5 #855

See https://hudson.apache.org/hudson/job/sling-trunk-1.5/855/changes Changes: [mykee] SLING-1700 contrib/scripting/velocity embeds velocity, causing classloading issue at runtime -- [...truncated 338 lines...]

Build failed in Hudson: sling-trunk-1.6 #547

See https://hudson.apache.org/hudson/job/sling-trunk-1.6/547/changes Changes: [mykee] SLING-1700 contrib/scripting/velocity embeds velocity, causing classloading issue at runtime -- [...truncated 270 lines...] [INFO] Building Apache Sling Scripting

Build failed in Hudson: sling-contrib-1.5 » Apache Sling Launchpad Contrib Testing #573

See https://hudson.apache.org/hudson/job/sling-contrib-1.5/org.apache.sling$org.apache.sling.launchpad.contrib-testing/573/ -- [...truncated 48 lines...] [INFO] snapshot org.apache.sling:org.apache.sling.jcr.resource:2.0.7-SNAPSHOT: checking for updates

Build failed in Hudson: sling-contrib-1.5 #573

See https://hudson.apache.org/hudson/job/sling-contrib-1.5/573/changes Changes: [mykee] SLING-1700 contrib/scripting/velocity embeds velocity, causing classloading issue at runtime -- [...truncated 3472 lines...] [INFO] Copying bundle from