Project

General

Profile

Actions

Bug #33256

closed

Incorrect search link from packages view for applicable or upgradable hosts

Added by Gerald Vogt over 2 years ago. Updated over 2 years ago.

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

Description

Katello 4.1.2.1.

If there is a package name with a + (plus) sign in the name, e.g. for the EL8 module packages, the links to search for applicable or upgradable hosts don't encode the search correctly.

Go to a package which is applicable or upgradable to some hosts, i.e. URL like !http://foreman.example.com/packages/12345

If the package name contains plus signs, e.g. conmon-2:2.0.29-1.module_el8.4.0+886+c9a8d9ad.x86_64, the links for the applicable or upgradeable hosts (e.g. (1 Host(s)") look like this:

!https://foreman.example.com/content_hosts?search=applicable_rpms=conmon-2.0.29-1.module_el8.4.0+886+c9a8d9ad.x86_64

which leads to this search string: "applicable_rpms=conmon-2.0.29-1.module_el8.4.0 886 c9a8d9ad.x86_64"

The plus signs are replaced by a space and thus nothing is found. If I manually replace the space with a plus it works: "applicable_rpms=conmon-2.0.29-1.module_el8.4.0+886+c9a8d9ad.x86_64"

So I guess the plus should be encoded for the search...

Actions #1

Updated by Gerald Vogt over 2 years ago

Sven Jansen wrote:

i can confirm this issue on katello 4.1.2.1, this are all packages of one of my systems that show up as applicable but they are not. All my hosts with this issues are packages with + sign in it.

I think you are confusing issues. Your problem is here https://projects.theforeman.org/issues/32739

This is issue here is just a conversion/encoding issue in the frontend. Those packages are in deed applicable, but the link is wrong.

Actions #2

Updated by Chris Roberts over 2 years ago

  • Category set to Hosts
  • Target version set to Katello 4.3.0
  • Triaged changed from No to Yes
Actions #3

Updated by The Foreman Bot over 2 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Lucy Fu
  • Pull request https://github.com/Katello/katello/pull/9606 added
Actions #4

Updated by The Foreman Bot over 2 years ago

  • Fixed in Releases Katello 4.3.0 added
Actions #5

Updated by Lucy Fu over 2 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF