DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=28140>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=28140

Problems when WebDAV servlet is not the default servlet

           Summary: Problems when WebDAV servlet is not the default servlet
           Product: Slide
           Version: 2.0
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Core
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]


Originally posted in 28079, but the patch that was applied was different from 
what I found.

In the XML returned in the reponse the following occur:

1) Some servlet paths being written out as null
<D:principal-collection-set>
    <D:href>/hab-samplesnull/users</D:href>
    <D:href>/hab-samplesnull</D:href>
    <D:href>/hab-samplesnull/roles</D:href>
</D:principal-collection-set>

2)Some servlet paths being written out with the server URL
<D:inherited>
    <D:href>/hab-sampleshttp%3A//localhost%3A8080/files</D:href>
</D:inherited>
<D:principal>
    <D:href>/hab-sampleshttp%3A//localhost%3A8080/roles/HAB_User</D:href>
</D:principal>

Searching the code I think that the calls to WebdavUtils.getAbsolutePath() in 
AbstractReport, PrincipalMatchReport, PrincipalPropertySearchReport, 
PropertyHelper and PropertyRetrieverImpl should be passing in servlet path not 
the server URL.

Also the nulls look like they're coming from calls to 
WebdavUtils.getAbsolutePath() in PropertyHelper that passing null for the 
servlet path.

I think that all references to serverURl need to be changed to servletPath in 
the above mentioned files, as well as servletPath needs to be passed in to the 
methods which used null.  Then, all calls to any of those methods would need 
to be checked.  Perhaps replacing all methods that currently take contextPath 
and serverURL should be replaced to take the request.  Then at compile time 
you can easily find all the callers of those methods.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to