Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 354258 - [config] A platform dependent orion.file.allowedPaths needs a correct path separator
Summary: [config] A platform dependent orion.file.allowedPaths needs a correct path se...
Status: RESOLVED WONTFIX
Alias: None
Product: Orion
Classification: ECD
Component: Server (show other bugs)
Version: 0.2   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL: http://dev.eclipse.org/mhonarc/lists/...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-09 10:28 EDT by Gabriel Petrovay CLA
Modified: 2015-05-05 14:49 EDT (History)
1 user (show)

See Also:

Attachments
partial patch (1.02 KB, patch)
2011-08-09 10:28 EDT, Gabriel Petrovay CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Gabriel Petrovay CLA 2011-08-09 10:28:06 EDT 
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.
Comment 1 Tomasz Zarna CLA 2011-08-19 05:12:25 EDT 
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.
Comment 2 John Arthorne CLA 2015-05-05 14:49:13 EDT 
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