[HDDS-4576] TestDefaultCertificateClient misuses chars param of random()

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.1.0
• Component/s: test
• Labels: pull-request-available
• Target Version/s: 1.1.0
• Assignee: Attila Doroszlai
• Reporter: Attila Doroszlai
• Created: 10/Dec/20 12:52
• Updated: 15/Dec/20 05:34
• Resolved: 14/Dec/20 22:17

Description

TestDefaultCertificateClient passes "UTF-8" to RandomStringUtils.random(int, String) with the intention to use UTF-8 encoding. It was added in HDDS-1087 to fix (intermittent?) failure in CI.

However, the parameter is not for character encoding, rather:

     * @param chars  the String containing the set of characters to use,
     *  may be null, but must not be empty

So this results in values like: "--8F8T8U8T...".

Issue Links

• links to: GitHub Pull Request #1684