Project

General

Profile

Bug #34984

ActionController::BadRequest when uploading RPMs via Hammer

Added by Ian Ballou 3 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Content Uploads
Target version:
Difficulty:
hard
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

For some reason, the Rails 6.1 upgrade suddenly cause the binary file data to go through a UTF-8 encoding check. This fails since it's just binary data (RPMs).

2022-05-25T19:44:44 [I|app|241d1a28] Started PUT "/katello/api/repositories/1/content_uploads/e43ff7cc-b87d-427d-9211-db6544be1f62" for 192.168.121.18 at 2022-05-25 19:44:44 +0000
2022-05-25T19:44:44 [F|app|241d1a28]
241d1a28 | ActionController::BadRequest (Invalid request parameters: Invalid encoding for parameter: [binary gibberish]

241d1a28 |   
241d1a28 | app/controllers/concerns/foreman/controller/filter_parameters.rb:15:in `process_action'
241d1a28 | katello (4.6.0.pre.master) lib/katello/middleware/organization_created_enforcer.rb:18:in `call'
241d1a28 | katello (4.6.0.pre.master) lib/katello/middleware/event_daemon.rb:10:in `call'
241d1a28 | lib/foreman/middleware/libvirt_connection_cleaner.rb:9:in `call'
241d1a28 | lib/foreman/middleware/telemetry.rb:10:in `call'
241d1a28 | lib/foreman/middleware/logging_context_session.rb:22:in `call'
241d1a28 | lib/foreman/middleware/logging_context_request.rb:11:in `call'
241d1a28 | katello (4.6.0.pre.master) lib/katello/prevent_json_parsing.rb:12:in `call'

Related thread: https://community.theforeman.org/t/katello-nightly-rpm-pipeline-1337-failed/28772

Associated revisions

Revision 2be708d3 (diff)
Added by Ian Ballou 3 months ago

Fixes #34984 - ActionController::BadRequest when uploading RPMs via Hammer (#10137)

History

#1 Updated by The Foreman Bot 3 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/10137 added

#2 Updated by The Foreman Bot 3 months ago

  • Fixed in Releases Katello 4.5.0 added

#3 Updated by Anonymous 3 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by Jeremy Lenz 3 months ago

  • Triaged changed from No to Yes

Also available in: Atom PDF