Github user jvwing commented on a diff in the pull request:
    --- Diff: 
    @@ -200,15 +201,24 @@
                 .displayName("Server Side Encryption")
                 .description("Specifies the algorithm used for server side 
    -            .allowableValues(NO_SERVER_SIDE_ENCRYPTION, 
    +            .allowableValues(NO_SERVER_SIDE_ENCRYPTION, 
    +    public static final PropertyDescriptor AWS_KMS_KEY = new 
    +            .name("aws-kms-key")
    +            .displayName("Server Side Encryption using KMS")
    --- End diff --
    I recommend the displayName be a bit more direct, this field is the alias 
of a KMS key, and it's going to be used for SSE.

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

Reply via email to