passwordstate 0.0.3 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1fb957850976d1356b56000ef102e03638ff25ded2fc7d978ec9cd4399a51b4d
-  data.tar.gz: 2dc78e6bd303d713bc5e61e2fdfc608f1d65e826edbe04e7bb17698fdc6e88af
+  metadata.gz: ead3a53cdb04468a85069d55f1e6764562ea518dfb1ccb2e0b640c883ecfc85f
+  data.tar.gz: 4d4dee12f8a862302b80c7fdb2027a0fb98bfe0a7ab45ece68622a6e42a9ed6a
 SHA512:
-  metadata.gz: 73027a2b795d5611b76c5733bc5ec1cbcde1de34cc956e48f3bc52c22f473df5d9ce0dcc54540688ef7481669d61db185711b64ada6f1d04528ff459dbd6c9bb
-  data.tar.gz: 6a9bfb4fd0d9f9579876334c219f0403fdb35a3dbf871bb0a884c9de07f86f7f95cdde9cd1434cdba86b0adbabe8c4613e8a30beab0d49db6ca404d741713f1f
+  metadata.gz: f306e2cf53329fc8f83f6364d02ec940a3df971f1f9b79e9e9866f40084c75e1fd96aeef42bca640f98e7507bb799006603f2007cf3cd01e5004eaeb93695ff9
+  data.tar.gz: 1f744f7a0f2bdd43b536b71de33bd218f1d496b79fcbfc4d0d284afdf9f80b35307c2acc21165e94d692268c22203ba1adb0a470ef76ba246c6292458cd971a2

data/.github/workflows/ruby.yml

@@ -0,0 +1,33 @@
+---
+name: Ruby
+
+on:
+  push:
+    tags: ["*"]
+    branches: ["master"]
+  pull_request:
+    branches: ["master"]
+
+permissions:
+  contents: read
+
+jobs:
+  tests:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby-version: ['2.7', '3.0', '3.1']
+
+    steps:
+      - uses: actions/checkout@v3
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby-version }}
+          bundler-cache: true
+      - name: Install rubocop
+        run: gem install -N rubocop
+      - name: Rubocop
+        run: rubocop lib/
+      - name: Run tests
+        run: bundle exec rake

data/.gitignore

@@ -4,7 +4,8 @@
 /coverage/
 /doc/
 /pkg/
-/spec/reports/
+/test/reports/
 /tmp/
 /vendor/
 Gemfile.lock
+*.gem

data/.gitlab-ci.yml

@@ -1,5 +1,5 @@
 ---
-image: "ruby:2.4"
+image: "ruby:2.7"
 
 # Cache gems in between builds
 cache:
@@ -12,12 +12,15 @@ before_script:
 
 rubocop:
   script:
-    - bundle exec rubocop
+    - bundle exec rubocop lib/ -f p -f ju -o junit.xml
+  artifacts:
+    reports:
+      junit: junit.xml
 
 pages:
-  stage: deploy
+  before_script: []
   script:
-    - gem install yard
+    - gem install yard redcarpet
     - yard doc -o public/
   artifacts:
     paths:
@@ -25,6 +28,6 @@ pages:
   only:
     - master
 
-# rspec:
-#   script:
-#   - rspec spec
+rake:
+  script:
+    - bundle exec rake

data/.rubocop.yml

@@ -1,10 +1,11 @@
 ---
 AllCops:
-  TargetRubyVersion: 2.4
+  TargetRubyVersion: 2.7
   Exclude:
     - '*.spec'
     - 'Rakefile'
     - 'vendor/**/*'
+  NewCops: enable
 
 # Don't enforce documentation
 Style/Documentation:
@@ -22,7 +23,7 @@ Style/SafeNavigation:
 Layout/ClosingHeredocIndentation:
   Enabled: false
 
-Layout/IndentHeredoc:
+Layout/HeredocIndentation:
   Enabled: false
 
 Metrics/PerceivedComplexity:
@@ -34,19 +35,15 @@ Metrics/CyclomaticComplexity:
 Style/RescueModifier:
   Enabled: false
 
+Layout/LineLength:
+  Max: 140
+
 Metrics/MethodLength:
   Max: 40
 
-Metrics/LineLength:
-  Max: 190
-
 Metrics/AbcSize:
   Enabled: false
 
-Performance/FixedSize:
-  Exclude:
-    - 'test/**/*'
-
 Metrics/BlockLength:
   Exclude:
     - 'test/**/*'

data/CHANGELOG.md

@@ -1,3 +1,14 @@
+## v0.1.0 2022-12-05
+
+- Reduced data transferred for regular queries
+- Fixed support for more modern Ruby (3+)
+- Fixed error handling and changes for newer Passwordstate versions
+- Improved pretty printing when debugging
+
+## v0.0.4 2019-10-23
+
+- Fixed a client request issue due to a rubocop change
+
 ## v0.0.3 2019-10-23
 
 - Added method to check if resource types are available

data/README.md

@@ -1,8 +1,8 @@
 # Passwordstate
 
-A ruby gem for communicating with a Passwordstate instance
+A Ruby gem for communicating with a [Passwordstate](https://clickstudios.com.au/passwordstate.aspx) instance
 
-The documentation for the development version can be found at http://iti.gitlab-pages.liu.se/ruby-passwordstate
+The documentation for the development version can be found at https://iti.gitlab-pages.liu.se/ruby-passwordstate
 
 ## Installation
 
@@ -22,22 +22,70 @@ Or install it yourself as:
 
 ## Usage example
 
-```irb
-irb(main):001:0> require 'passwordstate'
-irb(main):002:0> client = Passwordstate::Client.new 'https://passwordstate', username: 'user', password: 'password'
-irb(main):003:0> #        Passwordstate::Client.new 'https://passwordstate', apikey: 'key'
-irb(main):004:0> client.folders
-=> [#<Passwordstate::Resources::Folder:0x000055ed493636e8 @folder_name="Example", @folder_id=2, @tree_path="\\Example">, #<Passwordstate::Resources::Folder:0x000055ed49361fa0 @folder_name="Folder", @folder_id=3, @tree_path="\\Example\\Folder">]
-irb(main):005:0> client.password_lists.get(7).passwords
-=> [#<Passwordstate::Resources::Password:0x0000555fda8acdb8 @title="Webserver1", @user_name="test_web_account", @account_type_id=0, @password="[ REDACTED ]", @allow_export=false, @password_id=2>, #<Passwordstate::Resources::Password:0x0000555fda868640 @title="Webserver2", @user_name="test_web_account2", @account_type_id=0, @password="[ REDACTED ]", @allow_export=false, @password_id=3>, #<Passwordstate::Resources::Password:0x0000555fda84da48 @title="Webserver3", @user_name="test_web_account3", @account_type_id=0, @password="[ REDACTED ]", @allow_export=false, @password_id=4>]
-irb(main):006:0> pw = client.password_lists.first.passwords.create title: 'example', user_name: 'someone', generate_password: true
-=> #<Passwordstate::Resources::Password:0x0000555fdaf9ce98 @title="example", @user_name="someone", @account_type_id=0, @password="[ REDACTED ]", @allow_export=true, @password_id=12, @generate_password=true, @password_list_id=6>
-irb(main):007:0> pw.password
-=> "millionfE2rMrcb2LngBTHnDyxdpsGSmK3"
-irb(main):008:0> pw.delete
-=> true
+```ruby
+require 'passwordstate'
+client = Passwordstate::Client.new 'https://passwordstate.example.com', username: 'user', password: 'password'
+#        Passwordstate::Client.new 'https://passwordstate.example.com', apikey: 'key'
+# #<Passwordstate::Client:0x0000559eb1fabec8
+#  @headers=
+#   {"accept"=>"application/json", "user-agent"=>"RubyPasswordstate/0.1.0"},
+#  @server_url=#<URI::HTTPS https://passwordstate.example.com>,
+#  @timeout=15,
+#  @validate_certificate=true>
+
+client.folders
+# [#<Passwordstate::Resources::Folder:0x000055ed493636e8
+#   @folder_name="Example",
+#   @folder_id=2,
+#   @tree_path="\\Example">,
+#  #<Passwordstate::Resources::Folder:0x000055ed49361fa0
+#   @folder_name="Folder",
+#   @folder_id=3,
+#   @tree_path="\\Example\\Folder">]
+
+client.password_lists.get(7).passwords
+# [#<Passwordstate::Resources::Password:0x0000555fda8acdb8
+#   @title="Webserver1",
+#   @user_name="test_web_account",
+#   @account_type_id=0,
+#   @password="[ REDACTED ]",
+#   @allow_export=false,
+#   @password_id=2>,
+#  #<Passwordstate::Resources::Password:0x0000555fda868640
+#   @title="Webserver2",
+#   @user_name="test_web_account2",
+#   @account_type_id=0,
+#   @password="[ REDACTED ]",
+#   @allow_export=false,
+#   @password_id=3>,
+#  #<Passwordstate::Resources::Password:0x0000555fda84da48
+#   @title="Webserver3",
+#   @user_name="test_web_account3",
+#   @account_type_id=0,
+#   @password="[ REDACTED ]",
+#   @allow_export=false,
+#   @password_id=4>]
+
+pw = client.password_lists.first.passwords.create title: 'example', user_name: 'someone', generate_password: true
+# #<Passwordstate::Resources::Password:0x0000555fdaf9ce98
+#  @title="example",
+#  @user_name="someone",
+#  @account_type_id=0,
+#  @password="[ REDACTED ]",
+#  @allow_export=true,
+#  @password_id=12,
+#  @generate_password=true,
+#  @password_list_id=6>
+
+pw.password
+# "millionfE2rMrcb2LngBTHnDyxdpsGSmK3"
+
+pw.delete
+# true
 ```
 
+A larger - and much more convoluted - example can be found at https://github.com/ananace/foreman_passwordstate/
+
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/ananace/ruby-passwordstate

data/Rakefile

@@ -1,5 +1,5 @@
-require "bundler/gem_tasks"
-require "rake/testtask"
+require 'bundler/gem_tasks'
+require 'rake/testtask'
 
 Rake::TestTask.new(:test) do |t|
   t.libs << "test"

data/lib/passwordstate/client.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 require 'json'
 
 module Passwordstate
   class Client
-    USER_AGENT = "RubyPasswordstate/#{Passwordstate::VERSION}".freeze
+    USER_AGENT = "RubyPasswordstate/#{Passwordstate::VERSION}"
     DEFAULT_HEADERS = {
       'accept' => 'application/json',
       'user-agent' => USER_AGENT
@@ -34,22 +36,31 @@ module Passwordstate
       @http.read_timeout = sec if @http
     end
 
+    def address_book
+      ResourceList.new Passwordstate::Resources::AddressBook,
+                       client: self
+    end
+
     def folders
-      ResourceList.new self, Passwordstate::Resources::Folder,
+      ResourceList.new Passwordstate::Resources::Folder,
+                       client: self,
                        only: %i[all search post]
     end
 
     def hosts
-      ResourceList.new self, Passwordstate::Resources::Host,
+      ResourceList.new Passwordstate::Resources::Host,
+                       client: self,
                        except: %i[search put]
     end
 
     def passwords
-      ResourceList.new self, Passwordstate::Resources::Password
+      ResourceList.new Passwordstate::Resources::Password,
+                       client: self
     end
 
     def password_lists
-      ResourceList.new self, Passwordstate::Resources::PasswordList,
+      ResourceList.new Passwordstate::Resources::PasswordList,
+                       client: self,
                        except: %i[put delete]
     end
 
@@ -65,22 +76,32 @@ module Passwordstate
         html = request(:get, '', allow_html: true)
         version = html.find_line { |line| line.include? '<span>V</span>' }
         version = />(\d\.\d) \(Build (.+)\)</.match(version)
-        "#{version[1]}.#{version[2]}"
+        "#{version[1]}.#{version[2]}" if version
       end
     end
 
     def version?(compare)
+      if version.nil?
+        logger.debug 'Unable to detect Passwordstate version, assuming recent enough.'
+        return true
+      end
+
       Gem::Dependency.new(to_s, compare).match?(to_s, version)
     end
 
     def require_version(compare)
+      if version.nil?
+        logger.debug 'Unable to detect Passwordstate version, assuming recent enough.'
+        return true
+      end
+
       raise "Your version of Passwordstate (#{version}) doesn't support the requested feature" unless version? compare
     end
 
-    def request(method, api_path, options = {})
+    def request(method, api_path, query: nil, reason: nil, **options)
       uri = URI(server_url + "/#{api_type}/" + api_path)
-      uri.query = URI.encode_www_form(options.fetch(:query)) if options.key? :query
-      uri.query = nil unless uri.query&.any?
+      uri.query = URI.encode_www_form(query) unless query.nil?
+      uri.query = nil if uri.query.nil? || uri.query.empty?
 
       req_obj = Net::HTTP.const_get(method.to_s.capitalize.to_sym).new uri
       if options.key? :body
@@ -92,7 +113,7 @@ module Passwordstate
       req_obj.ntlm_auth(auth_data[:username], auth_data[:password]) if api_type == :winapi
       headers.each { |h, v| req_obj[h] = v }
       req_obj['APIKey'] = auth_data[:apikey] if api_type == :api
-      req_obj['Reason'] = options.fetch(:reason) if options.key?(:reason) && version?('>= 8.4.8449')
+      req_obj['Reason'] = reason if !reason.nil? && version?('>= 8.4.8449')
 
       print_http req_obj
       res_obj = http.request req_obj
@@ -104,9 +125,11 @@ module Passwordstate
       if data
         return data if res_obj.is_a? Net::HTTPSuccess
 
-        data = data&.first
+        # data = data.first if data.is_a? Array
+        # parsed = data.fetch('errors', []) if data.is_a?(Hash) && data.key?('errors')
+        parsed = [data].flatten
 
-        raise Passwordstate::HTTPError.new_by_code(res_obj.code, req_obj, res_obj, data&.fetch('errors', []) || [])
+        raise Passwordstate::HTTPError.new_by_code(res_obj.code, req_obj, res_obj, parsed || [])
       else
         return res_obj.body if res_obj.is_a?(Net::HTTPSuccess) && options.fetch(:allow_html, true)
 
@@ -114,10 +137,18 @@ module Passwordstate
       end
     end
 
-    def inspect
-      "#{to_s[0..-2]} #{instance_variables.reject { |k| %i[@auth_data @http @logger].include? k }.map { |k| "#{k}=#{instance_variable_get(k).inspect}" }.join ', '}>"
+    def pretty_print_instance_variables
+      instance_variables.reject { |k| %i[@auth_data @http @logger].include? k }.sort
     end
 
+    def pretty_print(pp)
+      return pp.pp self if respond_to? :mocha_inspect
+
+      pp.pp_object self
+    end
+
+    alias inspect pretty_print_inspect
+
     private
 
     def http
@@ -130,7 +161,7 @@ module Passwordstate
       @http.start
     end
 
-    def print_http(http, truncate = true)
+    def print_http(http, truncate: true)
       return unless logger.debug?
 
       if http.is_a? Net::HTTPRequest
@@ -147,9 +178,17 @@ module Passwordstate
 
       return if http.body.nil?
 
+      body_cleaner = lambda do |obj|
+        obj.each { |k, v| v.replace('[ REDACTED ]') if k.is_a?(String) && %w[password apikey].include?(k.downcase) } if obj.is_a? Hash
+      end
+
       clean_body = JSON.parse(http.body) rescue nil
       if clean_body
-        clean_body = clean_body.each { |k, v| v.replace('[ REDACTED ]') if k.is_a?(String) && %w[password apikey].include?(k.downcase) }.to_json if http.body
+        if clean_body.is_a? Array
+          clean_body.each { |val| body_cleaner.call(val) }
+        else
+          body_cleaner.call(clean_body)
+        end
       else
         clean_body = http.body
       end

data/lib/passwordstate/errors.rb

@@ -1,6 +1,10 @@
+# frozen_string_literal: true
+
 module Passwordstate
   class PasswordstateError < RuntimeError; end
 
+  class NotAcceptableError < PasswordstateError; end
+
   class HTTPError < PasswordstateError
     attr_reader :code, :request, :response, :errors
 
@@ -10,7 +14,8 @@ module Passwordstate
       @response = response
       @errors = errors
 
-      super "Passwordstate responded with an error to the request:\n#{errors.map { |err| err['message'] || err['phrase'] }.join('; ')}"
+      errorstr = errors.map { |err| err['message'] || err['phrase'] || err['error'] }.join('; ')
+      super "Passwordstate responded with an error to the request:\n#{errorstr}"
     end
 
     def self.new_by_code(code, req, res, errors = [])