actionpack 6.1.3.2 → 6.1.4

Sign up to get free protection for your applications and to get access to all the features.

Potentially problematic release.


This version of actionpack might be problematic. Click here for more details.

checksums.yaml CHANGED
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  SHA256:
3
- metadata.gz: 9cfba18fd1e2c6507de318e22c9cab8f878c22b9cbce70065f9c2ff9053f1d96
4
- data.tar.gz: 178c4347f79392aaa6d452c3eeb406cbecf66a9b3ece0d244f98d7a4fa15b6ab
3
+ metadata.gz: 9ecbd6190263018a55d9baa5737fbb8cadbcfe576eab8a37abe85f243bf7280f
4
+ data.tar.gz: 3eb5afac5cae0df74480ded35e48956a41b4017e43f9dfb7e895557ebf2efd02
5
5
  SHA512:
6
- metadata.gz: 6479340989a0677d3e64e0723a67c0cc64ea4dc9d5fcfe6bfad6538703582f3b95bc2426b68c56c2dda3a1b66842e7ed890b70fefbf136c8a198d23336525447
7
- data.tar.gz: 9021e02a953bf859d984e3722561745d0b562292b3fe3566ee1d362a9eb8c06af868f2d17db7dd5558e31871f626a39d8d9a7bb9b34697c5d26b8b58a533db4f
6
+ metadata.gz: d25fc0df75affc20fce6891e4da46226a851b48f4379b491fa887b194866a0d2f25f619f0401ffa524bd14369a0f5705f3ae8d12ea513976c48a3101f1327925
7
+ data.tar.gz: e7bc0083bb78130cbb4d2b0d421fdc69c146b7040e64da4d2ed0c18715f0a94b1c5d9dc2447439c31c7e63682dab1b03e5d0db0c6a14bb020e99a5da44ae9728
data/CHANGELOG.md CHANGED
@@ -1,3 +1,41 @@
1
+ ## Rails 6.1.4 (June 24, 2021) ##
2
+
3
+ * Ignore file fixtures on `db:fixtures:load`
4
+
5
+ *Kevin Sjöberg*
6
+
7
+ * Fix ActionController::Live controller test deadlocks by removing the body buffer size limit for tests.
8
+
9
+ *Dylan Thacker-Smith*
10
+
11
+ * Correctly place optional path parameter booleans.
12
+
13
+ Previously, if you specify a url parameter that is part of the path as false it would include that part
14
+ of the path as parameter for example:
15
+
16
+ ```
17
+ get "(/optional/:optional_id)/things" => "foo#foo", as: :things
18
+ things_path(optional_id: false) # => /things?optional_id=false
19
+ ```
20
+
21
+ After this change, true and false will be treated the same when used as optional path parameters. Meaning now:
22
+
23
+ ```
24
+ get '(this/:my_bool)/that' as: :that
25
+
26
+ that_path(my_bool: true) # => `/this/true/that`
27
+ that_path(my_bool: false) # => `/this/false/that`
28
+ ```
29
+
30
+ *Adam Hess*
31
+
32
+ * Add support for 'private, no-store' Cache-Control headers.
33
+
34
+ Previously, 'no-store' was exclusive; no other directives could be specified.
35
+
36
+ *Alex Smith*
37
+
38
+
1
39
  ## Rails 6.1.3.2 (May 05, 2021) ##
2
40
 
3
41
  * Prevent open redirects by correctly escaping the host allow list
@@ -44,7 +44,7 @@ module ActionController
44
44
  # template digest from the ETag.
45
45
  def pick_template_for_etag(options)
46
46
  unless options[:template] == false
47
- options[:template] || "#{controller_path}/#{action_name}"
47
+ options[:template] || lookup_context.find_all(action_name, _prefixes).first&.virtual_path
48
48
  end
49
49
  end
50
50
 
@@ -127,6 +127,11 @@ module ActionController
127
127
  class Buffer < ActionDispatch::Response::Buffer #:nodoc:
128
128
  include MonitorMixin
129
129
 
130
+ class << self
131
+ attr_accessor :queue_size
132
+ end
133
+ @queue_size = 10
134
+
130
135
  # Ignore that the client has disconnected.
131
136
  #
132
137
  # If this value is `true`, calling `write` after the client
@@ -136,7 +141,7 @@ module ActionController
136
141
  attr_accessor :ignore_disconnect
137
142
 
138
143
  def initialize(response)
139
- super(response, SizedQueue.new(10))
144
+ super(response, build_queue(self.class.queue_size))
140
145
  @error_callback = lambda { true }
141
146
  @cv = new_cond
142
147
  @aborted = false
@@ -214,6 +219,10 @@ module ActionController
214
219
  yield str
215
220
  end
216
221
  end
222
+
223
+ def build_queue(queue_size)
224
+ queue_size ? SizedQueue.new(queue_size) : Queue.new
225
+ end
217
226
  end
218
227
 
219
228
  class Response < ActionDispatch::Response #:nodoc: all
@@ -281,7 +281,10 @@ module ActionController
281
281
  return false unless request.has_content_type?
282
282
 
283
283
  ref = request.content_mime_type.ref
284
+
284
285
  _wrapper_formats.include?(ref) && _wrapper_key && !request.parameters.key?(_wrapper_key)
286
+ rescue ActionDispatch::Http::Parameters::ParseError
287
+ false
285
288
  end
286
289
 
287
290
  def _perform_parameter_wrapping
@@ -295,8 +298,6 @@ module ActionController
295
298
 
296
299
  # This will display the wrapped hash in the log file.
297
300
  request.filtered_parameters.merge! wrapped_filtered_hash
298
- rescue ActionDispatch::Http::Parameters::ParseError
299
- # swallow parse error exception
300
301
  end
301
302
  end
302
303
  end
@@ -24,6 +24,9 @@ module ActionController
24
24
  def new_controller_thread # :nodoc:
25
25
  yield
26
26
  end
27
+
28
+ # Avoid a deadlock from the queue filling up
29
+ Buffer.queue_size = nil
27
30
  end
28
31
 
29
32
  # ActionController::TestCase will be deprecated and moved to a gem in the future.
@@ -195,31 +195,30 @@ module ActionDispatch
195
195
 
196
196
  control.merge! cache_control
197
197
 
198
+ options = []
199
+
198
200
  if control[:no_store]
199
- self._cache_control = NO_STORE
201
+ options << PRIVATE if control[:private]
202
+ options << NO_STORE
200
203
  elsif control[:no_cache]
201
- options = []
202
204
  options << PUBLIC if control[:public]
203
205
  options << NO_CACHE
204
206
  options.concat(control[:extras]) if control[:extras]
205
-
206
- self._cache_control = options.join(", ")
207
207
  else
208
208
  extras = control[:extras]
209
209
  max_age = control[:max_age]
210
210
  stale_while_revalidate = control[:stale_while_revalidate]
211
211
  stale_if_error = control[:stale_if_error]
212
212
 
213
- options = []
214
213
  options << "max-age=#{max_age.to_i}" if max_age
215
214
  options << (control[:public] ? PUBLIC : PRIVATE)
216
215
  options << MUST_REVALIDATE if control[:must_revalidate]
217
216
  options << "stale-while-revalidate=#{stale_while_revalidate.to_i}" if stale_while_revalidate
218
217
  options << "stale-if-error=#{stale_if_error.to_i}" if stale_if_error
219
218
  options.concat(extras) if extras
220
-
221
- self._cache_control = options.join(", ")
222
219
  end
220
+
221
+ self._cache_control = options.join(", ")
223
222
  end
224
223
  end
225
224
  end
@@ -103,7 +103,7 @@ module ActionDispatch
103
103
  parameterized_parts = recall.merge(options)
104
104
 
105
105
  keys_to_keep = route.parts.reverse_each.drop_while { |part|
106
- !(options.key?(part) || route.scope_options.key?(part)) || (options[part] || recall[part]).nil?
106
+ !(options.key?(part) || route.scope_options.key?(part)) || (options[part].nil? && recall[part].nil?)
107
107
  } | route.required_parts
108
108
 
109
109
  parameterized_parts.delete_if do |bad_key, _|
@@ -118,7 +118,7 @@ module ActionDispatch
118
118
  end
119
119
  end
120
120
 
121
- parameterized_parts.keep_if { |_, v| v }
121
+ parameterized_parts.compact!
122
122
  parameterized_parts
123
123
  end
124
124
 
@@ -40,7 +40,7 @@ module ActionDispatch
40
40
  @parameters.each do |index|
41
41
  param = parts[index]
42
42
  value = hash[param.name]
43
- return "" unless value
43
+ return "" if value.nil?
44
44
  parts[index] = param.escape value
45
45
  end
46
46
 
@@ -6,6 +6,7 @@
6
6
  <%= @exception.message %>
7
7
  <% if defined?(ActiveStorage) && @exception.message.match?(%r{#{ActiveStorage::Blob.table_name}|#{ActiveStorage::Attachment.table_name}}) %>
8
8
  To resolve this issue run: bin/rails active_storage:install
9
+ <% end %>
9
10
  <% if defined?(ActionMailbox) && @exception.message.match?(%r{#{ActionMailbox::InboundEmail.table_name}}) %>
10
11
  To resolve this issue run: bin/rails action_mailbox:install
11
12
  <% end %>
@@ -9,8 +9,8 @@ module ActionPack
9
9
  module VERSION
10
10
  MAJOR = 6
11
11
  MINOR = 1
12
- TINY = 3
13
- PRE = "2"
12
+ TINY = 4
13
+ PRE = nil
14
14
 
15
15
  STRING = [MAJOR, MINOR, TINY, PRE].compact.join(".")
16
16
  end
metadata CHANGED
@@ -1,14 +1,14 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: actionpack
3
3
  version: !ruby/object:Gem::Version
4
- version: 6.1.3.2
4
+ version: 6.1.4
5
5
  platform: ruby
6
6
  authors:
7
7
  - David Heinemeier Hansson
8
- autorequire:
8
+ autorequire:
9
9
  bindir: bin
10
10
  cert_chain: []
11
- date: 2021-05-05 00:00:00.000000000 Z
11
+ date: 2021-06-24 00:00:00.000000000 Z
12
12
  dependencies:
13
13
  - !ruby/object:Gem::Dependency
14
14
  name: activesupport
@@ -16,14 +16,14 @@ dependencies:
16
16
  requirements:
17
17
  - - '='
18
18
  - !ruby/object:Gem::Version
19
- version: 6.1.3.2
19
+ version: 6.1.4
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: 6.1.3.2
26
+ version: 6.1.4
27
27
  - !ruby/object:Gem::Dependency
28
28
  name: rack
29
29
  requirement: !ruby/object:Gem::Requirement
@@ -98,28 +98,28 @@ dependencies:
98
98
  requirements:
99
99
  - - '='
100
100
  - !ruby/object:Gem::Version
101
- version: 6.1.3.2
101
+ version: 6.1.4
102
102
  type: :runtime
103
103
  prerelease: false
104
104
  version_requirements: !ruby/object:Gem::Requirement
105
105
  requirements:
106
106
  - - '='
107
107
  - !ruby/object:Gem::Version
108
- version: 6.1.3.2
108
+ version: 6.1.4
109
109
  - !ruby/object:Gem::Dependency
110
110
  name: activemodel
111
111
  requirement: !ruby/object:Gem::Requirement
112
112
  requirements:
113
113
  - - '='
114
114
  - !ruby/object:Gem::Version
115
- version: 6.1.3.2
115
+ version: 6.1.4
116
116
  type: :development
117
117
  prerelease: false
118
118
  version_requirements: !ruby/object:Gem::Requirement
119
119
  requirements:
120
120
  - - '='
121
121
  - !ruby/object:Gem::Version
122
- version: 6.1.3.2
122
+ version: 6.1.4
123
123
  description: Web apps on Rails. Simple, battle-tested conventions for building and
124
124
  testing MVC web applications. Works with any Rack-compatible server.
125
125
  email: david@loudthinking.com
@@ -309,11 +309,11 @@ licenses:
309
309
  - MIT
310
310
  metadata:
311
311
  bug_tracker_uri: https://github.com/rails/rails/issues
312
- changelog_uri: https://github.com/rails/rails/blob/v6.1.3.2/actionpack/CHANGELOG.md
313
- documentation_uri: https://api.rubyonrails.org/v6.1.3.2/
312
+ changelog_uri: https://github.com/rails/rails/blob/v6.1.4/actionpack/CHANGELOG.md
313
+ documentation_uri: https://api.rubyonrails.org/v6.1.4/
314
314
  mailing_list_uri: https://discuss.rubyonrails.org/c/rubyonrails-talk
315
- source_code_uri: https://github.com/rails/rails/tree/v6.1.3.2/actionpack
316
- post_install_message:
315
+ source_code_uri: https://github.com/rails/rails/tree/v6.1.4/actionpack
316
+ post_install_message:
317
317
  rdoc_options: []
318
318
  require_paths:
319
319
  - lib
@@ -330,7 +330,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
330
330
  requirements:
331
331
  - none
332
332
  rubygems_version: 3.1.2
333
- signing_key:
333
+ signing_key:
334
334
  specification_version: 4
335
335
  summary: Web-flow and rendering framework putting the VC in MVC (part of Rails).
336
336
  test_files: []