Bug 797993

Summary:	Insufficient error checking for config server
Product:	[Retired] CloudForms Cloud Engine	Reporter:	james labocki <jlabocki>
Component:	aeolus-conductor	Assignee:	Jozef Zigmund <jzigmund>
Status:	CLOSED ERRATA	QA Contact:	wes hayutin <whayutin>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	1.0.0	CC:	akarol, asettle, athomas, cpelland, dajohnso, deltacloud-maint, dgao, juwu, jzigmund, morazi, slinaber, ssachdev
Target Milestone:	1.0.2	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:	When configuring a new config-server for a cloud provider, the 'Server Endpoint (URL)' field did not provide adequate checking for URL strings with a trailing '/'. This bug fix allows Conductor to check if removing a trailing '/' allows the endpoint to be added successfully.	Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-12-04 14:56:45 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description james labocki 2012-02-27 18:16:34 UTC

Description of problem:
When configuring a new config server for a cloud provider account the "Server Endpoint (URL)" field does not provide adequate checking of the URL string to see if an ending "/" can be trimmed to allow adding of an endpoint URL to be successful. Currently:

Succeeds - https://dhcp47-239.lab.bos.redhat.com 
Fails -    https://dhcp47-239.lab.bos.redhat.com/

conductor should check to see if removing a trailing "/" allows the endpoint to be added successfully and then perform the action if it succeeds.

Comment 1 Jozef Zigmund 2012-03-07 19:09:53 UTC

Bugfix on the list : https://fedorahosted.org/pipermail/aeolus-devel/2012-March/009455.html

Comment 2 Jozef Zigmund 2012-03-08 16:47:33 UTC

Bugfix was pushed with commit # bbf6b907fe95370473ad96e1479e5ce9cf0cdf5d, please verify it.

Comment 3 Steve Linabery 2012-04-12 21:47:39 UTC

bbf6b907fe95370473ad96e1479e5ce9cf0cdf5d only exists on master as of this writing.

Comment 5 dgao 2012-09-25 16:04:15 UTC

Works with both

https://deaddonkey.usersys.redhat.com
https://deaddonkey.usersys.redhat.com/

Verified

Comment 8 errata-xmlrpc 2012-12-04 14:56:45 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2012-1516.html