Project

General

Profile

Bug #16909

Issue with Password having special character '@'

Added by Stephen Benjamin almost 3 years ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Category:
Installer
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1295814
Description of problem:

-->> bad component(expected user component): Potato!@19

Version-Release number of selected component (if applicable):

How reproducible:
Always

Steps to Reproduce:
1. Execute the katello-installer command with the proxy password containing the special character '@'.
2. Once the installer gets executed successfully, try to refresh the manifest to get the error message.

Actual results:
The Manifest refresh fails with the error message displaying the proxy password.
--> bad component(expected user component): Potato!@19

Expected results:
The Manifest refresh should get completed successfully without any errors.

Additional info:
I believe the below link might be helpful, where I have seen a similar issue reported in upstream.

https://github.com/rubygems/rubygems/issues/668

Associated revisions

Revision bea53437 (diff)
Added by Justin Sherrill 7 months ago

Fixes #16909 - properly escape user and pass in proxy auth

rest-client alongside URIs parse method introduces a bug
where by usernames or passwords cannot have symbols in them.
More Information:
https://github.com/rest-client/rest-client/issues/661
https://github.com/ManageIQ/manageiq/pull/17318

History

#1 Updated by Justin Sherrill almost 3 years ago

  • Subject changed from Issue with Password having special character '@' to Issue with Password having special character '@'
  • Legacy Backlogs Release (now unused) set to 114

#2 Updated by The Foreman Bot 8 months ago

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

#3 Updated by Anand Agrawal 8 months ago

  • Bugzilla link changed from 1295814 to 1502752

#4 Updated by The Foreman Bot 7 months ago

  • Assignee set to Justin Sherrill
  • Pull request https://github.com/Katello/katello/pull/7977 added

#5 Updated by Justin Sherrill 7 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by Justin Sherrill 7 months ago

  • Pull request deleted (https://github.com/Katello/katello/pull/7960)

Also available in: Atom PDF