nypl_ruby_util 0.0.7 → 0.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5ff3e8b400c7989c57e1398a642631a8c415a1bf982feadef5e95a813d7ad971
-  data.tar.gz: 7818da3d0d4fc85ddd6f10b259375463bc23a6f17b34a57026d60b4d1d03f057
+  metadata.gz: d2086b8b4b5bef52f3459a1245af09a7364cb35190ec40c0541a7bd72681a8f6
+  data.tar.gz: 4e1c6a34f318b81c6659cca2f57f1930bfbf0e406040e950f681c4631177bec3
 SHA512:
-  metadata.gz: a2a0a99e281a95583fbe84f69fdbd066556f911167527be86dbdea2cffda000d3bb132edb6c90da4846e2b4d8129f7939594ee57b1a646f167955b2d5d1ed324
-  data.tar.gz: 0126f82acca76fd1e6c77afecce13a30c500766fd57f1ebecba075fcf85961c5c4b1c4e01d1641803cb23d86a9adadd21aed2f01d6e174f182b364e30dcc8fc6
+  metadata.gz: 68670e8961ed473bb3854b2320cafc0c960b9418b37d336fdaac6f9cf41ae14acca33d131565986fad25f2d8eb6cb57f225828c93e5539c685bf497b80e6bab6
+  data.tar.gz: 433b9dea26fa844a28a6711eff1d510a8f21df2f3f71e011d76e497f77dd2988714162973345084b681bb068771bc95433957f513e26553243cefbad521c70b3

data/lib/deploy_helper.rb

@@ -1,152 +1,152 @@
-require 'aws-sdk-lambda'
-require 'aws-sdk-cloudwatchevents'
-require 'yaml'
+require "aws-sdk-lambda"
+require "aws-sdk-cloudwatchevents"
+require "yaml"
 
 # Utility class for running rake methods
 class DeployHelper
-    attr_reader(
-        :travis_branch,
-        :aws_access_key_id,
-        :aws_secret_access_key,
-        :aws_configuration,
-        :region,
-        :lambda_client,
-        :yaml,
-        :lambda_config,
-        :function_name,
-        :event
-    )
-
-    def initialize
-        @travis_branch = ENV['TRAVIS_BRANCH'].upcase
-        @travis_branch = ['MAIN', 'MASTER'].include?(@travis_branch) ? 'PRODUCTION' : @travis_branch
-        @aws_access_key_id = ENV["AWS_ACCESS_KEY_ID_#{travis_branch}"]
-        @aws_secret_access_key = ENV["AWS_SECRET_ACCESS_KEY_#{travis_branch}"]
-        @yaml = YAML.safe_load(File.read('.travis.yml'))
-        @lambda_config = yaml['deploy'].find { |conf| name_matches_branch?(conf['function_name'], travis_branch) }
-        @region = @lambda_config['region']
-        @function_name = @lambda_config['function_name']
-        @aws_configuration = {
-            region: region,
-            access_key_id: aws_access_key_id,
-            secret_access_key: aws_secret_access_key
-        }
-        p 'using configuration: ', aws_configuration
-        p 'lambda config: ', lambda_config
-        @lambda_client = Aws::Lambda::Client.new(aws_configuration) if configured?
+  attr_reader(
+    :travis_branch,
+    :aws_access_key_id,
+    :aws_secret_access_key,
+    :aws_configuration,
+    :region,
+    :lambda_client,
+    :yaml,
+    :lambda_config,
+    :function_name,
+    :event
+  )
+
+  def initialize
+    @travis_branch = ENV["TRAVIS_BRANCH"].upcase
+    @travis_branch = ["MAIN", "MASTER"].include?(@travis_branch) ? "PRODUCTION" : @travis_branch
+    @aws_access_key_id = ENV["AWS_ACCESS_KEY_ID_#{travis_branch}"]
+    @aws_secret_access_key = ENV["AWS_SECRET_ACCESS_KEY_#{travis_branch}"]
+    @yaml = YAML.safe_load(File.read(".travis.yml"))
+    @lambda_config = yaml["deploy"].find { |conf| name_matches_branch?(conf["function_name"], travis_branch) }
+    @region = @lambda_config["region"]
+    @function_name = @lambda_config["function_name"]
+    @aws_configuration = {
+      region: region,
+      access_key_id: aws_access_key_id,
+      secret_access_key: aws_secret_access_key
+    }
+    p("using configuration: ", aws_configuration)
+    p("lambda config: ", lambda_config)
+    @lambda_client = Aws::Lambda::Client.new(aws_configuration) if configured?
+  end
+
+  def configured?
+    aws_access_key_id && aws_secret_access_key && region
+  end
+
+  def name_matches_branch?(name, branch)
+    downcase_name = name.downcase
+    downcase_branch = branch.downcase
+    variants = [
+      ["dev", "development"],
+      ["qa"],
+      ["main", "master", "production", "prod"],
+    ]
+    variants.any? do |group|
+      group.any? { |variant| downcase_name.include?(variant) }\
+      && group.any? { |variant| downcase_branch.include?(variant) }
     end
+  end
 
-    def configured?
-        aws_access_key_id && aws_secret_access_key && region
+  def update_lambda_configuration
+    unless configured? && lambda_config
+      p("insufficient configuration")
+      return nil
     end
 
-    def name_matches_branch?(name, branch)
-        downcase_name = name.downcase
-        downcase_branch = branch.downcase
-        variants = [
-            ['dev', 'development'],
-            ['qa'],
-            ['main', 'master', 'production', 'prod'],
-        ]
-        variants.any? do |group|
-            group.any? { |variant| downcase_name.include? variant }\
-            && group.any? { |variant| downcase_branch.include?(variant) }
-        end
+    updated_lambda_configuration = {
+      function_name: function_name,
+      vpc_config: lambda_config["vpc_config"],
+      environment: lambda_config["environment"],
+      layers: lambda_config["layers"]
+    }
+    updated_lambda_configuration[:function_name] = function_name
+    p("updating_function_configuration with: ", updated_lambda_configuration)
+    update_configuration_resp = lambda_client.update_function_configuration(updated_lambda_configuration)
+    p("update_configuration_resp: ", update_configuration_resp)
+  end
+
+  def update_event
+    unless lambda_config["event"]
+      p("no event config")
+      return nil
     end
 
-    def update_lambda_configuration
-        unless configured? && lambda_config
-            p 'insufficient configuration'
-            return nil
-        end
-
-        updated_lambda_configuration = {
-            function_name: function_name,
-            vpc_config: lambda_config['vpc_config'],
-            environment: lambda_config['environment'],
-            layers: lambda_config['layers']
-        }
-        updated_lambda_configuration[:function_name] = function_name
-        p 'updating_function_configuration with: ', updated_lambda_configuration
-        update_configuration_resp = lambda_client.update_function_configuration(updated_lambda_configuration)
-        p 'update_configuration_resp: ', update_configuration_resp
+    @event = lambda_config["event"]
+    if event["event_source_arn"]
+      add_event_source
+    elsif event["schedule_expression"]
+      add_cron
     end
+  end
 
-    def update_event
-        unless lambda_config['event']
-            p 'no event config'
-            return nil
-        end
-
-        @event = lambda_config['event']
-        if event['event_source_arn']
-            add_event_source
-        elsif event['schedule_expression']
-            add_cron
-        end
-    end
+  def add_event_source
+    existing_events = lambda_client.list_event_source_mappings({
+      function_name: function_name
+    }).event_source_mappings
 
-    def add_event_source
-        existing_events = lambda_client.list_event_source_mappings({
-            function_name: function_name
-        }).event_source_mappings
-
-        existing_events.each do |existing_event|
-            p 'deleting event with uuid: ', existing_event.uuid, 'and arn: ', existing_event.event_source_arn
-            lambda_client.delete_event_source_mapping({ uuid: existing_event.uuid })
-        end
-        event_to_create = event.map { |k, v| [k.to_sym, v] }.to_h
-        event_to_create[:function_name] = function_name
-        p 'creating event: ', event_to_create
-        create_resp = lambda_client.create_event_source_mapping(event_to_create)
-        p 'created: ', create_resp
+    existing_events.each do |existing_event|
+      p("deleting event with uuid: ", existing_event.uuid, "and arn: ", existing_event.event_source_arn)
+      lambda_client.delete_event_source_mapping({ uuid: existing_event.uuid })
+    end
+    event_to_create = event.map { |k, v| [k.to_sym, v] }.to_h
+    event_to_create[:function_name] = function_name
+    p("creating event: ", event_to_create)
+    create_resp = lambda_client.create_event_source_mapping(event_to_create)
+    p("created: ", create_resp)
+  end
+
+  def add_cron
+    ## create the event
+    events_client = Aws::CloudWatchEvents::Client.new(aws_configuration)
+    schedule_expression = event["schedule_expression"]
+    rule_name = "#{function_name}-rule"
+    p("rule_name: ", rule_name, "schedule_expression: ", schedule_expression)
+    events_client.put_rule(name: rule_name, schedule_expression: schedule_expression)
+
+    ## next we have to connect the event to the lambda
+    ## the first step is to get the lambda
+
+    return "missing function_name" unless function_name
+
+    p("getting lambda with function name", function_name)
+    lambda_resp = lambda_client.get_function(function_name: function_name).configuration
+    arn = lambda_resp.function_arn
+
+    ## next figure out if the lambda already has granted cloudwatch
+    ## permission to invoke
+    begin
+      policy_resp = lambda_client.get_policy(function_name: function_name)
+      if policy_resp.policy.include?("#{function_name}-permission")
+        p("lambda already has permission")
+      else
+        add_policy = true
+      end
+    rescue Aws::Lambda::Errors::ResourceNotFoundException
+      add_policy = true
+      p("no policy")
     end
 
-    def add_cron
-        ## create the event
-        events_client = Aws::CloudWatchEvents::Client.new(aws_configuration)
-        schedule_expression = event['schedule_expression']
-        rule_name = "#{function_name}-rule"
-        p 'rule_name: ', rule_name, 'schedule_expression: ', schedule_expression
-        events_client.put_rule(name: rule_name, schedule_expression: schedule_expression)
-
-        ## next we have to connect the event to the lambda
-        ## the first step is to get the lambda
-
-        return 'missing function_name' unless function_name
-
-        p 'getting lambda with function name', function_name
-        lambda_resp = lambda_client.get_function(function_name: function_name).configuration
-        arn = lambda_resp.function_arn
-
-        ## next figure out if the lambda already has granted cloudwatch
-        ## permission to invoke
-        begin
-            policy_resp = lambda_client.get_policy(function_name: function_name)
-            unless policy_resp.policy.include?("#{function_name}-permission")
-                add_policy = true
-            else
-                p 'lambda already has permission'
-            end
-        rescue Aws::Lambda::Errors::ResourceNotFoundException
-            add_policy = true
-            p 'no policy'
-        end
-
-        ## if not, add permission to invoke
-        if add_policy
-            permission = lambda_client.add_permission({
-                function_name: function_name,
-                principal: 'events.amazonaws.com',
-                statement_id: "#{function_name}-permission",
-                action: 'lambda:InvokeFunction'
-            })
-            p 'permission: ', permission
-        end
-
-        ## finally we can tell the event to invoke the lambda
-        target_id = "#{function_name}-lambda"
-        p 'putting targets ', 'rule: ', rule_name, 'target_id: ', target_id, 'arn: ', arn
-        events_client.put_targets(rule: rule_name, targets: [{ id: target_id, arn: arn }])
+    ## if not, add permission to invoke
+    if add_policy
+      permission = lambda_client.add_permission({
+        function_name: function_name,
+        principal: "events.amazonaws.com",
+        statement_id: "#{function_name}-permission",
+        action: "lambda:InvokeFunction"
+      })
+      p("permission: ", permission)
     end
+
+    ## finally we can tell the event to invoke the lambda
+    target_id = "#{function_name}-lambda"
+    p("putting targets ", "rule: ", rule_name, "target_id: ", target_id, "arn: ", arn)
+    events_client.put_targets(rule: rule_name, targets: [{ id: target_id, arn: arn }])
+  end
 end

data/lib/kinesis_client.rb

@@ -1,7 +1,7 @@
-require 'securerandom'
-require 'aws-sdk-kinesis'
-require_relative 'nypl_avro'
-require_relative 'errors'
+require "securerandom"
+require "aws-sdk-kinesis"
+require_relative "nypl_avro"
+require_relative "errors"
 # Model representing the result message posted to Kinesis stream about everything that has gone on here -- good, bad, or otherwise.
 
 class KinesisClient
@@ -12,17 +12,15 @@ class KinesisClient
     @stream_name = @config[:stream_name]
     @avro = nil
     @batch_size = @config[:batch_size] || 1
-    @batch = []
-    @automatically_push = @config[:automatically_push] == false ? false : true
+    @batch_count = 0
+    @records = []
+    @automatically_push = !(@config[:automatically_push] == false)
     @client_options = config[:profile] ? { profile: config[:profile] } : {}
-    @client = Aws::Kinesis::Client.new @client_options
+    @client = Aws::Kinesis::Client.new(@client_options)
 
-    if config[:schema_string]
-      @avro = NYPLAvro.by_name(config[:schema_string])
-    end
-
-    @shovel_method = @batch_size > 1 ? :push_to_batch : :push_record
+    @avro = NYPLAvro.by_name(config[:schema_string]) if config[:schema_string]
 
+    @shovel_method = @batch_size > 1 ? :push_to_records : :push_record
   end
 
   def convert_to_record(json_message)
@@ -37,7 +35,6 @@ class KinesisClient
       data: message,
       partition_key: partition_key
     }
-
   end
 
   def <<(json_message)
@@ -45,10 +42,10 @@ class KinesisClient
   end
 
   def push_record(json_message)
-    record = convert_to_record json_message
+    record = convert_to_record(json_message)
     record[:stream_name] = @stream_name
 
-    @client.put_record record
+    @client.put_record(record)
 
     return_hash = {}
 
@@ -57,21 +54,19 @@ class KinesisClient
       return_hash["message"] = json_message, resp
       $logger.info("Message sent to #{config[:stream_name]} #{json_message}, #{resp}") if $logger
     else
-      $logger.error("message" => "FAILED to send message to HoldRequestResult #{json_message}, #{resp}.") if $logger
-      raise NYPLError.new json_message, resp
+      $logger.error("message" => "FAILED to send message to #{@stream_name} #{json_message}, #{resp}.") if $logger
+      raise(NYPLError.new(json_message, resp))
     end
     return_hash
   end
 
-  def push_to_batch(json_message)
+  def push_to_records(json_message)
     begin
-      @batch << convert_to_record(json_message)
+      @records << convert_to_record(json_message)
     rescue AvroError => e
       $logger.error("message" => "Avro encoding error #{e.message} for #{json_message}")
     end
-    if @automatically_push && @batch.length >= @batch_size
-      push_records
-    end
+    push_records if @automatically_push && @records.length >= @batch_size
   end
 
   def push_batch(batch)
@@ -82,21 +77,31 @@ class KinesisClient
 
     $logger.debug("Received #{resp} from #{@stream_name}")
 
-    return_message = {
-      failures: resp.failed_record_count,
-      error_messages: resp.records.map {|record| record.error_message }.compact
-    }
-
-    $logger.info("Message sent to #{config[:stream_name]} #{return_message}") if $logger
-
-    return {
-      "code": "200",
-      "message": return_message.to_json
-    }
+    if resp.failed_record_count > 0 
+      return_message = {
+        failures: resp.failed_record_count,
+        failures_data: filter_failures(resp)
+      }
+      $logger.warn("Message sent to #{config[:stream_name]} #{return_message}") if $logger
+    else
+      $logger.info("Message sent to #{config[:stream_name]} successfully") if $logger
+    end
   end
 
   def push_records
-    @batch.each_slice(@batch_size) {|slice| push_batch slice}
-    @batch = []
+    if @records.length > 0 
+      @records.each_slice(@batch_size) do |slice|
+        push_batch(slice)
+        @batch_count += 1
+      end
+      @records = []
+      @batch_count = 0
+    end
+  end
+
+  def filter_failures(resp)
+    resp.records.filter_map.with_index do |record, i|
+      avro.decode(@records[i + @batch_size * @batch_count]) if record.responds_to?(:error_message)
+    end
   end
 end

data/lib/kms_client.rb

@@ -1,5 +1,5 @@
-require 'aws-sdk-kms'
-require 'base64'
+require "aws-sdk-kms"
+require "base64"
 
 class KmsClient
   @@kms = nil
@@ -11,15 +11,15 @@ class KmsClient
 
   def decrypt(cipher)
     # Assume value is base64 encoded:
-    decoded = Base64.decode64 cipher
-    decrypted = @kms.decrypt ciphertext_blob: decoded
+    decoded = Base64.decode64(cipher)
+    decrypted = @kms.decrypt(ciphertext_blob: decoded)
     decrypted[:plaintext]
   end
 
   def self.aws_kms_client(options)
     params = {
-      region: 'us-east-1',
-      stub_responses: ENV['APP_ENV'] == 'test'
+      region: "us-east-1",
+      stub_responses: ENV["APP_ENV"] == "test"
     }.merge(options)
     @@kms = Aws::KMS::Client.new(params) if @@kms.nil?
     @@kms

metadata

@@ -1,11 +1,11 @@
 --- !ruby/object:Gem::Specification
 name: nypl_ruby_util
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.0.8
 platform: ruby
 authors:
 - Daniel Appel
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
 date: 2020-09-08 00:00:00.000000000 Z
@@ -109,7 +109,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 1.0.3
 description: A repository of common utilities for NYPL Ruby application
-email:
+email: 
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -126,7 +126,7 @@ homepage: https://github.com/NYPL/NYPLRubyUtil
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -141,8 +141,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.3
-signing_key:
+rubygems_version: 3.1.2
+signing_key: 
 specification_version: 4
 summary: A repository of common utilities for NYPL Ruby application
 test_files: []