From 357fbac8f4ac671d2542bf9d0ba3d59896b85c0e Mon Sep 17 00:00:00 2001 From: Petrik Date: Fri, 4 Sep 2026 09:31:25 +0200 Subject: [PATCH] [sinatra] Ignore POST params unless required request.params merges the GET and POST params. If we only need to read the GET params we don't need to parse the body for POST params. --- frameworks/sinatra/app.rb | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/frameworks/sinatra/app.rb b/frameworks/sinatra/app.rb index 21c2f970f..ae59ddfb0 100644 --- a/frameworks/sinatra/app.rb +++ b/frameworks/sinatra/app.rb @@ -72,17 +72,20 @@ class App < Sinatra::Base end get('/baseline11') do + params = request.GET total = params['a'].to_i + params['b'].to_i render_plain total.to_s end post('/baseline11') do + params = request.GET total = params['a'].to_i + params['b'].to_i total += request.body.read.to_i render_plain total.to_s end get '/baseline2' do + params = request.GET total = params['a'].to_i + params['b'].to_i render_plain total.to_s end @@ -106,6 +109,7 @@ class App < Sinatra::Base end get '/async-db' do + params = request.GET min_val = (params['min'] || 10).to_i max_val = (params['max'] || 50).to_i limit = (params['limit'] || 50).to_i.clamp(1, 50) @@ -121,9 +125,10 @@ class App < Sinatra::Base end get '/crud/items' do - category = request.params['category'] || 'electronics' - page = (request.params['page'] || 1).to_i - limit = (request.params['limit'] || 10).to_i + params = request.GET + category = params['category'] || 'electronics' + page = (params['page'] || 1).to_i + limit = (params['limit'] || 10).to_i offset = (page - 1) * limit rows = self.class.get_async_db&.with do |connection|