shark-on-lambda 1.0.0.rc4 → 2.0.0.rc3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/.gitignore +2 -3
- data/.rubocop.yml +4 -0
- data/README.md +70 -17
- data/changelog.md +19 -1
- data/doc/upgrade-from-0.6.x-to-1.x.md +122 -0
- data/lib/shark-on-lambda.rb +9 -0
- data/lib/shark_on_lambda.rb +14 -54
- data/lib/shark_on_lambda/api_gateway_handler.rb +3 -55
- data/lib/shark_on_lambda/application.rb +75 -5
- data/lib/shark_on_lambda/base_controller.rb +29 -9
- data/lib/shark_on_lambda/cacheable.rb +21 -0
- data/lib/shark_on_lambda/configuration.rb +1 -65
- data/lib/shark_on_lambda/inferrers/serializer_inferrer.rb +10 -7
- data/lib/shark_on_lambda/jsonapi_renderer.rb +16 -10
- data/lib/shark_on_lambda/middleware/base.rb +2 -4
- data/lib/shark_on_lambda/middleware/honeybadger.rb +14 -9
- data/lib/shark_on_lambda/middleware/jsonapi_rescuer.rb +21 -1
- data/lib/shark_on_lambda/middleware/lambda_logger.rb +8 -16
- data/lib/shark_on_lambda/rake_tasks.rb +16 -0
- data/lib/shark_on_lambda/request.rb +0 -3
- data/lib/shark_on_lambda/rspec/env_builder.rb +71 -34
- data/lib/shark_on_lambda/rspec/helpers.rb +5 -87
- data/lib/shark_on_lambda/rspec/request_helpers.rb +63 -0
- data/lib/shark_on_lambda/rspec/{jsonapi_helpers.rb → response_helpers.rb} +4 -10
- data/lib/shark_on_lambda/version.rb +1 -1
- data/shark-on-lambda.gemspec +7 -5
- metadata +32 -37
- data/lib/shark_on_lambda/concerns/resettable_singleton.rb +0 -18
- data/lib/shark_on_lambda/concerns/yaml_config_loader.rb +0 -28
- data/lib/shark_on_lambda/dispatcher.rb +0 -26
- data/lib/shark_on_lambda/inferrers/name_inferrer.rb +0 -66
- data/lib/shark_on_lambda/jsonapi_controller.rb +0 -29
- data/lib/shark_on_lambda/middleware/rescuer.rb +0 -37
- data/lib/shark_on_lambda/query.rb +0 -67
- data/lib/shark_on_lambda/rack_adapters/api_gateway.rb +0 -128
- data/lib/shark_on_lambda/secrets.rb +0 -43
data/shark-on-lambda.gemspec
CHANGED
@@ -30,18 +30,20 @@ Gem::Specification.new do |spec|
|
|
30
30
|
spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
|
31
31
|
spec.require_paths = ['lib']
|
32
32
|
|
33
|
-
spec.
|
34
|
-
|
35
|
-
spec.add_dependency '
|
33
|
+
spec.required_ruby_version = '>= 2.5'
|
34
|
+
|
35
|
+
spec.add_dependency 'actionpack', '~> 6.0.0'
|
36
|
+
spec.add_dependency 'activesupport', '~> 6.0.0'
|
37
|
+
spec.add_dependency 'jsonapi-rb', '~> 0.5.0'
|
36
38
|
spec.add_dependency 'rack', '>= 2.0.8', '< 3'
|
37
|
-
spec.add_dependency '
|
39
|
+
spec.add_dependency 'rack-on-lambda', '~> 1.0'
|
40
|
+
spec.add_dependency 'zeitwerk', '~> 2.3'
|
38
41
|
|
39
42
|
# TODO: Do we really need `activemodel`?
|
40
43
|
# Or can we get away with mocking out ActiveModel::Errors?
|
41
44
|
spec.add_development_dependency 'activemodel'
|
42
45
|
spec.add_development_dependency 'bundler'
|
43
46
|
spec.add_development_dependency 'factory_bot'
|
44
|
-
spec.add_development_dependency 'fasterer'
|
45
47
|
spec.add_development_dependency 'pry'
|
46
48
|
spec.add_development_dependency 'pry-byebug'
|
47
49
|
spec.add_development_dependency 'rake'
|
metadata
CHANGED
@@ -1,57 +1,57 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: shark-on-lambda
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version:
|
4
|
+
version: 2.0.0.rc3
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Huy Dinh
|
8
8
|
autorequire:
|
9
9
|
bindir: exe
|
10
10
|
cert_chain: []
|
11
|
-
date: 2020-
|
11
|
+
date: 2020-09-25 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: actionpack
|
15
15
|
requirement: !ruby/object:Gem::Requirement
|
16
16
|
requirements:
|
17
|
-
- - "
|
17
|
+
- - "~>"
|
18
18
|
- !ruby/object:Gem::Version
|
19
|
-
version:
|
19
|
+
version: 6.0.0
|
20
20
|
type: :runtime
|
21
21
|
prerelease: false
|
22
22
|
version_requirements: !ruby/object:Gem::Requirement
|
23
23
|
requirements:
|
24
|
-
- - "
|
24
|
+
- - "~>"
|
25
25
|
- !ruby/object:Gem::Version
|
26
|
-
version:
|
26
|
+
version: 6.0.0
|
27
27
|
- !ruby/object:Gem::Dependency
|
28
28
|
name: activesupport
|
29
29
|
requirement: !ruby/object:Gem::Requirement
|
30
30
|
requirements:
|
31
|
-
- - "
|
31
|
+
- - "~>"
|
32
32
|
- !ruby/object:Gem::Version
|
33
|
-
version:
|
33
|
+
version: 6.0.0
|
34
34
|
type: :runtime
|
35
35
|
prerelease: false
|
36
36
|
version_requirements: !ruby/object:Gem::Requirement
|
37
37
|
requirements:
|
38
|
-
- - "
|
38
|
+
- - "~>"
|
39
39
|
- !ruby/object:Gem::Version
|
40
|
-
version:
|
40
|
+
version: 6.0.0
|
41
41
|
- !ruby/object:Gem::Dependency
|
42
42
|
name: jsonapi-rb
|
43
43
|
requirement: !ruby/object:Gem::Requirement
|
44
44
|
requirements:
|
45
|
-
- - "
|
45
|
+
- - "~>"
|
46
46
|
- !ruby/object:Gem::Version
|
47
|
-
version:
|
47
|
+
version: 0.5.0
|
48
48
|
type: :runtime
|
49
49
|
prerelease: false
|
50
50
|
version_requirements: !ruby/object:Gem::Requirement
|
51
51
|
requirements:
|
52
|
-
- - "
|
52
|
+
- - "~>"
|
53
53
|
- !ruby/object:Gem::Version
|
54
|
-
version:
|
54
|
+
version: 0.5.0
|
55
55
|
- !ruby/object:Gem::Dependency
|
56
56
|
name: rack
|
57
57
|
requirement: !ruby/object:Gem::Requirement
|
@@ -73,35 +73,35 @@ dependencies:
|
|
73
73
|
- !ruby/object:Gem::Version
|
74
74
|
version: '3'
|
75
75
|
- !ruby/object:Gem::Dependency
|
76
|
-
name:
|
76
|
+
name: rack-on-lambda
|
77
77
|
requirement: !ruby/object:Gem::Requirement
|
78
78
|
requirements:
|
79
79
|
- - "~>"
|
80
80
|
- !ruby/object:Gem::Version
|
81
|
-
version: '
|
81
|
+
version: '1.0'
|
82
82
|
type: :runtime
|
83
83
|
prerelease: false
|
84
84
|
version_requirements: !ruby/object:Gem::Requirement
|
85
85
|
requirements:
|
86
86
|
- - "~>"
|
87
87
|
- !ruby/object:Gem::Version
|
88
|
-
version: '
|
88
|
+
version: '1.0'
|
89
89
|
- !ruby/object:Gem::Dependency
|
90
|
-
name:
|
90
|
+
name: zeitwerk
|
91
91
|
requirement: !ruby/object:Gem::Requirement
|
92
92
|
requirements:
|
93
|
-
- - "
|
93
|
+
- - "~>"
|
94
94
|
- !ruby/object:Gem::Version
|
95
|
-
version: '
|
96
|
-
type: :
|
95
|
+
version: '2.3'
|
96
|
+
type: :runtime
|
97
97
|
prerelease: false
|
98
98
|
version_requirements: !ruby/object:Gem::Requirement
|
99
99
|
requirements:
|
100
|
-
- - "
|
100
|
+
- - "~>"
|
101
101
|
- !ruby/object:Gem::Version
|
102
|
-
version: '
|
102
|
+
version: '2.3'
|
103
103
|
- !ruby/object:Gem::Dependency
|
104
|
-
name:
|
104
|
+
name: activemodel
|
105
105
|
requirement: !ruby/object:Gem::Requirement
|
106
106
|
requirements:
|
107
107
|
- - ">="
|
@@ -115,7 +115,7 @@ dependencies:
|
|
115
115
|
- !ruby/object:Gem::Version
|
116
116
|
version: '0'
|
117
117
|
- !ruby/object:Gem::Dependency
|
118
|
-
name:
|
118
|
+
name: bundler
|
119
119
|
requirement: !ruby/object:Gem::Requirement
|
120
120
|
requirements:
|
121
121
|
- - ">="
|
@@ -129,7 +129,7 @@ dependencies:
|
|
129
129
|
- !ruby/object:Gem::Version
|
130
130
|
version: '0'
|
131
131
|
- !ruby/object:Gem::Dependency
|
132
|
-
name:
|
132
|
+
name: factory_bot
|
133
133
|
requirement: !ruby/object:Gem::Requirement
|
134
134
|
requirements:
|
135
135
|
- - ">="
|
@@ -260,36 +260,31 @@ files:
|
|
260
260
|
- bin/console
|
261
261
|
- bin/setup
|
262
262
|
- changelog.md
|
263
|
+
- doc/upgrade-from-0.6.x-to-1.x.md
|
263
264
|
- gems.rb
|
264
265
|
- lib/shark-on-lambda.rb
|
265
266
|
- lib/shark_on_lambda.rb
|
266
267
|
- lib/shark_on_lambda/api_gateway_handler.rb
|
267
268
|
- lib/shark_on_lambda/application.rb
|
268
269
|
- lib/shark_on_lambda/base_controller.rb
|
269
|
-
- lib/shark_on_lambda/
|
270
|
-
- lib/shark_on_lambda/concerns/yaml_config_loader.rb
|
270
|
+
- lib/shark_on_lambda/cacheable.rb
|
271
271
|
- lib/shark_on_lambda/configuration.rb
|
272
|
-
- lib/shark_on_lambda/dispatcher.rb
|
273
272
|
- lib/shark_on_lambda/errors/base.rb
|
274
273
|
- lib/shark_on_lambda/errors/base_serializer.rb
|
275
|
-
- lib/shark_on_lambda/inferrers/name_inferrer.rb
|
276
274
|
- lib/shark_on_lambda/inferrers/serializer_inferrer.rb
|
277
|
-
- lib/shark_on_lambda/jsonapi_controller.rb
|
278
275
|
- lib/shark_on_lambda/jsonapi_parameters.rb
|
279
276
|
- lib/shark_on_lambda/jsonapi_renderer.rb
|
280
277
|
- lib/shark_on_lambda/middleware/base.rb
|
281
278
|
- lib/shark_on_lambda/middleware/honeybadger.rb
|
282
279
|
- lib/shark_on_lambda/middleware/jsonapi_rescuer.rb
|
283
280
|
- lib/shark_on_lambda/middleware/lambda_logger.rb
|
284
|
-
- lib/shark_on_lambda/
|
285
|
-
- lib/shark_on_lambda/query.rb
|
286
|
-
- lib/shark_on_lambda/rack_adapters/api_gateway.rb
|
281
|
+
- lib/shark_on_lambda/rake_tasks.rb
|
287
282
|
- lib/shark_on_lambda/request.rb
|
288
283
|
- lib/shark_on_lambda/response.rb
|
289
284
|
- lib/shark_on_lambda/rspec/env_builder.rb
|
290
285
|
- lib/shark_on_lambda/rspec/helpers.rb
|
291
|
-
- lib/shark_on_lambda/rspec/
|
292
|
-
- lib/shark_on_lambda/
|
286
|
+
- lib/shark_on_lambda/rspec/request_helpers.rb
|
287
|
+
- lib/shark_on_lambda/rspec/response_helpers.rb
|
293
288
|
- lib/shark_on_lambda/version.rb
|
294
289
|
- shark-on-lambda.gemspec
|
295
290
|
homepage: https://github.com/Skudo/shark-on-lambda
|
@@ -304,7 +299,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
|
|
304
299
|
requirements:
|
305
300
|
- - ">="
|
306
301
|
- !ruby/object:Gem::Version
|
307
|
-
version: '
|
302
|
+
version: '2.5'
|
308
303
|
required_rubygems_version: !ruby/object:Gem::Requirement
|
309
304
|
requirements:
|
310
305
|
- - ">"
|
@@ -1,18 +0,0 @@
|
|
1
|
-
# frozen_string_literal: true
|
2
|
-
|
3
|
-
module SharkOnLambda
|
4
|
-
module Concerns
|
5
|
-
module ResettableSingleton
|
6
|
-
def self.included(base)
|
7
|
-
base.include(Singleton)
|
8
|
-
base.extend(ClassMethods)
|
9
|
-
end
|
10
|
-
|
11
|
-
module ClassMethods
|
12
|
-
def reset
|
13
|
-
@singleton__instance__ = nil
|
14
|
-
end
|
15
|
-
end
|
16
|
-
end
|
17
|
-
end
|
18
|
-
end
|
@@ -1,28 +0,0 @@
|
|
1
|
-
# frozen_string_literal: true
|
2
|
-
|
3
|
-
module SharkOnLambda
|
4
|
-
module Concerns
|
5
|
-
module YamlConfigLoader
|
6
|
-
def load_yaml_files(stage:, fallback: :default, paths:)
|
7
|
-
result = HashWithIndifferentAccess.new
|
8
|
-
paths.each do |path|
|
9
|
-
data = load_yaml_file(stage: stage, fallback: fallback, path: path)
|
10
|
-
result.deep_merge!(data)
|
11
|
-
end
|
12
|
-
result
|
13
|
-
end
|
14
|
-
|
15
|
-
protected
|
16
|
-
|
17
|
-
def load_yaml_file(stage:, fallback:, path:)
|
18
|
-
return {} unless File.exist?(path)
|
19
|
-
|
20
|
-
data = YAML.load_file(path)
|
21
|
-
return {} unless data.is_a?(Hash)
|
22
|
-
|
23
|
-
data = data.with_indifferent_access
|
24
|
-
data[stage] || data[fallback] || {}
|
25
|
-
end
|
26
|
-
end
|
27
|
-
end
|
28
|
-
end
|
@@ -1,26 +0,0 @@
|
|
1
|
-
# frozen_string_literal: true
|
2
|
-
|
3
|
-
module SharkOnLambda
|
4
|
-
class Dispatcher
|
5
|
-
def call(env)
|
6
|
-
request = Request.new(env)
|
7
|
-
response = Response.new
|
8
|
-
|
9
|
-
controller = controller_class(env)
|
10
|
-
action = controller_action(env)
|
11
|
-
controller.dispatch(action, request, response)
|
12
|
-
|
13
|
-
response.prepare!
|
14
|
-
end
|
15
|
-
|
16
|
-
private
|
17
|
-
|
18
|
-
def controller_action(env)
|
19
|
-
env['shark.action']
|
20
|
-
end
|
21
|
-
|
22
|
-
def controller_class(env)
|
23
|
-
env['shark.controller'].camelize.constantize
|
24
|
-
end
|
25
|
-
end
|
26
|
-
end
|
@@ -1,66 +0,0 @@
|
|
1
|
-
# frozen_string_literal: true
|
2
|
-
|
3
|
-
module SharkOnLambda
|
4
|
-
module Inferrers
|
5
|
-
class NameInferrer
|
6
|
-
class << self
|
7
|
-
def from_controller_name(class_name)
|
8
|
-
from_name(:controller, class_name)
|
9
|
-
end
|
10
|
-
|
11
|
-
def from_deserializer_name(class_name)
|
12
|
-
from_name(:deserializer, class_name)
|
13
|
-
end
|
14
|
-
|
15
|
-
def from_handler_name(class_name)
|
16
|
-
from_name(:handler, class_name)
|
17
|
-
end
|
18
|
-
|
19
|
-
def from_model_name(class_name)
|
20
|
-
from_name(:model, class_name)
|
21
|
-
end
|
22
|
-
|
23
|
-
def from_serializer_name(class_name)
|
24
|
-
from_name(:serializer, class_name)
|
25
|
-
end
|
26
|
-
|
27
|
-
protected
|
28
|
-
|
29
|
-
def from_name(type, class_name)
|
30
|
-
base = class_name.underscore
|
31
|
-
base = case type
|
32
|
-
when :controller, :deserializer, :handler, :serializer
|
33
|
-
base.sub(/_#{type}\z/, '')
|
34
|
-
when :model
|
35
|
-
base
|
36
|
-
end
|
37
|
-
new(base)
|
38
|
-
end
|
39
|
-
end
|
40
|
-
|
41
|
-
def initialize(base)
|
42
|
-
@base = base
|
43
|
-
end
|
44
|
-
|
45
|
-
def controller
|
46
|
-
"#{@base}_controller".camelize
|
47
|
-
end
|
48
|
-
|
49
|
-
def deserializer
|
50
|
-
"#{@base}_deserializer".camelize
|
51
|
-
end
|
52
|
-
|
53
|
-
def handler
|
54
|
-
"#{@base}_handler".camelize
|
55
|
-
end
|
56
|
-
|
57
|
-
def model
|
58
|
-
@base.camelize
|
59
|
-
end
|
60
|
-
|
61
|
-
def serializer
|
62
|
-
"#{@base}_serializer".camelize
|
63
|
-
end
|
64
|
-
end
|
65
|
-
end
|
66
|
-
end
|
@@ -1,29 +0,0 @@
|
|
1
|
-
# frozen_string_literal: true
|
2
|
-
|
3
|
-
module SharkOnLambda
|
4
|
-
class JsonapiController < BaseController
|
5
|
-
ActionController::Renderers.add :jsonapi do |object, options|
|
6
|
-
response.set_header('content-type', 'application/vnd.api+json')
|
7
|
-
return { data: {} }.to_json if object.nil?
|
8
|
-
|
9
|
-
jsonapi_params = JsonapiParameters.new(params)
|
10
|
-
jsonapi_renderer = JsonapiRenderer.new
|
11
|
-
render_options = jsonapi_params.to_h.deep_merge(options)
|
12
|
-
|
13
|
-
jsonapi_renderer.render(object, render_options)
|
14
|
-
end
|
15
|
-
|
16
|
-
def redirect_to(options = {}, response_status = {})
|
17
|
-
super
|
18
|
-
return if response_status[:status] == 304
|
19
|
-
|
20
|
-
self.response_body = { data: {} }.to_json
|
21
|
-
end
|
22
|
-
|
23
|
-
def render(object, options = {})
|
24
|
-
options.merge!(jsonapi: object, content_type: 'application/vnd.api+json')
|
25
|
-
|
26
|
-
super(options)
|
27
|
-
end
|
28
|
-
end
|
29
|
-
end
|
@@ -1,37 +0,0 @@
|
|
1
|
-
# frozen_string_literal: true
|
2
|
-
|
3
|
-
module SharkOnLambda
|
4
|
-
module Middleware
|
5
|
-
class Rescuer < Base
|
6
|
-
private
|
7
|
-
|
8
|
-
def _call(env)
|
9
|
-
app.call(env)
|
10
|
-
rescue Errors::Base => e
|
11
|
-
rescue_shark_error(e)
|
12
|
-
rescue StandardError => e
|
13
|
-
rescue_standard_error(e)
|
14
|
-
end
|
15
|
-
|
16
|
-
def error_response(status, headers, message)
|
17
|
-
response_body = Rack::BodyProxy.new([message]) do
|
18
|
-
message.close if message.respond_to?(:close)
|
19
|
-
end
|
20
|
-
|
21
|
-
[status, headers, response_body]
|
22
|
-
end
|
23
|
-
|
24
|
-
def rescue_shark_error(error)
|
25
|
-
status = error.status || 500
|
26
|
-
error_response(status, {}, error.message)
|
27
|
-
end
|
28
|
-
|
29
|
-
def rescue_standard_error(error)
|
30
|
-
SharkOnLambda.logger.error(error.message)
|
31
|
-
SharkOnLambda.logger.error(error.backtrace.join("\n"))
|
32
|
-
|
33
|
-
error_response(500, {}, error.message)
|
34
|
-
end
|
35
|
-
end
|
36
|
-
end
|
37
|
-
end
|
@@ -1,67 +0,0 @@
|
|
1
|
-
# frozen_string_literal: true
|
2
|
-
|
3
|
-
module SharkOnLambda
|
4
|
-
# The `multiValueQueryStringParameters` object from the API Gateway event
|
5
|
-
# keeps _all_ values in an array, regardless of the actual size of the array
|
6
|
-
# and regardless of the "intent" of the query string parameter.
|
7
|
-
#
|
8
|
-
# In order to normalise this behaviour, we treat the query strings
|
9
|
-
#
|
10
|
-
# `key=value1&key=value2`
|
11
|
-
#
|
12
|
-
# and
|
13
|
-
#
|
14
|
-
# `key[]=value1&key[]=value2`
|
15
|
-
#
|
16
|
-
# the same. Both are to be serialised to the query string
|
17
|
-
#
|
18
|
-
# `key[]=value1&key[]=value2`
|
19
|
-
#
|
20
|
-
# However, the query strings
|
21
|
-
#
|
22
|
-
# `key=value`
|
23
|
-
#
|
24
|
-
# and
|
25
|
-
#
|
26
|
-
# `key[]=value`
|
27
|
-
#
|
28
|
-
# are _not_ to be treated the same.
|
29
|
-
class Query
|
30
|
-
def initialize(data = {})
|
31
|
-
@params = HashWithIndifferentAccess.new.merge(data)
|
32
|
-
end
|
33
|
-
|
34
|
-
def add(key, values)
|
35
|
-
if key.to_s.end_with?('[]')
|
36
|
-
actual_key = key[0..-3]
|
37
|
-
add_list(actual_key, values)
|
38
|
-
else
|
39
|
-
values.each { |value| add_item(key, value) }
|
40
|
-
end
|
41
|
-
end
|
42
|
-
|
43
|
-
def to_h
|
44
|
-
@params
|
45
|
-
end
|
46
|
-
|
47
|
-
def to_s
|
48
|
-
Rack::Utils.build_nested_query(to_h)
|
49
|
-
end
|
50
|
-
|
51
|
-
private
|
52
|
-
|
53
|
-
def add_item(key, value)
|
54
|
-
if @params[key].nil?
|
55
|
-
@params[key] = value
|
56
|
-
else
|
57
|
-
@params[key] = Array(@params[key])
|
58
|
-
@params[key] << value
|
59
|
-
end
|
60
|
-
end
|
61
|
-
|
62
|
-
def add_list(key, value)
|
63
|
-
@params[key] ||= []
|
64
|
-
@params[key].concat(value)
|
65
|
-
end
|
66
|
-
end
|
67
|
-
end
|