[CMIS-820] AtomPub GetRepositories mishandles repositoryId parameter

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: OpenCMIS 0.11.0
• Fix Version/s: OpenCMIS 0.12.0
• Component/s: opencmis-server
• Assignee: Florian Müller
• Reporter: Ron Gavlin
• Created: 02/Jul/14 09:12
• Updated: 19/Dec/14 14:32
• Resolved: 02/Jul/14 09:19

Description

AtomPub GetRepositories.serve() asserts repositoryId != null and then a few lines later sets repositoryId = getStringParameter(request, Constants.PARAM_REPOSITORY_ID).

This causes problems in the following CmisAtomPubServlet.dispatch() code where the Dispatcher invokes AtomPub GetRepositories.serve() with a null repositoryId:

            if (pathFragments.length < 2) {
                // root -> service document
                dispatcher.dispatch("", METHOD_GET, context, service, null, request, response);
                return;
            }

The simple fix is to remove the repositoryId != null assertion from GetRepositories.serve().

Issue Links

• relates to: CMIS-879 Bad assertion while trying to list repositories with browser binding