[
https://issues.apache.org/jira/browse/HDFS-15098?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17011296#comment-17011296
]
Wei-Chiu Chuang commented on HDFS-15098:
----------------------------------------
Some quick comments:
1. does it depend on a specific version of openssl? It appears SM4 is a new
thing in openssl and only the latest openssl 1.1.1
(https://www.openssl.org/blog/blog/2018/09/11/release111/) has it. It is not
clear to me if Hadoop support openssl 1.1.1 (HADOOP-16647). What happens if a
lower version of openssl is loaded? does it emit a clear error message?
2. please add documentation
3. please add the new configuration key/value/description in core-default.xml
4. if openssl is not available, will it simply fail? the existing crypto
implementation falls back to a Java implementation if openssl is not loaded.
5. please use OpensslSecureRandom instead of OsSecureRandom. The former has
better performance.
6. add tests please
7. what happens if a client configures SM4 but server side configures AES? does
it fail or is there a negotiation between them to find a common codec??
> Add SM4 encryption method for HDFS
> ----------------------------------
>
> Key: HDFS-15098
> URL: https://issues.apache.org/jira/browse/HDFS-15098
> Project: Hadoop HDFS
> Issue Type: New Feature
> Reporter: liusheng
> Priority: Major
> Attachments: HDFS-15098.001.patch
>
>
> SM4 (formerly SMS4)is a block cipher used in the Chinese National Standard
> for Wireless LAN WAPI (Wired Authentication and Privacy Infrastructure).
> SM4 was a cipher proposed to for the IEEE 802.11i standard, but has so far
> been rejected by ISO. One of the reasons for the rejection has been
> opposition to the WAPI fast-track proposal by the IEEE. please see:
> [https://en.wikipedia.org/wiki/SM4_(cipher)]
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
