Base64 encoding with non ASCII characters results in ArrayIndexOutOfBoundException

[jerseyrobot]

To calculate the authentication string in org.glassfish.jersey.client.authentication.BasicAuthenticator the method org.glassfish.jersey.internal.util.Base64.encodeAsString() is used to do the Base64 encoding.
Unfortunately this results in a ArrayIndexOutOfBoundException when the password contains non ASCII characters like § (the paragraph sign) or a German Umlaut.

Some examples:

• Base64.encodeAsString(new byte[]{(byte)128})
• Base64.encodeAsString("§")

[jerseyrobot]

• Issue Imported From: https://github.com/jersey/jersey/issues/3803
• Original Issue Raised By:@mdurawa
• Original Issue Assigned To: Unassigned

[mdurawa]

The easiest way to fix it, would be to replace org.glassfish.jersey.internal.util.Base64 with java.util.Base64. That would affect about 10 files. (I could try to do that, but I'm not familiar with github yet.)

Btw. in the test classes the behavior not to support bytes > 127 seems to be intended. Is there any reason for that?

[srbala]

@mdurawa created a pull request #3829

@mdurawa, I would recommend to deprecate org.glassfish.jersey.internal.util.Base64 not to delete it.

[jansupol]

@srbala Why deprecate? The usual contract is that whatever is internal package is not to be used by users, it can be changed/dropped any release...

[srbala]

In that case, class can be dropped.

[maffe]

This issue can be closed, #3829 was merged, the Base64 class does not exist anymore and a fixed version was released.