mike-jumper commented on a change in pull request #392: GUACAMOLE-774: Add in
MD4 support for MSCHAPv1/2
URL: https://github.com/apache/guacamole-client/pull/392#discussion_r278351277
##########
File path:
extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/conf/RadiusAuthenticationProtocol.java
##########
@@ -0,0 +1,64 @@
+/*
+ * To change this license header, choose License Headers in Project Properties.
+ * To change this template file, choose Tools | Templates
+ * and open the template in the editor.
+ */
+package org.apache.guacamole.auth.radius.conf;
+
+/**
+ * This enum represents supported RADIUS authentication protocols for
+ * the guacamole-auth-radius extension.
+ */
+public enum RadiusAuthenticationProtocol {
+
+ // Password authentication protocol
+ PAP("pap"),
+
+ // Challenge-Handshake AUthentication Protocol
+ CHAP("chap"),
+
+ // Microsoft CHAP version 1
+ MSCHAPv1("mschapv1"),
+
+ // Microsoft CHAP version 2
+ MSCHAPv2("mschapv2"),
+
+ // Extensible authentication protocol with MD5 hashing.
+ EAP_MD5("eap-md5"),
+
+ // Extensible authentication protocol with TLS
+ EAP_TLS("eap-tls"),
+
+ // Extensible authentication protocol with Tunneled TLS
+ EAP_TTLS("eap-ttls");
+
+ // Store the string value used in the configuration file.
Review comment:
JavaDoc.
Besides being the value used in `guacamole.properties`, it looks like we're
intentionally using the value used by JRadius (and relying on this later on).
That may a detail be worth documenting.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
