Github user necouchman commented on a diff in the pull request:
https://github.com/apache/guacamole-client/pull/242#discussion_r165833339
--- Diff:
extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/AuthenticationProviderService.java
---
@@ -155,9 +157,10 @@ public AuthenticatedUser authenticateUser(Credentials
credentials)
// This is a response to a previous challenge, authenticate with
that.
else {
try {
+ byte[] stateBytes =
javax.xml.bind.DatatypeConverter.parseHexBinary(request.getParameter(RadiusStateField.PARAMETER_NAME));
--- End diff --
Okay, removed the extra verbosity, handled the case where state field is
null, and the `IllegalArgumentException`.
---
