Add error message to wopi discovery

Fix ambigious open call
This commit is contained in:
Jure Grabnar 2016-09-30 09:59:03 +02:00
parent ff47f29dc2
commit a6ebff06a7
2 changed files with 5 additions and 4 deletions

View file

@ -338,7 +338,7 @@ class Asset < ActiveRecord::Base
def favicon_url(action)
file_ext = file_file_name.split('.').last
action = get_action(file_ext, action)
action.wopi_app.icon if action.wopi_app
action.wopi_app.icon if action.try(:wopi_app)
end
# locked?, lock_asset and refresh_lock rely on the asset

View file

@ -33,7 +33,7 @@ module WopiUtil
Rails.logger.warn 'Initializing discovery'
discovery.destroy if discovery
@doc = Nokogiri::XML(open(ENV['WOPI_DISCOVERY_URL']))
@doc = Nokogiri::XML(Kernel.open(ENV['WOPI_DISCOVERY_URL']))
discovery = WopiDiscovery.new
discovery.expires = Time.now.to_i + DISCOVERY_TTL
@ -65,8 +65,9 @@ module WopiUtil
end
end
discovery
rescue
Rails.logger.warn 'Initialization failed'
rescue => e
Rails.logger.warn 'WOPI: initialization failed: ' + e.message
e.backtrace.each { |line| Rails.logger.error line }
discovery = WopiDiscovery.first
discovery.destroy if discovery
end