This is good idea.

In order to upper case toupper() should be used and not manual guessing.

+  else if (streq (p[0], "x509-username-field") && p[1])
+    {
+      VERIFY_PERMISSION (OPT_P_GENERAL);
+      /* Uppercase if necessary */
+      {
+       char *s = p[1];
+       int c, flag = 0;
+
+       while ((c = *s) != '\0')
+          {
+           if (c >= 'a' && c <= 'z')
+              {
+                c = c + 'A' - 'a';
+               flag++;
+              }
+            *s = (char) c;
+            s++;
+         }
+      }
+      options->x509_username_field = p[1];
+    }

2010/6/17 Samuli Seppänen <sam...@openvpn.net>:
>
>> Hi,
>>
>> For my company, we use a PKI (linked to a LDAP) with OpenVPN. We can't
>> use "CN" to be username (few people can have the same "CN"). In our
>> case, we only use the UID.
>>
>> With my patch, you can choose another field to be username with a new
>> option called "x509-username-field", the default value is "CN".
>>
>> Best regards
>>
>> --
>> Emilien Mantel
> Hi Emilien,
>
> Thanks for the patch! Could somebody with better C skills take a look
> and see if it needs modifications?
>
> --
> Samuli Seppänen
> Community Manager
> OpenVPN Technologies, Inc
>
> irc freenode net: mattock
>
>
> ------------------------------------------------------------------------------
> ThinkGeek and WIRED's GeekDad team up for the Ultimate
> GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the
> lucky parental unit.  See the prize list and enter to win:
> http://p.sf.net/sfu/thinkgeek-promo
> _______________________________________________
> Openvpn-devel mailing list
> Openvpn-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/openvpn-devel
>

Reply via email to