Bug 1832559

Summary:	Http Proxy "Test connection" does not handle special characters in username or password
Product:	Red Hat Satellite	Reporter:	Justin Sherrill <jsherril>
Component:	Networking	Assignee:	satellite6-bugs <satellite6-bugs>
Status:	CLOSED NOTABUG	QA Contact:	Satellite QE Team <sat-qe-bz-list>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	6.7.0	CC:	lstejska
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2023-06-13 08:27:33 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Justin Sherrill 2020-05-06 20:18:10 UTC

Description of problem:

Trying to create an http proxy with a password of:

'foo#@' 

will not work correctly

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

How reproducible:
always

Steps to Reproduce:
1. Attempt to create an http proxy with a password of 'foo#@' 
2. Try to test the proxy or use it for ec2 communication 

Actual results:
'bad password component'

Expected results:
works as expected


Additional info:
This doesn't appear at first glance to affect repository syncing, but more testing needs to be done

Comment 3 Brad Buckingham 2020-05-07 13:11:50 UTC

This bugzilla was opened based upon : bug 1825930#c12

Comment 5 Justin Sherrill 2020-05-07 14:40:14 UTC

After digging into it a bit more, i think the only thing that is not working is:

1) the 'test connection' button
2) the redhat access plugin (associated with BZ1825930)

So i think this BZ is really just tied to the 'test connection' button.  I tested AWS access and pulp2 repo syncing, and both worked fine with an http proxy created via the webUI.  The test connection button not working, does give the impression that it won't work. 

I do not consider this a regression.

Comment 6 Leos Stejskal 2023-06-13 08:27:33 UTC

Hi, we were unable to reproduce the issue on the latest version of Satellite.
As a result, I am closing the BZ ticket. However, please note that if the issue reoccurs or if additional information becomes available, we can reopen the ticket for further investigation.