ovpn-key 0.7.5 → 0.7.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b44e2fdcdb2fc4a47b18a47bfb5ea09f349555a2547614b595c944581178e970
-  data.tar.gz: 97877899e12ab51f09a1e94c29995feb9e53156d42182349dc8cbe8e86143a86
+  metadata.gz: 13681f4cf8c6abc0badce3feceacf4ce489daab4ff9ea7b177bdca7c3fe983ed
+  data.tar.gz: 621ad232db5032b90b1b631f7765dc15aca7b8131465bb2658408c13a7c1d8b0
 SHA512:
-  metadata.gz: 47a2f7e26dd576ec913750834d45e3305466ae186489e99acf4e252acb0dcf81c16d0e0510505cef36c9dbd10bd30ee3fc759a6d4c19557cbad24f2891c5dfd9
-  data.tar.gz: 971a6d5befb69c5e6098969e42df5f2cd1c0e14173c2bab1ded94e40231c091243177dd8a65704cebcd5a7dfafde067c82758e39047517237222e8ba3e3f0a89
+  metadata.gz: cc2d031bd9f8a595fa1efd862c2e5e371643928d34c7d7398db179466b655e48d412dd5494cfc6d705e614114846acbcce717b1117026df685bac5a4eb6e65b7
+  data.tar.gz: d31fd3d8936ab9bbd94daed1db9f2334925e073dfb9319d6d96aabfa1db566bd2f340144dd0190ee27ee2652d64ef70d59e6c0a6a1c63af6392dfdba0191073e

data/README.md

@@ -29,13 +29,14 @@ If you're brave, [let me know](https://github.com/chillum/ovpn-key/issues), wher
 
 1. `ovpn-key --init`
 2. edit `ovpn-key.yml` and `openssl.ini`
-3. `ovpn-key --ca --dh --server --nopass`
-4. `ovpn-key --client somebody [--nopass]`
-5. `ovpn-key --revoke somebody`
-6. `ovpn-key --static` (generates `ta.key`)
-7. add a file with `.ovpn` extension to the directory  
+3. `ovpn-key --ca --dh`
+4. `ovpn-key --server --nopass`
+5. `ovpn-key --client somebody [--nopass]`
+6. `ovpn-key --revoke somebody`
+7. `ovpn-key --static` (generates `ta.key`)
+8. add a file with `.ovpn` extension to the directory  
    it should contain every setting except for `cert` and `key`
-8. `ovpn-key --zip somebody-else [--nopass]`
+9. `ovpn-key --zip somebody-else [--nopass]`
 
 ### Configuration
 

data/bin/ovpn-key

@@ -1,71 +1,73 @@
 #! /usr/bin/env ruby
+# frozen_string_literal: true
+
 require 'optparse'
 require 'fileutils'
 require 'yaml'
 require 'zip'
-require_relative '../lib/version.rb'
-require_relative '../lib/functions.rb'
+require_relative '../lib/version'
+require_relative '../lib/functions'
 
 SSL_CONF = 'openssl.ini'
 APP_CONF = 'ovpn-key.yml'
 
 options = {}
 OptionParser.new do |opts|
-  opts.banner = "Usage: #{File.basename $0} <options> [--nopass]"
-  opts.on("--init [directory]", "Init a CA directory (defaults to current)") do |v|
-    options[:init] = v ? v : "."
+  opts.banner = "Usage: #{File.basename $PROGRAM_NAME} <options> [--nopass]"
+  opts.on('--init [directory]', 'Init a CA directory (defaults to current)') do |v|
+    options[:init] = v || '.'
   end
-  opts.on("--ca", "Generate a CA (ca.crt)") do |v|
+  opts.on('--ca', 'Generate a CA (ca.crt)') do |v|
     check_crt('ca')
     options[:generate_ca] = v
   end
-  opts.on("--dh", "Generate a DH keyfile (dh.pem)") do |v|
+  opts.on('--dh', 'Generate a DH keyfile (dh.pem)') do |v|
     # it's safe to overwrite this file
     options[:generate_dh] = v
   end
-  opts.on("--static", "Generate OpenVPN static key (ta.key)") do |v|
+  opts.on('--static', 'Generate OpenVPN static key (ta.key)') do |v|
     options[:generate_static] = v
-    check_crt("ta")
+    check_crt('ta')
   end
-  opts.on("--server [name]", "Generate a server key (defaults to 'server')") do |v|
-    options[:generate_server] = v ? v : "server"
+  opts.on('--server [name]', "Generate a server key (defaults to 'server')") do |v|
+    options[:generate_server] = v || 'server'
     check_crt(options[:generate_server])
   end
-  opts.on("--client [name]", "Generate a client key and sign it") do |v|
+  opts.on('--client [name]', 'Generate a client key and sign it') do |v|
     check_client(v)
     options[:generate_client] = v
   end
-  opts.on("--zip    [name]", "Ditto plus pack it to ZIP with OpenVPN config") do |v|
+  opts.on('--zip    [name]', 'Ditto plus pack it to ZIP with OpenVPN config') do |v|
     check_client(v)
     options[:generate_zip] = v
   end
-  opts.on("--revoke [name]", "Revoke a certificate (using crl.pem) and delete it") do |v|
-    abort "Please specify what certificate to revoke" unless v
+  opts.on('--revoke [name]', 'Revoke a certificate (using crl.pem) and delete it') do |v|
+    abort 'Please specify what certificate to revoke' unless v
     options[:revoke] = v
   end
-  opts.on("--nopass", "Don't protect .key files with a password") do |v|
+  opts.on('--nopass', "Don't protect .key files with a password") do |v|
     options[:no_password] = v
   end
 end.parse!
-if ARGV.length > 0
-  abort "Error: invalid args: #{ARGV.join ' '}\nSee `#{File.basename $0} -h` for help"
+if ARGV.length.positive?
+  abort "Error: invalid args: #{ARGV.join ' '}\nSee `#{File.basename $PROGRAM_NAME} -h` for help"
 end
 unless options[:init] || options[:generate_ca] || options[:generate_dh] || options[:generate_static] \
   || options[:generate_server] || options[:generate_client] || options[:generate_zip] || options[:revoke]
-  abort "See `#{File.basename $0} -h` for usage"
+  abort "See `#{File.basename $PROGRAM_NAME} -h` for usage"
 end
-if options[:generate_client] and options[:generate_zip]
+if options[:generate_client] && options[:generate_zip]
   # I assume that user likely wants one of them and is confused with usage
-  abort "There can be only one: --client or --zip"
+  abort 'There can be only one: --client or --zip'
 end
-umask = File.umask 0077
+umask = File.umask 0o077
 
 if options[:init]
   unless options[:init] == '.'
     create_dir options[:init]
     Dir.chdir options[:init]
   end
-  ['certs', 'meta'].each {|dir| create_dir dir}
+  %w[certs meta].each {|dir| create_dir dir}
   ['meta/index.txt', 'meta/index.txt.attr', 'meta/serial', SSL_CONF, APP_CONF].each {|file|
     unless File.exist? file
       FileUtils.copy_file(File.expand_path("defaults/#{file}", "#{__dir__}/.."), "./#{file}")
@@ -74,7 +76,7 @@ if options[:init]
   }
 elsif !File.exist? APP_CONF
   begin
-    rc = YAML.load_file(File.expand_path "~/.#{APP_CONF}")
+    rc = YAML.load_file(File.expand_path("~/.#{APP_CONF}"))
   rescue Errno::ENOENT
     # no configuration file in home directory is not an error
   end
@@ -84,7 +86,7 @@ end
 begin
   settings = YAML.load_file(APP_CONF)
 rescue Errno::ENOENT
-  abort "Run `#{File.basename $0} --init` before generating certificates"
+  abort "Run `#{File.basename $PROGRAM_NAME} --init` before generating certificates"
 end
 ZIP_DIR  = settings['zip_dir']  || '~'
 OPENVPN  = settings['openvpn']  || 'openvpn'
@@ -118,9 +120,9 @@ if options[:generate_zip]
   when 1
     ovpn_file = ovpn_files.first
   when 0
-    abort "No .ovpn file in current directory, please add one"
+    abort 'No .ovpn file in current directory, please add one'
   else
-    abort "More than one .ovpn files in current directory, aborting"
+    abort 'More than one .ovpn files in current directory, aborting'
   end
 
   gen_and_sign('client', options[:generate_zip], options[:no_password])
@@ -133,7 +135,7 @@ if options[:generate_zip]
       File.open(ovpn_file).each {|line| f.write line}
       f.write "cert #{options[:generate_zip]}.crt\nkey #{options[:generate_zip]}.key\n"
     }
-    [ 'ca.crt', "#{options[:generate_zip]}.crt", "#{options[:generate_zip]}.key"].each {|i|
+    ['ca.crt', "#{options[:generate_zip]}.crt", "#{options[:generate_zip]}.key"].each {|i|
       zip.add(i, i)
     }
     zip.add('ta.key', 'ta.key') if File.exist? 'ta.key'
@@ -142,5 +144,5 @@ end
 if options[:revoke]
   exe "#{OPENSSL} ca -revoke '#{options[:revoke]}.crt' -config #{SSL_CONF}"
   gen_crl
-  ['crt', 'key'].each {|ext| File.delete "#{options[:revoke]}.#{ext}"}
+  %w[crt key].each {|ext| File.delete "#{options[:revoke]}.#{ext}"}
 end

data/lib/functions.rb

@@ -1,24 +1,26 @@
-def check_crt filename
-  ['key', 'crt'].each {|ext|
+# frozen_string_literal: true
+
+def check_crt(filename)
+  %w[key crt].each {|ext|
     abort "#{filename}.#{ext} already exists, exiting" if File.exist? "#{filename}.#{ext}"
   }
 end
 
-def check_client name
-  abort "Error: client should have an alphanumeric name" unless name
+def check_client(name)
+  abort 'Error: client should have an alphanumeric name' unless name
   check_crt(name)
 end
 
-def exe cmd
+def exe(cmd)
   system(cmd) or abort "error executing: #{cmd}"
 end
 
-def gen_and_sign type, certname, no_password
+def gen_and_sign(type, certname, no_password)
   gen_key(certname, no_password)
   sign_key(type, certname, certname)
 end
 
-def gen_key certname, no_password
+def gen_key(certname, no_password)
   if no_password
     exe "#{OPENSSL} genrsa -out '#{certname}.key' #{KEY_SIZE}"
   else
@@ -26,7 +28,7 @@ def gen_key certname, no_password
   end
 end
 
-def sign_key type, certname, cn
+def sign_key(type, certname, cn)
   if certname == 'ca'
     exe "#{OPENSSL} req -new -x509 -key '#{certname}.key' -out '#{certname}.crt' -config #{SSL_CONF} -subj '/CN=#{cn}#{REQ}' -extensions ext.#{type} -days #{CA_DAYS}"
   else
@@ -40,9 +42,9 @@ def gen_crl
   exe "#{OPENSSL} ca -gencrl -out crl.pem -config #{SSL_CONF}"
 end
 
-def create_dir name
-  unless Dir.exist? name
-    Dir.mkdir name
-    puts "Created directory: #{name}"
-  end
+def create_dir(name)
+  return if Dir.exist? name
+
+  Dir.mkdir name
+  puts "Created directory: #{name}"
 end

data/lib/version.rb

@@ -1 +1,3 @@
-::Version = '0.7.5'
+# frozen_string_literal: true
+
+::VERSION = '0.7.7'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ovpn-key
 version: !ruby/object:Gem::Version
-  version: 0.7.5
+  version: 0.7.7
 platform: ruby
 authors:
 - Vasily Korytov
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-12-26 00:00:00.000000000 Z
+date: 2021-03-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubyzip
@@ -16,17 +16,17 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '2.0'
 description: Generates and revokes certificates, also packs them to ZIP files with
   OpenVPN configuration
-email: vasily.korytov@icloud.com
+email: v.korytov@outlook.com
 executables:
 - ovpn-key
 extensions: []
@@ -46,7 +46,7 @@ homepage: https://github.com/chillum/ovpn-key
 licenses:
 - Apache-2.0
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -54,16 +54,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.0'
+      version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.7
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: Key management utility for OpenVPN
 test_files: []