[PATCH conductor] BZ 797993 Insufficient error checking for config server
Jan Provaznik
jprovazn at redhat.com
Thu Mar 8 15:27:16 UTC 2012
On 03/08/2012 03:53 PM, jzigmund at redhat.com wrote:
> From: Jozef Zigmund<jzigmund at redhat.com>
>
> Added connecting to a config server after removing '/' from endpoint URL.
>
> https://bugzilla.redhat.com/show_bug.cgi?id=797993
> ---
> src/app/models/config_server.rb | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/src/app/models/config_server.rb b/src/app/models/config_server.rb
> index 6444ee0..9baaf35 100644
> --- a/src/app/models/config_server.rb
> +++ b/src/app/models/config_server.rb
> @@ -122,6 +122,7 @@ class ConfigServer< ActiveRecord::Base
>
> def validate_connection
> # for validation hook
> + endpoint.chomp! '/' unless endpoint.nil?
> if not connection_valid?
> errors.add(:base, status.message)
> end
ACK
More information about the aeolus-devel
mailing list