Project

General

Profile

Bug #18115

Smart proxy - 1.14 - Windows - uninitialized constant error

Added by Jack Watroba over 2 years ago. Updated 11 months ago.

Status:
Closed
Priority:
Normal
Category:
Core
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

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 defaultClosed2016-07-28

Associated revisions

Revision 55583fb3 (diff)
Added by Dmitri Dolguikh over 2 years ago

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

History

#1 Updated by Dmitri Dolguikh over 2 years ago

  • Category set to Core

#2 Updated by Dominic Cleal over 2 years ago

#3 Updated by Dominic Cleal over 2 years 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 2 years ago

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

#5 Updated by The Foreman Bot over 2 years 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 2 years ago

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

#7 Updated by Ivan Necas over 2 years ago

  • Target version set to 1.11.1

Also available in: Atom PDF