Release 0.6.1 (#105)

Fix `BugzillaHelper#bug_id_from_string` failure for string containing url

.rubocop.yml

@@ -7,3 +7,6 @@ inherit_from: .rubocop_todo.yml
 
 AllCops:
   NewCops: enable
+
+Metrics/BlockLength:
+  Max: 27

.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2020-08-07 08:39:39 UTC using RuboCop version 0.89.0.
+# on 2021-04-08 14:51:36 UTC using RuboCop version 1.12.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new

CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## master (unreleased)
 
+## 0.6.1 (2021-04-08)
+
+### Fixes
+
+* Fix `BugzillaHelper#bug_id_from_string` failure for string containing url
+
 ## 0.6.0 (2021-03-15)
 
 ### New Features

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    onlyoffice_bugzilla_helper (0.6.0)
+    onlyoffice_bugzilla_helper (0.6.1)
 
 GEM
   remote: https://rubygems.org/

lib/onlyoffice_bugzilla_helper.rb

@@ -34,8 +34,6 @@ def initialize(bugzilla_url: 'https://bugzilla.onlyoffice.com',
     # @param string [String] string for error
     # @return [Integer, Nil] result of bug id from url
     def bug_id_from_string(string)
-      return nil unless string&.match?(URI::DEFAULT_PARSER.make_regexp)
-
       uri = URI.parse(string)
       return nil unless uri.host == url.host
       return nil unless uri.path == @show_bug_path
@@ -44,6 +42,8 @@ def bug_id_from_string(string)
       return nil if id.zero?
 
       id
+    rescue URI::InvalidURIError
+      nil
     end
 
     # Read access token from file system

lib/onlyoffice_bugzilla_helper/version.rb

@@ -2,5 +2,5 @@
 
 module OnlyofficeBugzillaHelper
   # [String] version of Gem
-  VERSION = '0.6.0'
+  VERSION = '0.6.1'
 end

spec/onlyoffice_bugzilla_helper/bugzilla_helper_bug_id_from_string_spec.rb

@@ -11,6 +11,11 @@
     expect(bugzilla.bug_id_from_string(string)).to eq(32_296)
   end
 
+  it 'bug_id_from_string that contains correct url but contains extra data' do
+    string = "Junk string #{bugzilla.url}/test?id=32296"
+    expect(bugzilla.bug_id_from_string(string)).to be_nil
+  end
+
   it 'bug_id_from_string that looks like correct bugzilla bug url' do
     string = "#{bugzilla.url}/test?id=32296"
     expect(bugzilla.bug_id_from_string(string)).to be_nil