Bugzilla – Full Text Bug Listing
| Summary: | [config] A platform dependent orion.file.allowedPaths needs a correct path separator | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [ECD] Orion | Reporter: | Gabriel Petrovay <gabipetrovay> | ||||
| Component: | Server | Assignee: | Project Inbox <orion.server-inbox> | ||||
| Status: | RESOLVED WONTFIX | QA Contact: | |||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | tomasz.zarna | ||||
| Version: | 0.2 | ||||||
| Target Milestone: | --- | ||||||
| Hardware: | PC | ||||||
| OS: | Windows 7 | ||||||
| URL: | http://dev.eclipse.org/mhonarc/lists/orion-dev/msg00726.html | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
Thanks for the patch Gabriel. Have you thought about this part of John's answer - "Since : is also a valid character in Unix paths, we would also need an escape mechanism to distinguish separators from internal path characters."? This would make the fix more robust. Closing as part of a mass clean up of inactive bugs. Please reopen if this problem still occurs or is relevant to you. For more details see: https://dev.eclipse.org/mhonarc/lists/orion-dev/msg03444.html |
Created attachment 201147 [details] partial patch Hi, The attached simple patch corrects the orion.file.allowedPaths logic to allow platform dependent path separators. As John said on the mailing list, one might also need to allow character escaping on Unix systems.