Ticket #912 (closed enhancement: fixed)

Opened 3 years ago

Last modified 8 months ago

OpenVPN keys should allow a passphrase to be set on them.

Reported by: JonTheNiceGuy <jon@…> Owned by:
Priority: normal Milestone: Firmware 2.3.7.0
Component: fon-plugin-openvpn Version: 2.3.6.1 (Gari jr.)
Severity: normal
Cc: Hardware: both

Description

Currrently, the OpenVPN configuration files are supplied without a passphrase on the SSL keys. While this is fine for limited implementations, on a larger roll-out, this lack of basic security may compromise the supplied keys.

Sadly, I don't have to-hand the method of automatically defining this passphrase, but it was used successfully on the OpenVPN plugin for Webmin and I have seen it in other OpenVPN deployment scripts.

Attachments

Change History

Changed 3 years ago by matthijs

  • hardware changed from 2.0n (FON2300) to both
  • status changed from new to confirmed
  • severity changed from unknown to normal
  • milestone set to Firmware 2.3

I guess end-users can always set their own passphrases using the openssl tools, but that's hardly user-friendly. Setting a passphrases when generating the client config is probably a good idea.

Changed 3 years ago by JonTheNiceGuy <jon@…>

Here's the code I found from the Webmin module (written by  http://www.openit.it)

"openssl req -days ".$$info{'KEY_EXPIRE'}." -batch -new -keyout ".$dir.".key -out ".$dir.".csr -passout pass:\"".$$info{'KEY_PASSWD'}."\" -config ".$$info{'KEY_CONFIG'}

So, it looks like there's the switch we need: -passout pass:"password"

And, accordingly, at  http://www.openssl.org/docs/apps/openssl.html#PASS_PHRASE_ARGUMENTS is the text we need to know:

We can use

-passout pass:"a password" or -passout file:"/path/to/temp/file/or/fifo/containing/password" or echo "password" | openssl -passout stdin

Changed 8 months ago by matthijs

  • status changed from confirmed to testing-fix
  • milestone changed from Firmware 2.3 to Firmware 2.3.7.0

I added support for this in my local tree just now, I'll push that out to SVN this week.

Changed 8 months ago by matthijs

  • status changed from testing-fix to closed
  • resolution set to fixed

(In [2182]) luci-openvpn: Allow setting a passphrase for new clients.

Closes: #912

Add/Change #912 (OpenVPN keys should allow a passphrase to be set on them.)

Author


E-mail address and user name can be saved in the Preferences.


Action
as closed
 
Note: See TracTickets for help on using tickets.