title_grabber 0.3.3 → 0.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 92a1bcb0847898f17c59d593ab9cba0399698ca4fb895a04e0f640ba13cabb23
-  data.tar.gz: 4a207b40f92fafbe8b6aff81fe982c7e572894bbb7b36950d80c515d741c920a
+  metadata.gz: 8a8b57f801507ef8c856ebd741b247c30e08472b5df3be45d30e6fd7fa3c0092
+  data.tar.gz: 75ff595060fe8dd71a669ebadcf38f040d5c47492a5c6ea770e893832493ddff
 SHA512:
-  metadata.gz: f908ad1996a3228474d75c73ae519a14cfaa3455973f4cac7a33547d5ab36ab6c39ff80837c205cb746bb74b9baca63e2ffb4fb72394cd7801a1c56b2ac162fc
-  data.tar.gz: 54cf0f2d8d67341964d7d8e5487eb89ef1672f7d96d4c92cabbad6fd2c1d5ffad4d6116d0793ee26722600268704dca840707f8c9300a9ebbb22b8be02ba0473
+  metadata.gz: 661dd87e4e60dcfd4a66168799f3b07babf5c21a890b2e61afb99b527b1e15cb7ebd875a60f39bd12cc2f06ef286b603efb89c4908f461606fa3c60a09ad5db6
+  data.tar.gz: b6b1152856c43702082a788a1a056ccc2cef93101faada8890eda27b949d424cf3b9a83991fd91e07733f2c860003b26644510ae9635ea5c6336a1990f997445

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    title_grabber (0.3.3)
+    title_grabber (0.3.4)
       http (~> 4.1)
       oga (~> 2.15)
 

data/lib/http_helper.rb

@@ -11,7 +11,7 @@ module HTTPHelper
   include TextHelper
 
   def open_w_timeout(url, write_to:, connect_to:, read_to:, max_retries:)
-    logger.info "[Thread: ##{Thread.current[:id]}] GET #{url}"
+    logger.info "[#{Thread.current.name}] GET #{url}"
     retries = 0
 
     begin
@@ -30,16 +30,16 @@ module HTTPHelper
 
         if retries <= max_retries
           rest_time = rand(REST_INTERVAL)
-          logger.warn "[Thread: ##{Thread.current[:id]}] URL: #{url} [#{msg}] - Going to sleep for #{rest_time.round(1)} secs - Retry ##{retries}"
+          logger.warn "[#{Thread.current.name}] URL: #{url} [#{msg}] - Going to sleep for #{rest_time.round(1)} secs - Retry ##{retries}"
           sleep(rest_time)
           retry
         else
-          logger.error "[Thread: ##{Thread.current[:id]}] URL: #{url} [#{msg}]"
+          logger.error "[#{Thread.current.name}] URL: #{url} [#{msg}]"
           nil
         end
       end
     rescue => err
-      logger.error "[Thread: ##{Thread.current[:id]}] URL: #{url} [#{err.message}]"
+      logger.error "[#{Thread.current.name}] URL: #{url} [#{err.message}]"
       nil
     else
       utf8_encode(body)

data/lib/title_grabber.rb

@@ -58,8 +58,6 @@ module TitleGrabber
     end
 
     def call
-      install_at_exit_handler
-
       queue = Queue.new
       CSV.open(tmp_path, "w", force_quotes: true) do |csv|
         csv << HEADERS
@@ -81,7 +79,7 @@ module TitleGrabber
         thr_cnt = [max_threads, queue.size].min
         threads = 1.upto(thr_cnt).map.with_index { |_, i|
           Thread.new(i) do |j|
-            Thread.current[:id] = i + 1
+            Thread.current.name = "Thread ##{i + 1}"
 
             url = begin
                     queue.pop(true)
@@ -92,7 +90,7 @@ module TitleGrabber
                 doc = begin
                         Oga.parse_html(html)
                       rescue ArgumentError, LL::ParserError => err
-                        logger.error "[Thread: ##{Thread.current[:id]}] Unable to parse HTML from URL '#{url}' - #{err.message}"
+                        logger.error "[#{Thread.current.name}] Unable to parse HTML from URL '#{url}' - #{err.message}"
                         nil
                       end
 
@@ -114,20 +112,16 @@ module TitleGrabber
           end
         }.each(&:join)
       end
+    ensure
+      if tmp_path.size?
+        FileUtils.mv(tmp_path, out_path)
+      else
+        tmp_path.unlink if tmp_path.exist?
+      end
     end
 
     private
 
-    def install_at_exit_handler
-      at_exit do
-        if tmp_path.size?
-          FileUtils.mv(tmp_path, out_path)
-        else
-          tmp_path.unlink if tmp_path.exist?
-        end
-      end
-    end
-
     def processed_urls
       @processed_urls ||= begin
                             urls = {}

data/lib/title_grabber/version.rb

@@ -1,3 +1,3 @@
 module TitleGrabber
-  VERSION = "0.3.3"
+  VERSION = "0.3.4"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: title_grabber
 version: !ruby/object:Gem::Version
-  version: 0.3.3
+  version: 0.3.4
 platform: ruby
 authors:
 - Cristian Rasch
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-04-06 00:00:00.000000000 Z
+date: 2019-04-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: http