some error handling

@@ -2,13 +2,12 @@ class DashboardController < ApplicationController
   def index
   def index
     @hosts = Host.all
     @hosts = Host.all
 
 
-    @hosts.map { |host|
-      if host.lxd_config.auth == 'untrusted'
-        session[:return_to] = request.env["REQUEST_URI"]
-        redirect_to controller: 'hosts', action: 'auth', id: host.id
-        return
-      end
-    }
+#    @hosts.map { |host|
+#      if host.connected and not host.authenticated
+#        session[:return_to] = request.env["REQUEST_URI"]
+#        redirect_to controller: 'hosts', action: 'auth', id: host.id
+#      end
+#    }
   end
   end
 end
 end
 # vim: set et ts=2 sw=2:
 # vim: set et ts=2 sw=2:

@@ -7,9 +7,7 @@ class Certificate < ActiveRecord::Base
   def self.get
   def self.get
     @@cert ||= find_by active: true
     @@cert ||= find_by active: true
     @@cert ||= create
     @@cert ||= create
-    if @@cert.is_expired?
-      @@cert = @@cert.update
-    end
+    @@cert = @@cert.update if @@cert.expires_soon?
     @@cert
     @@cert
   end
   end
 
 

@@ -10,7 +10,7 @@ class Host < ActiveRecord::Base
 			super(true)
 			super(true)
 		when super.expires_soon?
 		when super.expires_soon?
 			old = super
 			old = super
-			new = Certificate.get.update
+			new = Certificate.get
 			Lxd::Certificate.new(api: api(old), certificate: new.to_s).add
 			Lxd::Certificate.new(api: api(old), certificate: new.to_s).add
 			self.certificate_id = new.id
 			self.certificate_id = new.id
 			self.save
 			self.save
@@ -35,8 +35,16 @@ class Host < ActiveRecord::Base
 		Lxd::Certificate.new(api: api).add password
 		Lxd::Certificate.new(api: api).add password
 	end
 	end
 
 
+	def connected
+		lxd_config != nil
+	end
+
+	def authenticated
+		lxd_config != nil and lxd_config.auth == 'trusted'
+	end
+
 	private
 	private
-	def api certificate = nil
+	def api certificate=nil
 		@api ||= Lxd::API.get self, certificate
 		@api ||= Lxd::API.get self, certificate
 	end
 	end
 end
 end

 module Lxd::API
 module Lxd::API
   def self.get host, certificate = nil
   def self.get host, certificate = nil
     certificate ||= host.certificate
     certificate ||= host.certificate
+
     uri = URI.parse host.uri
     uri = URI.parse host.uri
     con = Net::HTTP.new uri.host, uri.port ? uri.port : 8443
     con = Net::HTTP.new uri.host, uri.port ? uri.port : 8443
     con.use_ssl = true
     con.use_ssl = true
@@ -8,14 +9,33 @@ module Lxd::API
     con.key = OpenSSL::PKey::RSA.new certificate.key
     con.key = OpenSSL::PKey::RSA.new certificate.key
     con.verify_mode = OpenSSL::SSL::VERIFY_NONE
     con.verify_mode = OpenSSL::SSL::VERIFY_NONE
 
 
-    resp = self.call con, Net::HTTP::Get.new('/')
-    return Lxd::API::V1_0.new con if resp['metadata'].include? '/1.0'
-    raise "unsupported api version"
+    resp = call con, Net::HTTP::Get.new('/')
+    api = Lxd::API::V1_0.new con if resp['metadata'].include? '/1.0'
+    raise Lxd::API::Exception 'unsupported api version' unless api
+    return api unless block_given?
+    yield api
+  rescue Lxd::API::Exception => e
+    Rails.logger.error { "#{e.message} #{e.backtrace.join("\n")}" }
+    nil
+  rescue => e
+    Rails.logger.error {
+      format(
+        'Error connecting: %s, %s %s',
+        host.uri, e.message, e.backtrace.join("\n")
+      )
+    }
+    nil
+  ensure
+    con.close if block_given?
   end
   end
 
 
   def self.call con, req
   def self.call con, req
     resp = con.request req
     resp = con.request req
-    raise "request failure: " + resp.code unless resp.code != 200
+    unless resp.code != 200
+      raise Lxd::API::Exception(
+        "request failure: (#{resp.code}) #{resp.message}"
+      )
+    end
     JSON.parse resp.body
     JSON.parse resp.body
   end
   end
 
 
@@ -25,6 +45,17 @@ module Lxd::API
 
 
   def call req
   def call req
     handle_response(Lxd::API.call @con, req)
     handle_response(Lxd::API.call @con, req)
+  rescue Lxd::API::Exception => e
+    Rails.logger.error { "#{e.message} #{e.backtrace.join("\n")}" }
+    nil
+  rescue => e
+    Rails.logger.error {
+      format(
+        'Error connecting: %s, %s %s',
+        host.uri, e.message, e.backtrace.join("\n")
+      )
+    }
+    nil
   end
   end
 
 
   def get uri
   def get uri

+class Lxd::API::Exception < StandardError
+end
+# vim: set et ts=2 sw=2:

@@ -48,7 +48,11 @@ class Lxd::API::V1_0
     400 to 599: negative action result
     400 to 599: negative action result
     600 to 999: future use
     600 to 999: future use
     """
     """
-    raise "api error: (" + resp['error_code'].to_s + ") " + resp['error'] if resp['error_code'] and resp['error_code'] != 403
+    if resp['error_code'] and resp['error_code'] != 403
+      raise Lxd::API::Exception(
+        "api error: (#{resp['error_code']}) #{resp['error']}"
+      )
+    end
     resp['metadata']
     resp['metadata']
   end
   end
 end
 end

@@ -4,6 +4,7 @@ class Lxd::Certificate
   attr_accessor :api, :uri, :type, :certificate, :fingerprint
   attr_accessor :api, :uri, :type, :certificate, :fingerprint
 
 
   def self.all api
   def self.all api
+    return [] unless api
     api.certificates.map { |cert|
     api.certificates.map { |cert|
       Lxd::Certificate.new({api: api}.merge cert)
       Lxd::Certificate.new({api: api}.merge cert)
     }
     }

@@ -5,6 +5,7 @@ class Lxd::Config
 		:config, :environment, :public
 		:config, :environment, :public
 
 
 	def self.get api
 	def self.get api
+		return nil unless api
 		Lxd::Config.new({api: api}.merge api.config)
 		Lxd::Config.new({api: api}.merge api.config)
 	end
 	end
 
 

 <h1>Dashboard#index</h1>
 <h1>Dashboard#index</h1>
-<% Certificate.all.each do |cert| -%>
-<p>Fingerprint: <%= cert.cert_fpr %>&nbsp;
-Serial: <%= cert.cert.serial %></p>
-<% end -%>
+<p>
+<h2>Lex-deeit certificate</h2>
+<h3>Fingerprint</h3>
+<%= Certificate.get.cert_fpr %>
+<h3>Serial</h3>
+<%= Certificate.get.cert.serial %>
+</p>
 <hr/>
 <hr/>
 <% @hosts.each do |host| -%>
 <% @hosts.each do |host| -%>
-<p><%= host.lxd_config.inspect %></p>
-<% host.lxd_certificates.each do |certificate| -%>
-<p><%= certificate.fingerprint %></p>
-<% end -%>
+	<p>
+	<h2><%= host.name %></h2>
+	<h3>Url:</h3>
+	<%= host.uri %>
+	<h3>Connection status:</h3>
+	<% case -%>
+	<% when host.authenticated -%>
+		authenticated
+	<% when host.connected -%>
+		connected
+	<% else -%>
+		not connected
+	<% end -%>
+	<% if host.authenticated -%>
+		<h3>Config:</h3>
+		<%= host.lxd_config.config %>
+		<h3>Host known certificates</h3>
+		<ul>
+		<% host.authenticated and host.lxd_certificates.each do |certificate| -%>
+			<li><%= certificate.fingerprint %></li>
+		<% end -%>
+		</ul>
+	<% end -%>
+	</p>
+	<hr/>
 <% end -%>
 <% end -%>
+<!-- vim: set ts=2 sw=2: -->

@@ -19,14 +19,6 @@
     <%= f.label :uri %><br>
     <%= f.label :uri %><br>
     <%= f.text_field :uri %>
     <%= f.text_field :uri %>
   </div>
   </div>
-  <div class="field">
-    <%= f.label :password %><br>
-    <%= f.password_field :password %>
-  </div>
-  <div class="field">
-    <%= f.label :password_confirmation %><br>
-    <%= f.password_field :password_confirmation %>
-  </div>
   <div class="actions">
   <div class="actions">
     <%= f.submit %>
     <%= f.submit %>
   </div>
   </div>