diff --git a/.github/workflows/push_gem.yml b/.github/workflows/push_gem.yml new file mode 100644 index 0000000..73c515c --- /dev/null +++ b/.github/workflows/push_gem.yml @@ -0,0 +1,46 @@ +name: Publish gem to rubygems.org + +on: + push: + tags: + - 'v*' + +permissions: + contents: read + +jobs: + push: + if: github.repository == 'ruby/logger' + runs-on: ubuntu-latest + + environment: + name: rubygems.org + url: https://rubygems.org/gems/logger + + permissions: + contents: write + id-token: write + + steps: + - name: Harden Runner + uses: step-security/harden-runner@0080882f6c36860b6ba35c610c98ce87d4e2f26f # v2.10.2 + with: + egress-policy: audit + + - uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # v4.1.4 + + - name: Set up Ruby + uses: ruby/setup-ruby@a6e6f86333f0a2523ece813039b8b4be04560854 # v1.190.0 + with: + bundler-cache: true + ruby-version: ruby + + - name: Publish to RubyGems + uses: rubygems/release-gem@9e85cb11501bebc2ae661c1500176316d3987059 # v1.1.0 + + - name: Create GitHub release + run: | + tag_name="$(git describe --tags --abbrev=0)" + gh release create "${tag_name}" --verify-tag --generate-notes + env: + GITHUB_TOKEN: ${{ secrets.MATZBOT_GITHUB_WORKFLOW_TOKEN }} diff --git a/README.md b/README.md index 649684f..ceb1a21 100644 --- a/README.md +++ b/README.md @@ -101,4 +101,4 @@ Bug reports and pull requests are welcome on GitHub at https://github.com/ruby/l ## License -The gem is available as open source under the terms of the [BSD-2-Clause](LICENSE.txt). +The gem is available as open source under the terms of the [BSD-2-Clause](BSDL). diff --git a/Rakefile b/Rakefile index 35dd06e..d630ceb 100644 --- a/Rakefile +++ b/Rakefile @@ -14,7 +14,7 @@ require "rdoc/task" RDoc::Task.new do |doc| doc.main = "README.md" doc.title = "Logger -- Ruby Standard Logger" - doc.rdoc_files = FileList.new %w[README.md lib LICENSE.txt] + doc.rdoc_files = FileList.new %w[README.md lib BSDL COPYING] doc.rdoc_dir = "html" end diff --git a/lib/logger.rb b/lib/logger.rb index 4735209..8d6e0d7 100644 --- a/lib/logger.rb +++ b/lib/logger.rb @@ -381,7 +381,7 @@ class Logger # Logging severity threshold (e.g. Logger::INFO). def level - level_override[Fiber.current] || @level + level_override[level_key] || @level end # Sets the log level; returns +severity+. @@ -406,14 +406,14 @@ def level=(severity) # logger.debug { "Hello" } # end def with_level(severity) - prev, level_override[Fiber.current] = level, Severity.coerce(severity) + prev, level_override[level_key] = level, Severity.coerce(severity) begin yield ensure if prev - level_override[Fiber.current] = prev + level_override[level_key] = prev else - level_override.delete(Fiber.current) + level_override.delete(level_key) end end end @@ -751,6 +751,10 @@ def level_override @level_override ||= {} end + def level_key + Fiber.current + end + def format_message(severity, datetime, progname, msg) (@formatter || @default_formatter).call(severity, datetime, progname, msg) end diff --git a/lib/logger/version.rb b/lib/logger/version.rb index 2a0801b..15fc5bc 100644 --- a/lib/logger/version.rb +++ b/lib/logger/version.rb @@ -1,5 +1,5 @@ # frozen_string_literal: true class Logger - VERSION = "1.6.1" + VERSION = "1.6.2" end diff --git a/logger.gemspec b/logger.gemspec index 5e8232e..ed55111 100644 --- a/logger.gemspec +++ b/logger.gemspec @@ -15,7 +15,7 @@ Gem::Specification.new do |spec| spec.homepage = "https://github.com/ruby/logger" spec.licenses = ["Ruby", "BSD-2-Clause"] - spec.files = Dir.glob("lib/**/*.rb") + ["logger.gemspec"] + spec.files = Dir.glob("lib/**/*.rb") + ["logger.gemspec", "BSDL", "COPYING"] spec.require_paths = ["lib"] spec.required_ruby_version = ">= 2.5.0" diff --git a/test/logger/test_severity.rb b/test/logger/test_severity.rb index e1069c8..fb26939 100644 --- a/test/logger/test_severity.rb +++ b/test/logger/test_severity.rb @@ -26,7 +26,7 @@ def test_level_assignment end end - def test_thread_local_level + def test_fiber_local_level logger = Logger.new(nil) logger.level = INFO # default level other = Logger.new(nil) @@ -40,14 +40,50 @@ def test_thread_local_level logger.with_level(DEBUG) do # verify reentrancy assert_equal(logger.level, DEBUG) - Thread.new do + Fiber.new do assert_equal(logger.level, INFO) logger.with_level(:WARN) do assert_equal(other.level, ERROR) assert_equal(logger.level, WARN) end assert_equal(logger.level, INFO) - end.join + end.resume + + assert_equal(logger.level, DEBUG) + end + assert_equal(logger.level, WARN) + end + assert_equal(logger.level, INFO) + end + + def test_thread_local_level + subclass = Class.new(Logger) do + def level_key + Thread.current + end + end + + logger = subclass.new(nil) + logger.level = INFO # default level + other = subclass.new(nil) + other.level = ERROR # default level + + assert_equal(other.level, ERROR) + logger.with_level(:WARN) do + assert_equal(other.level, ERROR) + assert_equal(logger.level, WARN) + + logger.with_level(DEBUG) do # verify reentrancy + assert_equal(logger.level, DEBUG) + + Fiber.new do + assert_equal(logger.level, DEBUG) + logger.with_level(:WARN) do + assert_equal(other.level, ERROR) + assert_equal(logger.level, WARN) + end + assert_equal(logger.level, DEBUG) + end.resume assert_equal(logger.level, DEBUG) end