nypl_ruby_util 0.0.5 → 0.0.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bd76f2db6e4d52927503aec7dbe8a5e50c5e0fea01bafe5b7caf184e814e3ca4
-  data.tar.gz: 2c59a7d630e18ee839780b450eab83370b32d9231e5c23c244ffe795a0323c3f
+  metadata.gz: 05c91cd9d8cd4ffd258f3a2c1f9b3cb41c2c6cf9b5df78d21f1483510e34c544
+  data.tar.gz: 55e61c0097b9721728b147b6201e0856e815c3de8631e3f41ed78b00c12d11bb
 SHA512:
-  metadata.gz: 79f01aa5a487725cc37793f1b5b4656bf9e969a8e0e234bf2c6d9927c008107c71f161630e1539037a1974663c4df95d5f415d80089aaaaa19512257b771a3dd
-  data.tar.gz: 3d3d398b64516e59736d254ff2c709aec0980b2682401216e945f78be77bd6f2f1b8f3162025bf177fc85c6d631caf08da5ff42560861caa00d1d08a9c37c0a4
+  metadata.gz: 6c2042cc492affeb2ae6d60d6c96fb6d0db3ea69c23f1ad6a31da3ff5025c52cd4d5f669112e174153e88b84dde7395f59b23b12182017c079f1ce6a024807f6
+  data.tar.gz: 832f11059c1267393afb6b2f5140dd5a8f98950f9e69c495e6fa1cfa31af947c78fbf4cd740016214b4fe1dc4bce939a6f57de295039a3673aa195da0ce2be2c

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
@@ -9,40 +9,108 @@ class KinesisClient
 
   def initialize(config)
     @config = config
+    @stream_name = @config[:stream_name]
     @avro = nil
+    @batch_size = @config[:batch_size] || 1
+    @client_options = set_config(config)
+    @batch_count = 0
+    @records = []
+    @automatically_push = !(@config[:automatically_push] == false)
+    @client = Aws::Kinesis::Client.new(@client_options)
 
-    if config[:schema_string]
-      @avro = NYPLAvro.by_name(config[:schema_string])
-    end
+    @avro = NYPLAvro.by_name(config[:schema_string]) if config[:schema_string]
 
+    @shovel_method = @batch_size > 1 ? :push_to_records : :push_record
   end
 
-  def <<(json_message)
+  def set_config(config)
+    if config[:profile]
+      { profile: config[:profile] }
+    elsif config[:custom_aws_config]
+      config[:custom_aws_config]
+    else
+      {}
+    end
+  end
+
+  def convert_to_record(json_message)
     if config[:schema_string]
       message = avro.encode(json_message, false)
     else
       message = json_message
     end
 
-    client = Aws::Kinesis::Client.new
     partition_key = (config[:partition_key] ? json_message[config[:partition_key]] : SecureRandom.hex(20)).hash.to_s
-
-    resp = client.put_record({
-      stream_name: config[:stream_name],
+    {
       data: message,
       partition_key: partition_key
-      })
+    }
+  end
+
+  def <<(json_message)
+    send(@shovel_method, json_message)
+  end
+
+#This method is broken
+  def push_record(json_message)
+    record = convert_to_record(json_message)
+    record[:stream_name] = @stream_name
+
+    @client.put_record(record)
+
+    return_hash = {}
 
-      return_hash = {}
+    if resp.successful?
+      return_hash["code"] = "200"
+      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 #{@stream_name} #{json_message}, #{resp}.") if $logger
+      raise(NYPLError.new(json_message, resp))
+    end
+    return_hash
+  end
+
+  def push_to_records(json_message)
+    begin
+      @records << convert_to_record(json_message)
+    rescue AvroError => e
+      $logger.error("message" => "Avro encoding error #{e.message} for #{json_message}")
+    end
+    push_records if @automatically_push && @records.length >= @batch_size
+  end
 
-      if resp.successful?
-        return_hash["code"] = "200"
-        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
+  def push_batch(batch)
+    resp = @client.put_records({
+      records: batch.to_a,
+      stream_name: @stream_name
+    })
+
+    if resp.failed_record_count > 0
+      failure_message = {
+        failures: resp.failed_record_count,
+        failures_data: filter_failures(resp)
+      }
+      $logger.warn("Batch sent to #{config[:stream_name]} with failures: #{failure_message}")
+    else
+      $logger.info("Batch sent to #{config[:stream_name]} successfully")
+    end
+  end
+
+  def push_records
+    if @records.length > 0
+      @records.each_slice(@batch_size) do |slice|
+        push_batch(slice)
+        @batch_count += 1
       end
-      return_hash
+      @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

data/lib/nypl_avro.rb

@@ -38,7 +38,7 @@ class NYPLAvro
     begin
       bin_encoder.write(decoded_data, encoder)
     rescue Avro::IO::AvroTypeError => e
-      raise AvroError.new(e), "Error encoding data using #{@schema.name} schema due to #{e.message}"
+      raise AvroError.new(e), "Error encoding data #{decoded_data} using #{@schema.name} schema due to #{e.message}"
     end
 
     buffer.rewind

data/lib/platform_api_client.rb

@@ -61,6 +61,7 @@ class PlatformApiClient
 
   # Authorizes the request.
   def authenticate!
+    $logger.debug('authenticating')
     # NOOP if we've already authenticated
     return nil if ! access_token.nil?
 
@@ -81,8 +82,10 @@ class PlatformApiClient
     end
 
     if response.code == '200'
-      access_token = JSON.parse(response.body)["access_token"]
+      self.access_token = JSON.parse(response.body)["access_token"]
+      $logger.debug('got token')
     else
+      $logger.debug('no token')
       nil
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nypl_ruby_util
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.9
 platform: ruby
 authors:
 - Daniel Appel
@@ -141,7 +141,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.3
+rubygems_version: 3.1.4
 signing_key:
 specification_version: 4
 summary: A repository of common utilities for NYPL Ruby application