pprof 0.3.9 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 8426b4eb39b37e5229a8e338e4348e81aa93e671
-  data.tar.gz: ac5988b00d9287a1b8dc491ec9ae7f60ec6119f7
+SHA256:
+  metadata.gz: b8843e85a32a30167805e1aa3cef04d1626b05f7f1d1c130b1558d9e3bfaa809
+  data.tar.gz: 4ef87037d3779a327d32bb87fb73fc3501d94d4406bb9529fc1049775a23f93e
 SHA512:
-  metadata.gz: 9f1e6dcc8770c0198910a060dc5fd00526515a41c1be82e145df7a428b4f1463fecd1738aefc4af550d11ae8d528abc6f4d8751a71721ffbddaa2302b5bb8117
-  data.tar.gz: 644b8df71ed9a7238cd160bc967dcaab72f9a180727ddd9eaacf858c88519cbbde8767792f940a621405c70751324ddc120196d1541a09709869b8ef86c2027f
+  metadata.gz: 19fe84e13c4007da01357055deb1350eb0108fe31dd45466827208d7d59f9a858dedc8450e1f0cc2f7863db567263638371ce97536d66b5db054c021c5caa612
+  data.tar.gz: 065fbe7cd8cc1af06ca4e739ca1e901694cf711039c345cd2bd9086090b80af503b616ce2babeedf0120bb692c8d425472e8ad8b2d645efb4569f42771cd63bc

data/bin/pprof

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require 'pprof'
 require 'optparse'
@@ -7,7 +8,7 @@ require 'optparse'
 # Or a standard match if the string is bare.
 #
 def matcher(string)
-  m = string.match(%r[^/(.*)/(.*)$])
+  m = string.match(%r{^/(.*)/(.*)$})
   if m.nil?
     Regexp.new(Regexp.escape(string))
   else
@@ -19,10 +20,11 @@ end
 
 filters = {}
 options = {}
-list_options = { :mode => :table }
+list_options = { mode: :table }
 
+# rubocop:disable Metrics/BlockLength
 parser = OptionParser.new do |opts|
-  opts.banner = <<-BANNER.gsub(/^ *\|/,'')
+  opts.banner = <<-BANNER.gsub(/^ *\|/, '')
     |Usage:
     |  pprof [print_options] (PATH|UUID)
     |  pprof [list_options] [filters]
@@ -40,125 +42,124 @@ parser = OptionParser.new do |opts|
     |  - To remove all expired certificates, you can use `#{opts.program_name} -e0 | xargs -0 rm`
   BANNER
 
-  opts.separator ""
-  opts.separator "Print options (when file given)"
+  opts.separator ''
+  opts.separator 'Print options (when file given)'
 
-  opts.on("-i", "--info", "Print general info (default)") do
+  opts.on('-i', '--info', 'Print general info (default)') do
     options[:info] = true
   end
-  opts.on("-c", "--certs", "Print certificates") do
+  opts.on('-c', '--certs', 'Print certificates') do
     options[:certs] = true
   end
-  opts.on("-d", "--devices", "Print provisioned devices") do
+  opts.on('-d', '--devices', 'Print provisioned devices') do
     options[:devices] = true
   end
 
-  opts.separator ""
-  opts.separator "List options (when no file given)"
+  opts.separator ''
+  opts.separator 'List options (when no file given)'
 
-  opts.on("-l", "--list", "Print only the UUIDs, one per line (instead of an ASCII table)") do
+  opts.on('-l', '--list', 'Print only the UUIDs, one per line (instead of an ASCII table)') do
     list_options[:mode] = :list
   end
-  opts.on("-p", "--path", "Print only the paths, one per line (instead of an ASCII table)") do
+  opts.on('-p', '--path', 'Print only the paths, one per line (instead of an ASCII table)') do
     list_options[:mode] = :path
   end
-  opts.on("-0", "--print0", "Separate each found entry by \\0, to be used by `xargs -0`") do
+  opts.on('-0', '--print0', 'Separate each found entry by \\0, to be used by `xargs -0`') do
     list_options[:zero] = true
   end
 
-  opts.separator ""
-  opts.separator "Filters (when no file given)"
+  opts.separator ''
+  opts.separator 'Filters (when no file given)'
 
-  opts.on("--name NAME", "Filter by name") do |name|
+  opts.on('--name NAME', 'Filter by name') do |name|
     filters[:name] = matcher(name)
   end
 
-  opts.on("--appid-name APPID", "Filter by App ID Name") do |appid_name|
+  opts.on('--appid-name APPID', 'Filter by App ID Name') do |appid_name|
     filters[:appid_name] = matcher(appid_name)
   end
 
-  opts.on("--appid APPID", "Filter by App ID") do |appid|
+  opts.on('--appid APPID', 'Filter by App ID') do |appid|
     filters[:appid] = matcher(appid)
   end
 
-  opts.on("--uuid UUID", "Filter by UUID") do |uuid|
+  opts.on('--uuid UUID', 'Filter by UUID') do |uuid|
     filters[:uuid] = matcher(uuid)
   end
 
-  opts.on("--team TEAM", "Filter by team name or ID") do |team|
+  opts.on('--team TEAM', 'Filter by team name or ID') do |team|
     filters[:team] = matcher(team)
   end
 
-  opts.on("--[no-]exp", "Only profiles (not) expired") do |flag|
+  opts.on('--[no-]exp', 'Only profiles (not) expired') do |flag|
     filters[:exp] = flag
   end
 
-  opts.on("--[no-]has-devices", "Filter by profiles having (no) provisioned devices") do |d|
+  opts.on('--[no-]has-devices', 'Filter by profiles having (no) provisioned devices') do |d|
     filters[:has_devices] = d
   end
 
-  opts.on("--[no-]all-devices", "Filter by profiles (not) provisioning all devices") do |d|
+  opts.on('--[no-]all-devices', 'Filter by profiles (not) provisioning all devices') do |d|
     filters[:all_devices] = d
   end
 
-  opts.on("--aps [ENV]", "Only profiles having Push entitlements (or a specific aps env)") do |env|
+  opts.on('--aps [ENV]', 'Only profiles having Push entitlements (or a specific aps env)') do |env|
     filters[:aps_env] = env.nil? ? true : matcher(env)
   end
 
+  opts.separator ''
+  opts.separator 'Misc'
 
-  opts.separator ""
-  opts.separator "Misc"
-
-  opts.on_tail("-h", "--help", "Show this message") do
+  opts.on_tail('-h', '--help', 'Show this message') do
     puts opts
     exit
   end
 
-  opts.on_tail("-v", "--version", "Show version") do
+  opts.on_tail('-v', '--version', 'Show version') do
     puts PProf::VERSION
     exit
   end
 end
+# rubocop:enable Metrics/BlockLength
 
 # Parse the options, catching parse errors nicely
 begin
   parser.parse!
-rescue OptionParser::InvalidOption => err
-  puts err
-  puts parser.help()
+rescue OptionParser::InvalidOption => e
+  puts e
+  puts parser.help
   exit 1
 end
 
 # Don't mix filters and options together, that doesn't make sense
 
 unless filters.empty? || ARGV.empty?
-  puts "You should use either filter flags to filter the whole list, or an specific path, not both."
+  puts 'You should use either filter flags to filter the whole list, or an specific path, not both.'
   puts parser # Usage
   exit
 end
 
 unless options.empty? || !ARGV.empty?
-  puts "You should use option flags only when providing an specific path."
+  puts 'You should use option flags only when providing an specific path.'
   puts parser # Usage
   exit
 end
 
 # Call the appropriate action
 
+o = PProf::OutputFormatter.new
 if ARGV.empty?
   # Print list of matching profiles
-  o = PProf::OutputFormatter.new
   list_options[:mode] = :path if list_options[:zero] && list_options[:mode] == :table
   o.print_filtered_list(PProf::ProvisioningProfile::DEFAULT_DIR, filters, list_options)
 else
-  # Print info about given profile path/UUID
-  o = PProf::OutputFormatter.new
   begin
+    # Print info about given profile path/UUID
     p = PProf::ProvisioningProfile.new(ARGV[0])
-    
-    options = { :info => true } if options.empty?
+
+    options = { info: true } if options.empty?
     o.print_info(p, options)
-  rescue Exception => e
+  rescue StandardError => e
     o.print_error(e, ARGV[0])
   end
 end

data/lib/pprof/entitlements.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Module for the pprof tool to manipulate Provisioning Profiles
 module PProf
   # Represents the list of entitlements in a Provisioning Profile
@@ -23,7 +25,7 @@ module PProf
     # True if we can attach a debugger to the executable, false if not.
     #
     # @return [Bool]
-    def get_task_allow
+    def get_task_allow # rubocop:disable Naming/AccessorMethodName
       @dict['get-task-allow']
     end
 
@@ -98,9 +100,10 @@ module PProf
     # @param [#to_s] key
     #        The key to check
     #
-    def has_key?(key)
-      @dict.has_key?(key.to_s)
+    def key?(key)
+      @dict.key?(key.to_s)
     end
+    alias has_key? key?
 
     # The list of all entitlement keys, as String
     #

data/lib/pprof/output_formatter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Module for the pprof tool to manipulate Provisioning Profiles
 module PProf
   # A helper tool to pretty-print Provisioning Profile informations
@@ -27,14 +29,16 @@ module PProf
       # @param [String...] cols
       #        The content of each column of the row to add
       def row(*cols)
-        '| ' + cols.zip(@widths).map do |c,w|
+        justified_cols = cols.zip(@widths).map do |c, w|
           (c || '<nil>').to_s.ljust(w)[0...w]
-        end.join(' | ') + ' |'
+        end
+        "| #{justified_cols.join(' | ')} |"
       end
 
       # Add a separator line to the ASCII table
       def separator
-        '+' + @widths.map { |w| '-' * (w+2) }.join('+') + '+' 
+        columns_dashes = @widths.map { |w| '-' * (w + 2) }
+        "+#{columns_dashes.join('+')}+"
       end
     end
 
@@ -57,25 +61,36 @@ module PProf
     #        Decide what to print. Valid keys are :info, :certs and :devices
     #
     def print_info(profile, options = nil)
-      options ||= { :info => true }
+      options ||= { info: true }
       if options[:info]
-        keys = [:name, :uuid, :app_id_name, :app_id_prefix, :creation_date, :expiration_date, :ttl, :team_ids, :team_name]
+        keys = %i[name uuid app_id_name app_id_prefix creation_date expiration_date ttl team_ids
+                  team_name]
         keys.each do |key|
-          @output.puts "- #{key.to_s}: #{profile.send(key.to_sym)}"
+          @output.puts "- #{key}: #{profile.send(key.to_sym)}"
         end
-        @output.puts "- Entitlements:"
-        @output.puts profile.entitlements.to_s.split("\n").map { |line| "   #{line}" }
+        @output.puts '- Entitlements:'
+        @output.puts(profile.entitlements.to_s.split("\n").map { |line| "   #{line}" })
       end
 
-      if options[:info] || options[:certs] 
+      # rubocop:disable Style/GuardClause
+      if options[:info] || options[:certs]
         @output.puts "- #{profile.developer_certificates.count} Developer Certificates"
-        profile.developer_certificates.each { |cert| @output.puts "   - #{cert.subject}" } if options[:certs]
+        if options[:certs]
+          profile.developer_certificates.each do |cert|
+            @output.puts "   - #{cert.subject}"
+            @output.puts "     issuer: #{cert.issuer}"
+            @output.puts "     serial: #{cert.serial}"
+            @output.puts "     expires: #{cert.not_after}"
+          end
+        end
       end
+
       if options[:info] || options[:devices]
         @output.puts "- #{(profile.provisioned_devices || []).count} Provisioned Devices"
         profile.provisioned_devices.each { |udid| @output.puts "   - #{udid}" } if options[:devices]
         @output.puts "- Provision all devices: #{profile.provisions_all_devices.inspect}"
       end
+      # rubocop:enable Style/GuardClause
     end
 
     # Prints the filtered list of Provisioning Profiles
@@ -97,18 +112,18 @@ module PProf
     #        * Valid values for key `:zero` are `true` or `false` to decide if we print `\0` at the end of each output.
     #          Only used by `:list` and `:path` modes
     #
-    def print_filtered_list(dir = PProf::ProvisioningProfile::DEFAULT_DIR, filters = {}, list_options = { :mode => :table })
+    def print_filtered_list(dir = PProf::ProvisioningProfile::DEFAULT_DIR, filters = {}, list_options = { mode: :table })
       filter_func = lambda do |p|
         (filters[:name].nil? || p.name =~ filters[:name]) &&
-        (filters[:appid_name].nil? || p.app_id_name =~ filters[:appid_name]) &&
-        (filters[:appid].nil? || p.entitlements.app_id =~ filters[:appid]) &&
-        (filters[:uuid].nil? || p.uuid =~ filters[:uuid]) &&
-        (filters[:team].nil? || p.team_name =~ filters[:team] || p.team_ids.any? { |id| id =~ filters[:team] }) &&
-        (filters[:exp].nil? || (p.expiration_date < DateTime.now) == filters[:exp]) &&
-        (filters[:has_devices].nil? || !(p.provisioned_devices || []).empty? == filters[:has_devices]) &&
-        (filters[:all_devices].nil? || p.provisions_all_devices == filters[:all_devices]) &&
-        (filters[:aps_env].nil? || match_aps_env(p.entitlements.aps_environment, filters[:aps_env])) &&
-        true
+          (filters[:appid_name].nil? || p.app_id_name =~ filters[:appid_name]) &&
+          (filters[:appid].nil? || p.entitlements.app_id =~ filters[:appid]) &&
+          (filters[:uuid].nil? || p.uuid =~ filters[:uuid]) &&
+          (filters[:team].nil? || p.team_name =~ filters[:team] || p.team_ids.any? { |id| id =~ filters[:team] }) &&
+          (filters[:exp].nil? || (p.expiration_date < DateTime.now) == filters[:exp]) &&
+          (filters[:has_devices].nil? || !(p.provisioned_devices || []).empty? == filters[:has_devices]) &&
+          (filters[:all_devices].nil? || p.provisions_all_devices == filters[:all_devices]) &&
+          (filters[:aps_env].nil? || match_aps_env(p.entitlements.aps_environment, filters[:aps_env])) &&
+          true
       end
 
       case list_options[:mode]
@@ -138,16 +153,17 @@ module PProf
       @output.puts table.row('UUID', 'Name', 'AppID', 'Expiration Date', ' ', 'Team Name')
       @output.puts table.separator
 
-      Dir[dir + '/*.mobileprovision'].each do |file|
+      Dir['*.mobileprovision', base: dir].each do |file_name|
+        file = File.join(dir, file_name)
         begin
           p = PProf::ProvisioningProfile.new(file)
-        
+
           next if block_given? && !yield(p)
 
           state = DateTime.now < p.expiration_date ? "\u{2705}" : "\u{274c}" # 2705=checkmark, 274C=red X
           @output.puts table.row(p.uuid, p.name, p.entitlements.app_id, p.expiration_date.to_time, state, p.team_name)
-        rescue Exception => e
-          errors << { :message => e, :file => file }
+        rescue StandardError => e
+          errors << { message: e, file: file }
         end
         count += 1
       end
@@ -155,9 +171,7 @@ module PProf
       @output.puts table.separator
       @output.puts "#{count} Provisioning Profiles found."
 
-      unless errors.empty?
-        errors.each { |e| print_error(e[:message], e[:file]) }
-      end
+      errors.each { |e| print_error(e[:message], e[:file]) } unless errors.empty?
     end
 
     # Prints the filtered list of UUIDs or Paths only
@@ -165,35 +179,35 @@ module PProf
     # @param [String] dir
     #        The directory containing the mobileprovision files to list.
     #        Defaults to '~/Library/MobileDevice/Provisioning Profiles'
+    # @param [Hash] options
+    #        The options hash typically filled while parsing the command line arguments.
+    #         - :mode: will print the UUIDs if set to `:uuid`, the file path otherwise
+    #         - :zero: will concatenate the entries with `\0` instead of `\n` if set
     #
-    # @yield each provisioning provile for filtering/validation
+    # @yield each provisioning profile for filtering/validation
     #        The block is given ProvisioningProfile object and should
     #        return true to display the row, false to filter it out
     #
-    def print_list(dir = PProf::ProvisioningProfile::DEFAULT_DIR, options)
+    def print_list(dir = PProf::ProvisioningProfile::DEFAULT_DIR, options) # rubocop:disable Style/OptionalArguments
       errors = []
-      Dir[dir + '/*.mobileprovision'].each do |file|
-        begin
-          p = PProf::ProvisioningProfile.new(file)
-          next if block_given? && !yield(p)
-
-          @output.print options[:mode] == :uuid ? p.uuid.chomp : file.chomp
-          @output.print options[:zero] ? "\0" : "\n"
-        rescue Exception => e
-          errors << { :message => e, :file => file }
-        end
-      end
-      unless errors.empty?
-        errors.each { |e| print_error(e[:message], e[:file]) }
+      Dir['*.mobileprovision', base: dir].each do |file_name|
+        file = File.join(dir, file_name)
+        p = PProf::ProvisioningProfile.new(file)
+        next if block_given? && !yield(p)
+
+        @output.print options[:mode] == :uuid ? p.uuid.chomp : file.chomp
+        @output.print options[:zero] ? "\0" : "\n"
+      rescue StandardError => e
+        errors << { message: e, file: file }
       end
+      errors.each { |e| print_error(e[:message], e[:file]) } unless errors.empty?
     end
 
-
-    private
     def self.match_aps_env(actual, expected)
-      return false if actual.nil?      # false if no Push entitlements
-      return true if expected === true # true if Push present but we don't filter on specific env
-      return actual =~ expected        # true if Push present and we filter on specific env
+      return false if actual.nil? # false if no Push entitlements
+      return true if expected == true # true if Push present but we don't filter on specific env
+
+      actual =~ expected        # true if Push present and we filter on specific env
     end
   end
 end

data/lib/pprof/provisioning_profile.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'openssl'
 require 'plist'
 require 'time'
@@ -7,7 +9,7 @@ module PProf
   # Represents the content of a Provisioning Profile file
   class ProvisioningProfile
     # The default location where all the Provisioning Profiles are stored on a Mac
-    DEFAULT_DIR="#{ENV['HOME']}/Library/MobileDevice/Provisioning Profiles"
+    DEFAULT_DIR = "#{ENV['HOME']}/Library/MobileDevice/Provisioning Profiles"
 
     # Create a new ProvisioningProfile object from a file path or UUID
     #
@@ -19,26 +21,26 @@ module PProf
     #        File path or UUID of the ProvisioningProfile
     #
     def initialize(file)
-      if file =~ %r/^[0-9A-F-]*$/i
-        path = "#{PProf::ProvisioningProfile::DEFAULT_DIR}/#{file}.mobileprovision"
-      else
-        path = file
-      end
+      path = if file =~ /^[0-9A-F-]*$/i
+               "#{PProf::ProvisioningProfile::DEFAULT_DIR}/#{file}.mobileprovision"
+             else
+               file
+             end
       xml = nil
       begin
         pkcs7 = OpenSSL::PKCS7.new(File.read(path))
         pkcs7.verify([], OpenSSL::X509::Store.new)
         xml = pkcs7.data
-        raise "Empty PKCS7 payload" if xml.nil? || xml.empty?
-      rescue
+        raise 'Empty PKCS7 payload' if xml.nil? || xml.empty?
+      rescue StandardError
         # Seems like sometimes OpenSSL fails to parse the PKCS7 payload
         # Besides, OpenSSL is deprecated on macOS so might not be up-to-date on all machines
         # So as a fallback, we run the `security` command line.
-        # (We could use `security` everytime, but invoking a command line is generally less 
+        # (We could use `security` everytime, but invoking a command line is generally less
         #  efficient than calling the OpenSSL gem if available, so for now it's just used as fallback)
         xml = `security cms -D -i "#{path}" 2> /dev/null`
       end
-      @plist = Plist::parse_xml(xml)
+      @plist = Plist.parse_xml(xml)
       raise "Unable to parse file #{file}." if @plist.nil?
     end
 
@@ -101,7 +103,7 @@ module PProf
     def team_ids
       @plist['TeamIdentifier']
     end
-    
+
     # The name of the Team associated with this Provisioning Profile
     #
     # @return [String]
@@ -109,7 +111,7 @@ module PProf
       @plist['TeamName']
     end
 
-    # The list of X509 Developer Certifiates associated with this profile
+    # The list of X509 Developer Certificates associated with this profile
     #
     # @return [Array<OpenSSL::X509::Certificate>]
     def developer_certificates
@@ -152,14 +154,14 @@ module PProf
     #
     # @return [String]
     def to_s
-      lines = [:name, :uuid, :app_id_name, :app_id_prefix, :creation_date, :expiration_date, :ttl, :team_ids, :team_name].map do |key|
-        "- #{key.to_s}: #{self.send(key.to_sym)}"
+      lines = %i[name uuid app_id_name app_id_prefix creation_date expiration_date ttl team_ids team_name].map do |key|
+        "- #{key}: #{send(key.to_sym)}"
       end +
-      [
-        "- #{developer_certificates.count} Developer Certificates",
-        "- #{provisioned_devices.count} Provisioned Devices",
-        "- Entitlements:"
-      ] + entitlements.to_hasg.map { |key, value| "   - #{key}: #{value}" }
+              [
+                "- #{developer_certificates.count} Developer Certificates",
+                "- #{provisioned_devices.count} Provisioned Devices",
+                '- Entitlements:'
+              ] + entitlements.to_hash.map { |key, value| "   - #{key}: #{value}" }
       lines.join("\n")
     end
   end

data/lib/pprof/version.rb

@@ -1,4 +1,6 @@
+# frozen_string_literal: true
+
 module PProf
   # Module version
-  VERSION = '0.3.9'
+  VERSION = '0.4.0'
 end

data/lib/pprof.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pprof/version'
 
 require 'pprof/provisioning_profile'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pprof
 version: !ruby/object:Gem::Version
-  version: 0.3.9
+  version: 0.4.0
 platform: ruby
 authors:
 - Olivier Halligon
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-02-23 00:00:00.000000000 Z
+date: 2022-02-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: plist
@@ -42,7 +42,8 @@ files:
 homepage: https://github.com/AliSoftware/pprof
 licenses:
 - MIT
-metadata: {}
+metadata:
+  rubygems_mfa_required: 'true'
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -51,15 +52,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.0.0
+      version: 2.7.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.2
+rubygems_version: 3.2.19
 signing_key: 
 specification_version: 4
 summary: A Provisioning Profiles library