Bug 876427

Summary:	CopyTrans REST service returns 404 for docs in sub directories
Product:	[Retired] Zanata	Reporter:	Carlos Munoz <camunoz>
Component:	Component-Logic	Assignee:	Carlos Munoz <camunoz>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Alex Eng <aeng>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	2.0	CC:	aeng, sflaniga, zanata-bugs
Target Milestone:	---
Target Release:	2.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	Zanata version 2.0.3-SNAPSHOT (20121128-1124)	Doc Type:	Bug Fix
Doc Text:	Cause: Incorrect Rest service mappings for copy trans REST endpoint. Consequence: Invocation of copy trans for documents with slashes (/) in their doc ids fails with a 404 Not found error. Fix: Change the mapping to allow for slashes when invoking the copy trans service. Result: Copy trans can now be invoked via REST for documents with slashes in their paths.	Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-02-26 03:46:15 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Carlos Munoz 2012-11-14 04:51:14 UTC

Description of problem:
Copy Trans fails when running from the maven client and the source files are contained in sub directories.


Version-Release number of selected component (if applicable):
2.0.2

How reproducible:
Always

Steps to Reproduce:
1. Push source files that are contained in sub directories with the maven client. Be sure to use the -Dzanata.copyTrans option.
2. Make sure the source push does not fail.
  
Actual results:
Copy Trans fails for the first document and the maven client is aborted.

Expected results:
Copy Trans should complete and continue pushing until it completes all documents.

Comment 1 Carlos Munoz 2012-11-14 23:17:38 UTC

Added changes to allow doc ids with 'slashes' on the copy trans rest service interface.

See server:
https://github.com/zanata/zanata/commit/75200697b93cf5051d6a661802e5f578ef8f0bda
https://github.com/zanata/zanata/commit/a0794cf28d795d2a667706742183bd26584f8f0c

See API:
https://github.com/zanata/zanata-api/commit/d938d950f2b78f25fdbbac772a6e76108b78edb5

Comment 2 Alex Eng 2012-11-28 02:44:00 UTC

Verified in Zanata version 2.0.3-SNAPSHOT (20121128-1124)