Project

General

Profile

Actions

Bug #37722

closed

[DEV] links from package details page incorrectly parse plus signs

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

Status:
Closed
Priority:
Normal
Assignee:
Category:
Repositories
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

[DEV] links from package details page incorrectly parse plus signs
Cloned from 16201128

Description of problem:

When clicking on the page in the web interface showing the details of a package, there are hyperlinks for "Installed On", "Applicable To" and "Upgradable For".  When you click on any of those three links, they open search pages with a search term like this:

applicable_rpms=httpd-2.4.37-65.module+el8.10.0 22069 b47f5c72.1.x86_64

However, if the package comes from a module, then the package name contains plus signs. These plus signs get replaced by spaces (as in the example above) and the search fails. If they are manually corrected to use plus signs instead, as in the example below, the search works:

applicable_rpms=httpd-2.4.37-65.module+el8.10.0+22069+b47f5c72.1.x86_64

 

How reproducible:

100%

 

Is this issue a regression from an earlier version:

Yes, before Katello 4.11

 

 

Steps to Reproduce:

1.  go to content > content types > packages

2. search for a package from a module (eg httpd-2.4.37-65.module+el8.10.0+22069+b47f5c72.1.x86_64 or ruby-3.3.1-2.module+el9.4.0+21859+16f07581.x86_64)

3. click on the package link
4. click on the links "Installed On", "Applicable To" and "Upgradable For"

Actual behavior:
katello searches for applicable_rpms= followed by the package name, but omits the plus signs

Expected behavior:
katello should search for applicable_rpms= followed by the package name, including the plus signs (if they are part of the package name)

Business Impact / *Additional info:*

some links in the katello server's web interface fail to work properly

DEV Tracker for https://issues.redhat.com/browse/SAT-26967

Actions #1

Updated by The Foreman Bot 4 months ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ian Ballou
  • Pull request https://github.com/Katello/katello/pull/11104 added
Actions #2

Updated by Chris Roberts 4 months ago

  • Category set to Repositories
  • Target version changed from Katello 4.15.0 to Katello 4.14.0
  • Triaged changed from No to Yes
Actions #3

Updated by The Foreman Bot 3 months ago

  • Fixed in Releases Katello 4.14.0 added
Actions #4

Updated by Anonymous 3 months ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF