github: add Ruby 3.4 for CI (#25)

* github: add Ruby 3.4 for CI

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>

* Fix unexpected postposition resque and if behavior

Since Ruby 3.4, postposition resque and if causes
unexpected behavior.
To keep compatibility with previous versions, use if condition.

See https://bugs.ruby-lang.org/issues/21132

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>

---------

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>

Author: kenhys

.github/workflows/linux-test.yaml

@@ -13,7 +13,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: ['3.3', '3.2', '3.1', '3.0', '2.7']
+        ruby: ['3.4', '3.3', '3.2', '3.1', '3.0', '2.7']
         os: [ubuntu-latest]
         experimental: [false]
         include:

.github/workflows/macos-test.yaml

@@ -12,7 +12,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: ['3.3', '3.2', '3.1', '3.0', '2.7']
+        ruby: ['3.4', '3.3', '3.2', '3.1', '3.0', '2.7']
         os: [macos-latest]
 
     name: Ruby ${{ matrix.ruby }} on ${{ matrix.os }}

.github/workflows/windows-test.yaml

@@ -12,7 +12,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: ['3.3', '3.2', '3.1', '3.0', '2.7']
+        ruby: ['3.4', '3.3', '3.2', '3.1', '3.0', '2.7']
         os:
           - windows-latest
     name: Ruby ${{ matrix.ruby }} on ${{ matrix.os }}

lib/fluent/plugin/parser_winevt_xml.rb

@@ -20,7 +20,9 @@ def MAKELONG(low, high)
     end
 
     def event_id(system_elem)
-      return (system_elem/'EventID').text rescue nil if @preserve_qualifiers
+      if @preserve_qualifiers
+        return (system_elem/'EventID').text rescue nil
+      end
 
       qualifiers = (system_elem/'EventID').attribute("Qualifiers").text rescue nil
       if qualifiers