[GEODE-7938] dev rest api should allow '.' and '/' characters in region keys

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.13.0
• Component/s: rest (dev)
• Assignee: Darrel Schneider
• Reporter: Darrel Schneider
• Created: 01/Apr/20 16:23
• Updated: 10/Sep/20 15:51
• Resolved: 08/Apr/20 15:14

Description

The dev rest api has multiple endpoints that can take region entry keys either as path variables or as query parameters. If the key contains a "/" then it can cause problems with parsing the URL. The '.' character also causes problems.
The operations and endpoints that do not allow '.' and '/' in region keys are:

GET, PUT, and DELETE on /v1/{region}/{keys}
POST on /v1/{region}?key={key}

A solution should be found that does not break backwards compatibility but that also allows these characters to be in the specified region key(s).

Issue Links

• is related to: GEODE-7974 CI Failure: RestAccessControllerTest > getSpecificKeysUsingQueryParamWithNonExistentKey
• links to: GitHub Pull Request #4885
• links to: GitHub Pull Request #4901
• links to: GitHub Pull Request #4929
• links to: GitHub Pull Request #4952