diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index d4ca584..1ec61f6 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -16,8 +16,6 @@ jobs: strategy: matrix: ruby-version: - - '2.5' - - '2.6' - '2.7' - '3.0' - '3.1' diff --git a/.rubocop.yml b/.rubocop.yml index b239c83..c33a31e 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -3,7 +3,7 @@ require: - rubocop-rspec AllCops: - TargetRubyVersion: '2.5' + TargetRubyVersion: '2.7' NewCops: enable Gemspec/RequireMFA: diff --git a/lib/openvpn-status-web/parser/modern_stateless.rb b/lib/openvpn-status-web/parser/modern_stateless.rb index 6861474..44b7617 100644 --- a/lib/openvpn-status-web/parser/modern_stateless.rb +++ b/lib/openvpn-status-web/parser/modern_stateless.rb @@ -13,10 +13,10 @@ module OpenVPNStatusWeb text.lines.each do |line| parts = line.strip.split(sep) - status.client_list_headers = parts[2..-1] if parts[0] == 'HEADER' && parts[1] == 'CLIENT_LIST' - status.client_list << parse_client(parts[1..-1], status.client_list_headers) if parts[0] == 'CLIENT_LIST' - status.routing_table_headers = parts[2..-1] if parts[0] == 'HEADER' && parts[1] == 'ROUTING_TABLE' - status.routing_table << parse_route(parts[1..-1], status.routing_table_headers) if parts[0] == 'ROUTING_TABLE' + status.client_list_headers = parts[2..] if parts[0] == 'HEADER' && parts[1] == 'CLIENT_LIST' + status.client_list << parse_client(parts[1..], status.client_list_headers) if parts[0] == 'CLIENT_LIST' + status.routing_table_headers = parts[2..] if parts[0] == 'HEADER' && parts[1] == 'ROUTING_TABLE' + status.routing_table << parse_route(parts[1..], status.routing_table_headers) if parts[0] == 'ROUTING_TABLE' status.global_stats << parse_global(parts[1..2]) if parts[0] == 'GLOBAL_STATS' end diff --git a/lib/openvpn-status-web/parser/v1.rb b/lib/openvpn-status-web/parser/v1.rb index f46ed9d..64580a4 100644 --- a/lib/openvpn-status-web/parser/v1.rb +++ b/lib/openvpn-status-web/parser/v1.rb @@ -27,9 +27,9 @@ module OpenVPNStatusWeb status = Status.new status.client_list_headers = ['Common Name', 'Real Address', 'Data Received', 'Data Sent', 'Connected Since'] - status.client_list = client_list[2..-1].map { |client| parse_client(client) } + status.client_list = client_list[2..].map { |client| parse_client(client) } status.routing_table_headers = ['Virtual Address', 'Common Name', 'Real Address', 'Last Ref'] - status.routing_table = routing_table[1..-1].map { |route| parse_route(route) } + status.routing_table = routing_table[1..].map { |route| parse_route(route) } status.global_stats = global_stats.map { |global| parse_global(global) } status end diff --git a/openvpn-status-web.gemspec b/openvpn-status-web.gemspec index cd77a49..f95ca1a 100644 --- a/openvpn-status-web.gemspec +++ b/openvpn-status-web.gemspec @@ -24,7 +24,7 @@ Gem::Specification.new do |s| s.executables = ['openvpn-status-web'] s.extra_rdoc_files = Dir['README.md', 'LICENSE'] - s.required_ruby_version = '>= 2.5' + s.required_ruby_version = '>= 2.7' s.add_runtime_dependency 'metriks' s.add_runtime_dependency 'rack', '~> 2.0'