[
https://issues.apache.org/jira/browse/HADOOP-12962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15212247#comment-15212247
]
Andrew Wang commented on HADOOP-12962:
--------------------------------------
LGTM +1, do you mind setting the target/affects versions too? Will commit
afterwards.
I was slightly worried about special chars because of a previous key name issue
at HADOOP-11311, but some googling didn't turn up anything besides the
HADOOP-11311 case-sensitivity issue. Good to see JCEKS test coverage.
> KMS key names are correctly encoded when creating key
> -----------------------------------------------------
>
> Key: HADOOP-12962
> URL: https://issues.apache.org/jira/browse/HADOOP-12962
> Project: Hadoop Common
> Issue Type: Bug
> Reporter: Xiao Chen
> Assignee: Xiao Chen
> Attachments: HADOOP-12962.01.patch, HADOOP-12962.02.patch
>
>
> Creating a key that contains special character(s) in its name will result in
> failure when creating, while that key is in fact created ok on the underlying
> key provider.
> E.g.
> {noformat}
> $hadoop key create "key name"
> key name has not been created. java.io.IOException: HTTP status [500],
> exception [java.net.URISyntaxException], message [Illegal character in path
> at index 11: /v1/key/key name]
> java.io.IOException: HTTP status [500], exception
> [java.net.URISyntaxException], message [Illegal character in path at index
> 11: /v1/key/key name]
> at
> org.apache.hadoop.util.HttpExceptionUtils.validateResponse(HttpExceptionUtils.java:159)
> at
> org.apache.hadoop.crypto.key.kms.KMSClientProvider.call(KMSClientProvider.java:548)
> at
> org.apache.hadoop.crypto.key.kms.KMSClientProvider.call(KMSClientProvider.java:506)
> at
> org.apache.hadoop.crypto.key.kms.KMSClientProvider.createKeyInternal(KMSClientProvider.java:672)
> at
> org.apache.hadoop.crypto.key.kms.KMSClientProvider.createKey(KMSClientProvider.java:680)
> at
> org.apache.hadoop.crypto.key.KeyShell$CreateCommand.execute(KeyShell.java:483)
> at org.apache.hadoop.crypto.key.KeyShell.run(KeyShell.java:79)
> at org.apache.hadoop.util.ToolRunner.run(ToolRunner.java:70)
> at org.apache.hadoop.crypto.key.KeyShell.main(KeyShell.java:515)
> {noformat}
> but
> {noformat}
> $ hadoop key list
> Listing keys for KeyProvider:
> KMSClientProvider[https://hostname:16000/kms/v1/]
> key name
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)