rails_semantic_logger icon indicating copy to clipboard operation
rails_semantic_logger copied to clipboard

fix: avoid false positives during Resque detection

Open FraDim opened this issue 1 year ago • 0 comments
trafficstars

Issue

When combining Semantic Logger with other gems that use the Resque namespace and the host app does not use the resque gem, Semantic logger causes a failure during the boot up sequence.

Description of changes

The change makes Resque detection more robust in order to avoid a false positive detection and the subsequent failure in the host's app boot up sequence.

This also aligns the detection with other instances of detecting Resque:

https://github.com/reidmorrison/rails_semantic_logger/blob/eadda203513334f850bc0bc809c295d26642649c/lib/rails_semantic_logger/engine.rb#L108

and detecting other forking:

https://github.com/reidmorrison/rails_semantic_logger/blob/eadda203513334f850bc0bc809c295d26642649c/lib/rails_semantic_logger/engine.rb#L245


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

FraDim avatar May 09 '24 14:05 FraDim