chetanmeh commented on issue #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#issuecomment-388734954 This may pose problem with other `ArtifactStore` implementations. For e.g. with CosmosDB following [chars are not allowed in id][1]. These may cause issue when used id is used in URL > The following characters are restricted and cannot be used in the Id property: '/', '\\', '?', '#' So for '/' `CosmosDBArtifactStore` would replace `/` with `|` as that is not a valid char allowed. If we need to open up the chars allowed in id to any char except '/' then other store implementations would need to store id in encoded form (say base64). Also see [here][2] for some discussion in Apache Jackrabbit project around what aspects may influence which are chars are allowed in names like fields. For e.g. most systems would recommend using certain safe chars which can be used in URL across all apps. Some would disallow whitespace chars like new line, carriage return or other chars considered whitespace in UTF-8 (which lead to visible ambiguity in names) [1]: https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.documents.resource.id [2]: https://issues.apache.org/jira/browse/OAK-4857?focusedCommentId=15547113&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15547113
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
