[
https://issues.apache.org/jira/browse/GEODE-7956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17120794#comment-17120794
]
ASF GitHub Bot commented on GEODE-7956:
---------------------------------------
mkevo commented on a change in pull request #5159:
URL: https://github.com/apache/geode/pull/5159#discussion_r433077472
##########
File path:
geode-core/src/main/java/org/apache/geode/internal/cache/RegionNameValidation.java
##########
@@ -24,7 +24,7 @@
public class RegionNameValidation {
- private static final Pattern NAME_PATTERN =
Pattern.compile("[aA-zZ0-9-_.]+");
+ private static final Pattern NAME_PATTERN =
Pattern.compile("[a-zA-Z\\[\\]0-9-_.]+");
Review comment:
I didn't find RFC for this, but old pattern include this characters and
you can see it if you add logger for names in test
`startingWithMatchingCharactersAreOk` in RegionNameValidationTest where it
validates all characters which is allowed by this pattern.
From ASCII table A is 65 and z is 122, and this part **A-z** include all
characters between these two values, and this characters(^, `, [, ], and
backslash) are between 91 and 96.
Old pattern **[aA-zZ0-9-_.]+** is the same as **[A-z0-9-_.]+**.\
So we can go with change pattern to **[A-z0-9-_.]+** and be sure that if
someone is using it we don't break them. And also document all charactes which
is allowed.
Do you agree with that?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Correct documentation of legal region names
> -------------------------------------------
>
> Key: GEODE-7956
> URL: https://issues.apache.org/jira/browse/GEODE-7956
> Project: Geode
> Issue Type: Bug
> Components: docs
> Reporter: Darrel Schneider
> Assignee: Mario Kevo
> Priority: Major
>
> According to this:
> https://geode.apache.org/docs/guide/111/basic_config/data_regions/region_naming.html
> region names can only contain alphanumeric, dash, and underscore.
> But the product also supports a dot '.'.
> See RegionNameValidation and RegionNameValidationTest
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
