aptly_cli 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ceb07d514a0630db6a211ef67c423e70bbfb7366891566fca5eb82633ca45cd2
-  data.tar.gz: 64e5864e9bd063af826abf8aa1f8b4ba65f4288f0bd0bf3ba52a61352b7db34c
+  metadata.gz: c4fb76b5a05ffe081ad5adac38b250f0566dc13b3f6d4b59d69d5cb9b444c7ad
+  data.tar.gz: 7f70ac81c20310b8b5ac91ad50a2f52e84dc9d028713c15184d612d97ddcc8d6
 SHA512:
-  metadata.gz: 348202e54b4fade990163e8cf1a2107f7f538d58ca894f07401c9cc57b3495f450cd032956010bcf5d84472c5b4c4a8895f8d0e69d25aa772258015875d409e2
-  data.tar.gz: 0d11506ac7051aef167bbfa558a22d982a373194bdb1f02dce97e16f6bff15bee4fdd85100a408c65c5e45a9f7d9e0fd0d75662fb599f2dfd8296de8442b8e8b
+  metadata.gz: c251302d85569e13f307f085a3864ecc706f730ff33438c37c530d08b1373eb196bafe1b5ae7f7172d29765131336893fc17ffaa387ab8eb21e2e2f6958322c1
+  data.tar.gz: dba94fca72f1cb48f6ef051a2d9af5123912435352c3057ad06d9394cf3c2637b0c5d496947b6ff6c73e620088b1aaac9c79d50f637d102cc33e3dd116c07d85

data/.github/workflows/publish.yml

@@ -0,0 +1,42 @@
+# This workflow uses actions that are not certified by GitHub.
+# They are provided by a third-party and are governed by
+# separate terms of service, privacy policy, and support
+# documentation.
+
+name: Ruby Gem
+
+on:
+  # Manually publish
+  workflow_dispatch:
+  # Alternatively, publish whenever changes are merged to the `main` branch.
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  build:
+    name: Build + Publish
+    runs-on: ubuntu-latest
+    permissions:
+      packages: write
+      contents: read
+
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Ruby 2.6
+        uses: ruby/setup-ruby@477b21f02be01bcb8030d50f37cfec92bfa615b6
+        with:
+          ruby-version: 2.6
+      - run: bundle install
+
+      - name: Publish to RubyGems
+        run: |
+          mkdir -p $HOME/.gem
+          touch $HOME/.gem/credentials
+          chmod 0600 $HOME/.gem/credentials
+          printf -- "---\n:rubygems_api_key: ${GEM_HOST_API_KEY}\n" > $HOME/.gem/credentials
+          gem build *.gemspec
+          gem push *.gem
+        env:
+          GEM_HOST_API_KEY: "${{secrets.RUBYGEMS_AUTH_TOKEN}}"

data/aptly_cli.gemspec

@@ -26,7 +26,7 @@ Gem::Specification.new do |spec|
   spec.required_ruby_version = '>= 2.0.0'
 
   spec.add_development_dependency "bundler"
-  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "rake", ">= 12.3.3"
   spec.add_development_dependency "minitest"
   spec.add_development_dependency "coveralls"
   spec.add_development_dependency "simplecov"

data/bin/aptly-cli

@@ -115,7 +115,6 @@ command :repo_create do |c|
   c.option '--default_distribution DISTRIBUTION', String, 'Default distribution when publishing from this local repo'
   c.option '--default_component COMPONENT', String, 'Default component when publishing from this local repo'
   c.action do |args, options|
-    puts options.name
     config = AptlyCli::AptlyLoad.new.configure_with($config_file)
     handle_global_options options
     aptly_command = AptlyCli::AptlyRepo.new(config, options)
@@ -166,7 +165,7 @@ command :repo_edit do |c|
     if options.comment
       repo_options = { :Comment => options.comment.to_s }
     end
-    puts aptly_command.repo_edit(options.name.to_s, repo_options)
+    puts aptly_command.repo_edit(options.name.to_s, repo_options || {})
   end
 end
 

data/lib/aptly_cli/version.rb

@@ -1,3 +1,3 @@
 module AptlyCli
-  VERSION = '0.5.0'.freeze
+  VERSION = '0.6.0'.freeze
 end

data/lib/aptly_command.rb

@@ -1,4 +1,8 @@
 module AptlyCli
+  class HttpError < StandardError; end
+  class HttpNotFoundError < HttpError; end
+  class HttpInternalServerError < HttpError; end
+
   class AptlyCommand
     include HTTParty
 
@@ -64,5 +68,41 @@ module AptlyCli
 
       self.class.debug_output @config[:debug] ? $stdout : nil
     end
+
+    def delete(path, options = {})
+      response = self.class.delete(path, options)
+      process_response(response)
+    end
+
+    def get(path, options = {})
+      response = self.class.get(path, options)
+      process_response(response)
+    end
+
+    def post(path, options = {})
+      response = self.class.post(path, options)
+      process_response(response)
+    end
+
+    def put(path, options = {})
+      response = self.class.put(path, options)
+      process_response(response)
+    end
+
+    def process_response(response)
+      json_response = JSON.parse response.body
+
+      raise "[Server] #{json_response['error']}" unless !json_response.is_a?(Hash) || json_response.dig('error').nil?
+
+      raise HttpNotFoundError, "#{json_response}" if response.code == 404
+      raise HttpInternalServerError, "#{json_response}" if response.code == 500
+
+      response
+    rescue JSON::ParserError
+      raise HttpNotFoundError, "#{json_response}" if response.code == 404
+      raise HttpInternalServerError, "#{json_response}" if response.code == 500
+
+      response
+    end
   end
 end

data/lib/aptly_file.rb

@@ -10,7 +10,7 @@ module AptlyCli
 
     def file_dir
       uri = '/files'
-      response = self.class.get uri
+      response = get uri
       response.parsed_response
     end
 
@@ -20,19 +20,19 @@ module AptlyCli
             else
               '/files/' + file_uri
             end
-      response = self.class.get uri
+      response = get uri
       response.parsed_response
     end
 
     def file_delete(file_uri)
       uri = '/files' + file_uri
-      response = self.class.delete uri
+      response = delete uri
       response.parsed_response
     end
 
     def file_post(post_options = {})
       api_file_uri = '/files' + post_options[:file_uri].to_s
-      response = self.class.post(api_file_uri,
+      response = post(api_file_uri,
                       body: {
                         package: post_options[:package],
                         file: File.new(post_options[:local_file])

data/lib/aptly_misc.rb

@@ -11,12 +11,12 @@ module AptlyCli
 
     def get_graph(extension)
       uri = "/graph.#{extension}"
-      self.class.get(uri)
+      get(uri)
     end
 
     def get_version
       uri = '/version'
-      response = self.class.get(uri)
+      response = get(uri)
       response.parsed_response
     end
   end

data/lib/aptly_package.rb

@@ -11,7 +11,7 @@ module AptlyCli
 
     def package_show(package_key)
       uri = "/packages/#{package_key}"
-      self.class.get(uri)
+      get(uri)
     end
   end
 end

data/lib/aptly_publish.rb

@@ -26,12 +26,12 @@ module AptlyCli
 
       uri += "/#{publish_options[:distribution]}"
       uri += '?force=1' if publish_options[:force] == true
-      self.class.delete(uri)
+      delete(uri)
     end
 
     def publish_list
       uri = '/publish'
-      self.class.get(uri)
+      get(uri)
     end
 
     def _parse_snapshots(names)
@@ -92,7 +92,7 @@ module AptlyCli
 
       @body_json = @body.to_json
 
-      self.class.post(uri, :headers => { 'Content-Type' => 'application/json' },
+      post(uri, :headers => { 'Content-Type' => 'application/json' },
                            :body => @body_json)
     end
 
@@ -112,13 +112,13 @@ module AptlyCli
       uri += if publish_options[:prefix]
                "/#{publish_options[:prefix]}"
              else
-               '/'
+               '/:.'
              end
 
       uri += "/#{publish_options[:distribution]}"
 
       @body_json = @body.to_json
-      self.class.put(uri, :headers =>  { 'Content-Type' => 'application/json' },
+      put(uri, :headers =>  { 'Content-Type' => 'application/json' },
                           :body => @body_json)
     end
   end

data/lib/aptly_repo.rb

@@ -14,12 +14,14 @@ module AptlyCli
                                      comment: nil,
                                      DefaultDistribution: nil,
                                      DefaultComponent: nil })
+      raise ArgumentError, 'Repository name is required' if repo_options[:name].nil? || repo_options[:name].empty?
+
       uri = '/repos'
       name = repo_options[:name]
       comment = repo_options[:comment]
       default_distribution = repo_options[:DefaultDistribution]
       default_component = repo_options[:DefaultComponent]
-      self.class.post(uri,
+      post(uri,
                       :body => 
                       { 'Name' => name, 'Comment' => comment,
                         'DefaultDistribution' => default_distribution,
@@ -28,36 +30,32 @@ module AptlyCli
     end
 
     def repo_delete(repo_options = { name: nil, force: nil })
+      raise ArgumentError, 'Repository name is required' if repo_options[:name].nil? || repo_options[:name].empty?
+
       uri = '/repos/' + repo_options[:name]
       uri += '?force=1' if repo_options[:force] == true
-      self.class.delete(uri)
+      delete(uri)
     end
 
-    def repo_edit(name, repo_options = { k => v })
-      repo_option = ''
-      repo_value = ''
-      uri = '/repos/' + name unless name.nil?
-      repo_options.each do |k, v|
-        repo_option = k
-        repo_value = v
-      end
+    def repo_edit(name, repo_options)
+      raise ArgumentError, 'Repository name is required' if name.nil? || name.empty?
 
-      self.class.put(uri, :body => { repo_option => repo_value }.to_json,
+      uri = "/repos/#{name}"
+
+      put(uri, :body => repo_options.to_json,
                           :headers => { 'Content-Type' => 'application/json' })
     end
 
     def repo_list
       uri = '/repos'
-      self.class.get(uri)
+      get(uri)
     end
 
     def repo_package_add(repo_options, packages)
-      if !repo_options.is_a?(Hash) || repo_options[:name].nil?
-        raise ArgumentError.new('Must pass a repository name')
-      end
+      raise ArgumentError, 'Repository name is required' if repo_options[:name].nil? || repo_options[:name].empty?
 
       uri = '/repos/' + repo_options[:name] + '/packages'
-      self.class.post(
+      post(
         uri,
         :body => { PackageRefs: packages }.to_json,
         :headers => { 'Content-Type' => 'application/json' }
@@ -65,12 +63,10 @@ module AptlyCli
     end
 
     def repo_package_delete(repo_options, packages)
-      if !repo_options.is_a?(Hash) || repo_options[:name].nil?
-        raise ArgumentError.new('Must pass a repository name')
-      end
+      raise ArgumentError, 'Repository name is required' if repo_options[:name].nil? || repo_options[:name].empty?
 
       uri = '/repos/' + repo_options[:name] + '/packages'
-      self.class.delete(
+      delete(
         uri,
         :body => { PackageRefs: packages }.to_json,
         :headers => { 'Content-Type' => 'application/json' }
@@ -80,18 +76,16 @@ module AptlyCli
     def repo_package_query(repo_options = { name: nil, query: nil,
                                             with_deps: false,
                                             format: nil })
-      if repo_options[:name].nil?
-        raise ArgumentError.new('Must pass a repository name')
-      else
-        uri = '/repos/' + repo_options[:name] + '/packages'
-      end
+      raise ArgumentError, 'Repository name is required' if repo_options[:name].nil? || repo_options[:name].empty?
+
+      uri = '/repos/' + repo_options[:name] + '/packages'
 
       qs_hash = {}
       qs_hash['q'] = repo_options[:query] if repo_options[:query]
       qs_hash['format'] = repo_options[:format] if repo_options[:format]
       qs_hash['withDeps'] = 1 if repo_options[:with_deps]
       uri += '?' + URI.encode_www_form(qs_hash) if qs_hash
-      self.class.get uri
+      get uri
     end
 
     def repo_show(name)
@@ -100,11 +94,13 @@ module AptlyCli
             else
               '/repos/' + name
             end
-      self.class.get uri
+      get uri
     end
 
     def repo_upload(repo_options = { name: nil, dir: nil, file: nil,
                                      noremove: false, forcereplace: false })
+      raise ArgumentError, 'Repository name is required' if repo_options[:name].nil? || repo_options[:name].empty?
+
       name = repo_options[:name]
       dir  = repo_options[:dir]
       file = repo_options[:file]

data/lib/aptly_snapshot.rb

@@ -12,20 +12,20 @@ module AptlyCli
     def snapshot_delete(name, force=nil)
       uri = "/snapshots/#{name}"
       uri += '?force=1' if force == true
-      self.class.delete(uri)
+      delete(uri)
     end
 
     def snapshot_list(sort=nil)
       uri = '/snapshots'
       uri += "?sort=#{sort}" if sort
-      self.class.get(uri)
+      get(uri)
     end
 
     def snapshot_create(name, repo, description=nil)
       # Build uri to create snapshot, requires name of snap and name of repo
       uri = "/repos/#{repo}/" + 'snapshots'
 
-      self.class.post(uri, :body => 
+      post(uri, :body =>
                       { 'Name' => name,
                         'Description' => description }.to_json,
                            :headers => { 'Content-Type' => 'application/json' })
@@ -35,7 +35,7 @@ module AptlyCli
                             sourcesnapshots=[], packagerefs=[])
       uri = '/snapshots'
       begin
-        self.class.post(uri,
+        post(uri,
                         :body => { 'Name' => name, 'Description' => description,
                                  'SourceSnapshots' => sourcesnapshots,
                                  'PackageRefs' => packagerefs }.to_json,
@@ -47,7 +47,7 @@ module AptlyCli
 
     def snapshot_diff(name, with_snapshot)
       uri = "/snapshots/#{name}/diff/#{with_snapshot}"
-      self.class.get(uri)
+      get(uri)
     end
 
     def snapshot_search(name, search_options={})
@@ -63,12 +63,12 @@ module AptlyCli
       end
 
       @options[:query] = { withDeps:  '1' } if search_options[:withDeps] == true
-      self.class.get(uri, @options)
+      get(uri, @options)
     end
 
     def snapshot_show(name)
       uri = "/snapshots/#{name}"
-      self.class.get(uri)
+      get(uri)
     end
 
     def snapshot_update(name, new_name, description=nil)
@@ -83,7 +83,7 @@ module AptlyCli
       @query[:Description] = description unless description.nil?
       @query_json = @query.to_json
       begin
-        self.class.put(uri, :body => @query_json, :headers =>
+        put(uri, :body => @query_json, :headers =>
                        { 'Content-Type' => 'application/json' })
       rescue HTTParty::Error => e
         puts e

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: aptly_cli
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Zane
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-02-04 00:00:00.000000000 Z
+date: 2022-03-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -28,16 +28,16 @@ dependencies:
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 12.3.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: 12.3.3
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -130,6 +130,7 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/publish.yml"
 - ".gitignore"
 - ".hound.yml"
 - ".rubocop.yml"
@@ -177,7 +178,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.2
+rubygems_version: 3.0.3.1
 signing_key: 
 specification_version: 4
 summary: Command line client to interact with Aptly package management system