[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