Project

General

Profile

Bug #33256

Incorrect search link from packages view for applicable or upgradable hosts

Added by Gerald Vogt 9 months ago. Updated 8 months ago.

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

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...

Associated revisions

Revision 5077fdc7 (diff)
Added by Lucy Fu 8 months ago

Fixes #33256 - encode the search link from packages view

History

#1 Updated by Gerald Vogt 9 months 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.

#2 Updated by Chris Roberts 9 months ago

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

#3 Updated by The Foreman Bot 9 months ago

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

#4 Updated by The Foreman Bot 8 months ago

  • Fixed in Releases Katello 4.3.0 added

#5 Updated by Lucy Fu 8 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF