Bug #18115

Smart proxy - 1.14 - Windows - uninitialized constant error

Added by Jack Watroba over 1 year ago. Updated 7 days ago.

Assignee:Dmitri Dolguikh
Target version:1.14.1
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link: Found in Releases:
Pull request:https://github.com/theforeman/smart-proxy/pull/495


When trying to upgrade from 1.13 to 1.14 smart proxy on Windows 2008R2, I receive the following error when trying to run the proxy:

C:\smart-proxy>bundle exec ruby C:\smart-proxy\bin\smart-proxy
C:/smart-proxy/lib/proxy/log.rb:35:in `<class:LoggerFactory>': uninitialized con
stant Proxy::LoggerFactory::Syslog (NameError)
from C:/smart-proxy/lib/proxy/log.rb:22:in `<module:Proxy>'
from C:/smart-proxy/lib/proxy/log.rb:15:in `<top (required)>'
from C:/smart-proxy/lib/launcher.rb:1:in `require'
from C:/smart-proxy/lib/launcher.rb:1:in `<top (required)>'
from C:/smart-proxy/lib/smart_proxy_main.rb:4:in `require'
from C:/smart-proxy/lib/smart_proxy_main.rb:4:in `<top (required)>'
from C:/smart-proxy/bin/smart-proxy:5:in `require'
from C:/smart-proxy/bin/smart-proxy:5:in `<main>'

I've tried Ruby 2.3.3. and ruby 2.2.6 and both have this issue. Smart proxy version 1.13 works fine on this machine.

Related issues

Related to Smart Proxy - Feature #15874: Log X-Request-ID by default Closed 07/28/2016

Associated revisions

Revision 55583fb3
Added by Dmitri Dolguikh over 1 year ago

Fixes #18115 - Logger is initialized when Syslog isn't available.


#1 Updated by Dmitri Dolguikh over 1 year ago

  • Category set to Core

#2 Updated by Dominic Cleal over 1 year ago

#3 Updated by Dominic Cleal over 1 year ago

  • Legacy Backlogs Release (now unused) set to 210

The constant is referenced without checking if it's defined, or if the syslog library loaded successfully. This was added in #15874.

#4 Updated by Dmitri Dolguikh over 1 year ago

  • Status changed from New to Assigned
  • Assignee set to Dmitri Dolguikh

#5 Updated by The Foreman Bot over 1 year ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/495 added

#6 Updated by Anonymous over 1 year ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#7 Updated by Ivan Necas over 1 year ago

  • Target version set to 1.11.1

Also available in: Atom PDF