[GitHub] rabbah commented on issue #3644: Allow more special characters in EntityName.

2018-05-14 Thread GitBox
rabbah commented on issue #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#issuecomment-388786806 To clarify the set in my comment above was the original PR. It has to exclude ?# to not break other dbs. Maybe only & for now?

[GitHub] rabbah commented on issue #3644: Allow more special characters in EntityName.

2018-05-14 Thread GitBox
rabbah commented on issue #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#issuecomment-388780970 Thanks for weighing in. Maybe we will need to factor the char set into a deployment configuration... The set expansion that was

[GitHub] rabbah commented on issue #3644: Allow more special characters in EntityName.

2018-05-14 Thread GitBox
rabbah commented on issue #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#issuecomment-388780970 Thanks for weighing in. Should we factor the char set into a deployment configuration then? The set proposed first by the pr