browser 2.5.3 → 5.3.1

Sign up to get free protection for your applications and to get access to all the features.
Files changed (147) hide show
  1. checksums.yaml +4 -4
  2. data/.github/ISSUE_TEMPLATE.md +1 -1
  3. data/.github/workflows/tests.yml +57 -0
  4. data/.gitignore +1 -0
  5. data/.prettierignore +1 -0
  6. data/.rubocop.yml +22 -107
  7. data/CHANGELOG.md +170 -6
  8. data/FUNDING.yml +3 -0
  9. data/Gemfile +2 -0
  10. data/README.md +139 -89
  11. data/Rakefile +11 -2
  12. data/bot_exceptions.yml +2 -0
  13. data/bots.yml +302 -265
  14. data/browser.gemspec +12 -7
  15. data/gemfiles/rails6_0.gemfile +6 -0
  16. data/gemfiles/rails6_1.gemfile +6 -0
  17. data/lib/browser/accept_language.rb +6 -9
  18. data/lib/browser/action_controller.rb +1 -3
  19. data/lib/browser/aliases.rb +5 -5
  20. data/lib/browser/alipay.rb +2 -2
  21. data/lib/browser/base.rb +101 -19
  22. data/lib/browser/blackberry.rb +3 -3
  23. data/lib/browser/bot/empty_user_agent_matcher.rb +11 -0
  24. data/lib/browser/bot/keyword_matcher.rb +11 -0
  25. data/lib/browser/bot/known_bots_matcher.rb +11 -0
  26. data/lib/browser/bot.rb +39 -27
  27. data/lib/browser/browser.rb +65 -54
  28. data/lib/browser/chrome.rb +18 -5
  29. data/lib/browser/detect_version.rb +5 -5
  30. data/lib/browser/device/android.rb +20 -0
  31. data/lib/browser/device/blackberry_playbook.rb +1 -1
  32. data/lib/browser/device/ipad.rb +1 -1
  33. data/lib/browser/device/iphone.rb +1 -1
  34. data/lib/browser/device/ipod_touch.rb +1 -1
  35. data/lib/browser/device/kindle.rb +1 -1
  36. data/lib/browser/device/kindle_fire.rb +1 -1
  37. data/lib/browser/device/playstation3.rb +1 -1
  38. data/lib/browser/device/playstation4.rb +1 -1
  39. data/lib/browser/device/psp.rb +1 -1
  40. data/lib/browser/device/psvita.rb +1 -1
  41. data/lib/browser/device/samsung.rb +33 -0
  42. data/lib/browser/device/surface.rb +2 -4
  43. data/lib/browser/device/switch.rb +19 -0
  44. data/lib/browser/device/tv.rb +1 -1
  45. data/lib/browser/device/unknown.rb +1 -1
  46. data/lib/browser/device/wii.rb +1 -1
  47. data/lib/browser/device/wiiu.rb +1 -1
  48. data/lib/browser/device/xbox_360.rb +1 -1
  49. data/lib/browser/device/xbox_one.rb +1 -1
  50. data/lib/browser/device.rb +47 -30
  51. data/lib/browser/duck_duck_go.rb +22 -0
  52. data/lib/browser/edge.rb +6 -2
  53. data/lib/browser/electron.rb +2 -2
  54. data/lib/browser/facebook.rb +4 -2
  55. data/lib/browser/firefox.rb +2 -2
  56. data/lib/browser/google_search_app.rb +21 -0
  57. data/lib/browser/huawei_browser.rb +21 -0
  58. data/lib/browser/instagram.rb +21 -0
  59. data/lib/browser/internet_explorer.rb +9 -10
  60. data/lib/browser/maxthon.rb +21 -0
  61. data/lib/browser/meta/base.rb +0 -1
  62. data/lib/browser/meta/generic_browser.rb +1 -3
  63. data/lib/browser/meta.rb +12 -13
  64. data/lib/browser/micro_messenger.rb +2 -2
  65. data/lib/browser/middleware/context/additions.rb +1 -1
  66. data/lib/browser/middleware.rb +4 -3
  67. data/lib/browser/miui_browser.rb +21 -0
  68. data/lib/browser/nokia.rb +2 -2
  69. data/lib/browser/opera.rb +2 -2
  70. data/lib/browser/otter.rb +2 -2
  71. data/lib/browser/phantom_js.rb +2 -2
  72. data/lib/browser/platform/adobe_air.rb +2 -2
  73. data/lib/browser/platform/android.rb +1 -1
  74. data/lib/browser/platform/base.rb +3 -2
  75. data/lib/browser/platform/blackberry.rb +2 -2
  76. data/lib/browser/platform/chrome_os.rb +1 -1
  77. data/lib/browser/platform/firefox_os.rb +1 -1
  78. data/lib/browser/platform/ios.rb +17 -4
  79. data/lib/browser/platform/kai_os.rb +23 -0
  80. data/lib/browser/platform/linux.rb +1 -1
  81. data/lib/browser/platform/mac.rb +5 -3
  82. data/lib/browser/platform/{other.rb → unknown.rb} +3 -3
  83. data/lib/browser/platform/windows.rb +2 -2
  84. data/lib/browser/platform/windows_mobile.rb +1 -1
  85. data/lib/browser/platform/windows_phone.rb +1 -1
  86. data/lib/browser/platform.rb +37 -28
  87. data/lib/browser/qq.rb +5 -5
  88. data/lib/browser/rails.rb +7 -5
  89. data/lib/browser/safari.rb +19 -4
  90. data/lib/browser/samsung_browser.rb +21 -0
  91. data/lib/browser/snapchat.rb +21 -0
  92. data/lib/browser/sougou_browser.rb +24 -0
  93. data/lib/browser/sputnik.rb +21 -0
  94. data/lib/browser/uc_browser.rb +2 -2
  95. data/lib/browser/{generic.rb → unknown.rb} +6 -8
  96. data/lib/browser/version.rb +1 -1
  97. data/lib/browser/weibo.rb +2 -2
  98. data/lib/browser/yandex.rb +21 -0
  99. data/lib/browser.rb +2 -2
  100. data/samsung.yml +138 -0
  101. data/search_engines.yml +2 -2
  102. data/test/browser_test.rb +42 -10
  103. data/test/rails_test.rb +10 -0
  104. data/test/sample_app.rb +14 -0
  105. data/test/test_helper.rb +9 -1
  106. data/test/ua.yml +147 -109
  107. data/test/ua_bots.yml +95 -60
  108. data/test/ua_search_engines.yml +7 -6
  109. data/test/unit/accept_language_test.rb +16 -0
  110. data/test/unit/adobe_air_test.rb +1 -1
  111. data/test/unit/alipay_test.rb +6 -0
  112. data/test/unit/blackberry_test.rb +0 -6
  113. data/test/unit/bots_test.rb +37 -36
  114. data/test/unit/chrome_test.rb +8 -19
  115. data/test/unit/console_test.rb +2 -2
  116. data/test/unit/device_test.rb +60 -3
  117. data/test/unit/duck_duck_go_test.rb +37 -0
  118. data/test/unit/edge_test.rb +49 -5
  119. data/test/unit/facebook_test.rb +20 -0
  120. data/test/unit/firefox_test.rb +0 -3
  121. data/test/unit/google_search_app_test.rb +54 -0
  122. data/test/unit/huawei_browser_test.rb +25 -0
  123. data/test/unit/instagram_test.rb +30 -0
  124. data/test/unit/internet_explorer_test.rb +0 -12
  125. data/test/unit/ios_test.rb +7 -5
  126. data/test/unit/kai_os_test.rb +31 -0
  127. data/test/unit/kindle_test.rb +0 -2
  128. data/test/unit/maxthon_test.rb +25 -0
  129. data/test/unit/meta_test.rb +10 -2
  130. data/test/unit/micro_messenger_test.rb +21 -6
  131. data/test/unit/miui_browser_test.rb +25 -0
  132. data/test/unit/opera_test.rb +1 -2
  133. data/test/unit/platform_test.rb +34 -8
  134. data/test/unit/qq_test.rb +12 -0
  135. data/test/unit/safari_test.rb +12 -7
  136. data/test/unit/samsung_browser_test.rb +23 -0
  137. data/test/unit/snapchat_test.rb +40 -0
  138. data/test/unit/sougou_browser_test.rb +41 -0
  139. data/test/unit/sputnik_test.rb +22 -0
  140. data/test/unit/yandex_test.rb +37 -0
  141. metadata +83 -27
  142. data/.bundle/config +0 -2
  143. data/.travis.yml +0 -16
  144. data/bin/rake +0 -17
  145. data/gemfiles/rails4.gemfile +0 -4
  146. data/gemfiles/rails5.gemfile +0 -4
  147. data/lib/browser/meta/modern.rb +0 -11
data/README.md CHANGED
@@ -1,8 +1,7 @@
1
1
  # Browser
2
2
 
3
- [![Travis-CI](https://travis-ci.org/fnando/browser.png)](https://travis-ci.org/fnando/browser)
3
+ [![Tests](https://github.com/fnando/browser/workflows/Tests/badge.svg)](https://github.com/fnando/browser)
4
4
  [![Code Climate](https://codeclimate.com/github/fnando/browser/badges/gpa.svg)](https://codeclimate.com/github/fnando/browser)
5
- [![Test Coverage](https://codeclimate.com/github/fnando/browser/badges/coverage.svg)](https://codeclimate.com/github/fnando/browser/coverage)
6
5
  [![Gem](https://img.shields.io/gem/v/browser.svg)](https://rubygems.org/gems/browser)
7
6
  [![Gem](https://img.shields.io/gem/dt/browser.svg)](https://rubygems.org/gems/browser)
8
7
 
@@ -25,6 +24,7 @@ browser = Browser.new("Some User Agent", accept_language: "en-us")
25
24
  browser.bot?
26
25
  browser.chrome?
27
26
  browser.core_media?
27
+ browser.duck_duck_go?
28
28
  browser.edge? # Newest MS browser
29
29
  browser.electron? # Electron Framework
30
30
  browser.firefox?
@@ -33,8 +33,8 @@ browser.ie?
33
33
  browser.ie?(6) # detect specific IE version
34
34
  browser.ie?([">8", "<10"]) # detect specific IE (IE9).
35
35
  browser.known? # has the browser been successfully detected?
36
+ browser.unknown? # the browser wasn't detected.
36
37
  browser.meta # an array with several attributes
37
- browser.modern? # Webkit, Firefox 17+, IE 9+ and Opera 12+
38
38
  browser.name # readable browser name
39
39
  browser.nokia?
40
40
  browser.opera?
@@ -43,24 +43,32 @@ browser.phantom_js?
43
43
  browser.quicktime?
44
44
  browser.safari?
45
45
  browser.safari_webapp_mode?
46
+ browser.samsung_browser?
46
47
  browser.to_s # the meta info joined by space
47
48
  browser.uc_browser?
48
49
  browser.version # major version number
49
50
  browser.webkit?
50
51
  browser.webkit_full_version
51
52
  browser.yandex?
52
- browser.wechat? # detect the MicroMessenger(WeChat)
53
- browser.weibo? # detect Weibo embedded browser (Sina Weibo)
53
+ browser.wechat?
54
+ browser.qq?
55
+ browser.weibo?
56
+ browser.yandex?
57
+ browser.sputnik?
58
+ browser.sougou_browser?
54
59
 
55
60
  # Get bot info
56
61
  browser.bot.name
57
62
  browser.bot.search_engine?
58
63
  browser.bot?
64
+ browser.bot.why? # shows which matcher detected this user agent as a bot.
65
+ Browser::Bot.why?(ua)
59
66
 
60
67
  # Get device info
61
68
  browser.device
62
69
  browser.device.id
63
70
  browser.device.name
71
+ browser.device.unknown?
64
72
  browser.device.blackberry_playbook?
65
73
  browser.device.console?
66
74
  browser.device.ipad?
@@ -81,6 +89,8 @@ browser.device.tv?
81
89
  browser.device.vita?
82
90
  browser.device.wii?
83
91
  browser.device.wiiu?
92
+ browser.device.samsung?
93
+ browser.device.switch?
84
94
  browser.device.xbox?
85
95
  browser.device.xbox_360?
86
96
  browser.device.xbox_one?
@@ -105,7 +115,7 @@ browser.platform.ios_app? # detect webview in an iOS app
105
115
  browser.platform.ios_webview? # alias for ios_app?
106
116
  browser.platform.linux?
107
117
  browser.platform.mac?
108
- browser.platform.other?
118
+ browser.platform.unknown?
109
119
  browser.platform.windows10?
110
120
  browser.platform.windows7?
111
121
  browser.platform.windows8?
@@ -120,11 +130,14 @@ browser.platform.windows_wow64?
120
130
  browser.platform.windows_x64?
121
131
  browser.platform.windows_x64_inclusive?
122
132
  browser.platform.windows_xp?
133
+ browser.platform.kai_os?
123
134
  ```
124
135
 
125
136
  ### Aliases
126
137
 
127
- To add aliases like `mobile?` and `tablet?` to the base object (e.g `browser.mobile?`), require the `browser/aliases` file and extend the Browser::Base object like the following:
138
+ To add aliases like `mobile?` and `tablet?` to the base object (e.g
139
+ `browser.mobile?`), require the `browser/aliases` file and extend the
140
+ Browser::Base object like the following:
128
141
 
129
142
  ```ruby
130
143
  require "browser/aliases"
@@ -136,27 +149,35 @@ browser.mobile? #=> false
136
149
 
137
150
  ### What's being detected?
138
151
 
139
- - For a list of platform detections, check [lib/browser/platform.rb](https://github.com/fnando/browser/blob/master/lib/browser/platform.rb)
140
- - For a list of device detections, check [lib/browser/device.rb](https://github.com/fnando/browser/blob/master/lib/browser/device.rb)
141
- - For a list of bot detections, check [bots.yml](https://github.com/fnando/browser/blob/master/bots.yml)
142
-
143
- ### What defines a modern browser?
152
+ - For a list of platform detections, check
153
+ [lib/browser/platform.rb](https://github.com/fnando/browser/blob/master/lib/browser/platform.rb)
154
+ - For a list of device detections, check
155
+ [lib/browser/device.rb](https://github.com/fnando/browser/blob/master/lib/browser/device.rb)
156
+ - For a list of bot detections, check
157
+ [bots.yml](https://github.com/fnando/browser/blob/master/bots.yml)
144
158
 
145
- The current rules that define a modern browser are pretty loose:
159
+ ### Detecting modern browsers
146
160
 
147
- * Webkit
148
- * IE9+
149
- * Microsoft Edge
150
- * Firefox 17+
151
- * Firefox Tablet 14+
152
- * Opera 12+
153
-
154
- You can define your own rules. A rule must be a proc/lambda or any object that implements the method === and accepts the browser object. To redefine all rules, clear the existing rules before adding your own.
161
+ To detect whether a browser can be considered as modern or not, create a method
162
+ that abstracts your versioning constraints. The following example will consider
163
+ any of the following browsers as a modern:
155
164
 
156
165
  ```ruby
157
- # Only Chrome Canary is considered modern.
158
- Browser.modern_rules.clear
159
- Browser.modern_rules << -> b { b.chrome? && b.version.to_i >= 37 }
166
+ # Expects an Browser instance,
167
+ # like in `Browser.new(user_agent, accept_language: language)`.
168
+ def modern_browser?(browser)
169
+ [
170
+ browser.chrome?(">= 65"),
171
+ browser.safari?(">= 10"),
172
+ browser.firefox?(">= 52"),
173
+ browser.ie?(">= 11") && !browser.compatibility_view?,
174
+ browser.edge?(">= 15"),
175
+ browser.opera?(">= 50"),
176
+ browser.facebook?
177
+ && browser.safari_webapp_mode?
178
+ && browser.webkit_full_version.to_i >= 602
179
+ ].any?
180
+ end
160
181
  ```
161
182
 
162
183
  ### Rails integration
@@ -167,7 +188,8 @@ Just add it to the Gemfile.
167
188
  gem "browser"
168
189
  ```
169
190
 
170
- This adds a helper method called `browser`, that inspects your current user agent.
191
+ This adds a helper method called `browser`, that inspects your current user
192
+ agent.
171
193
 
172
194
  ```erb
173
195
  <% if browser.ie?(6) %>
@@ -175,7 +197,8 @@ This adds a helper method called `browser`, that inspects your current user agen
175
197
  <% end %>
176
198
  ```
177
199
 
178
- If you want to use Browser on your Rails app but don't want to taint your controller, use the following line on your Gemfile:
200
+ If you want to use Browser on your Rails app but don't want to taint your
201
+ controller, use the following line on your Gemfile:
179
202
 
180
203
  ```ruby
181
204
  gem "browser", require: "browser/browser"
@@ -183,7 +206,8 @@ gem "browser", require: "browser/browser"
183
206
 
184
207
  ### Accept Language
185
208
 
186
- Parses the accept-language header from an HTTP request and produces an array of language objects sorted by quality.
209
+ Parses the accept-language header from an HTTP request and produces an array of
210
+ language objects sorted by quality.
187
211
 
188
212
  ```ruby
189
213
  browser = Browser.new("Some User Agent", accept_language: "en-us")
@@ -209,16 +233,22 @@ language.name
209
233
  #=> "English/United States"
210
234
  ```
211
235
 
212
- Result is always sorted in quality order from highest -> lowest. As per the HTTP spec:
236
+ Result is always sorted in quality order from highest to lowest. As per the HTTP
237
+ spec:
213
238
 
214
239
  - omitting the quality value implies 1.0.
215
240
  - quality value equal to zero means that is not accepted by the client.
216
241
 
217
242
  ### Internet Explorer
218
243
 
219
- Internet Explorer has a compatibility view mode that allows newer versions (IE8+) to run as an older version. Browser will always return the navigator version, ignoring the compatibility view version, when defined. If you need to get the engine's version, you have to use `Browser#msie_version` and `Browser#msie_full_version`.
244
+ Internet Explorer has a compatibility view mode that allows newer versions
245
+ (IE8+) to run as an older version. Browser will always return the navigator
246
+ version, ignoring the compatibility view version, when defined. If you need to
247
+ get the engine's version, you have to use `Browser#msie_version` and
248
+ `Browser#msie_full_version`.
220
249
 
221
- So, let's say an user activates compatibility view in a IE11 browser. This is what you'll get:
250
+ So, let's say an user activates compatibility view in a IE11 browser. This is
251
+ what you'll get:
222
252
 
223
253
  ```ruby
224
254
  browser.version
@@ -235,16 +265,16 @@ browser.msie_full_version
235
265
 
236
266
  browser.compatibility_view?
237
267
  #=> true
238
-
239
- browser.modern?
240
- #=> false
241
268
  ```
242
269
 
243
- This behavior changed in `v1.0.0`; previously there wasn't a way of getting the real browser version.
270
+ This behavior changed in `v1.0.0`; previously there wasn't a way of getting the
271
+ real browser version.
244
272
 
245
273
  ### Safari
246
274
 
247
- iOS webviews and web apps aren't detect as Safari anymore, so be aware of that if that's your case. You can use a combination of platform and webkit detection to do whatever you want.
275
+ iOS webviews and web apps aren't detected as Safari anymore, so be aware of that
276
+ if that's your case. You can use a combination of platform and webkit detection
277
+ to do whatever you want.
248
278
 
249
279
  ```ruby
250
280
  # iPad's Safari running as web app mode.
@@ -262,62 +292,78 @@ browser.platform.ios?
262
292
 
263
293
  ### Bots
264
294
 
265
- Browser used to detect empty user agents as bots, but this behavior has changed. If you want to bring this detection back, you can activate it through the following call:
295
+ The bot detection is quite aggressive. Anything that matches at least one of the
296
+ following requirements will be considered a bot.
297
+
298
+ - Empty user agent string
299
+ - User agent that matches `/crawl|fetch|search|monitoring|spider|bot/`
300
+ - Any known bot listed under
301
+ [bots.yml](https://github.com/fnando/browser/blob/master/bots.yml)
302
+
303
+ To add custom matchers, you can add a callable object to
304
+ `Browser::Bot.matchers`. The following example matches everything that has a
305
+ `externalhit` substring on it. The bot name will always be `General Bot`.
266
306
 
267
307
  ```ruby
268
- Browser::Bot.detect_empty_ua!
308
+ Browser::Bot.matchers << ->(ua, _browser) { ua.match?(/externalhit/i) }
269
309
  ```
270
310
 
271
- ### Middleware
311
+ To clear all matchers, including the ones that are bundled, use
312
+ `Browser::Bot.matchers.clear`. You can re-add built-in matchers by doing the
313
+ following:
272
314
 
273
- You can use the `Browser::Middleware` to redirect user agents.
315
+ ```ruby
316
+ Browser::Bot.matchers += Browser::Bot.default_matchers
317
+ ```
318
+
319
+ To restore v2's bot detection, remove the following matchers:
274
320
 
275
321
  ```ruby
276
- use Browser::Middleware do
277
- redirect_to "/upgrade" unless browser.modern?
278
- end
322
+ Browser::Bot.matchers.delete(Browser::Bot::KeywordMatcher)
323
+ Browser::Bot.matchers.delete(Browser::Bot::EmptyUserAgentMatcher)
279
324
  ```
280
325
 
281
- If you're using Rails, you can use the route helper methods. Just add something like the following to a initializer file (`config/initializers/browser.rb`).
326
+ ### Middleware
327
+
328
+ You can use the `Browser::Middleware` to redirect user agents.
282
329
 
283
330
  ```ruby
284
- Rails.configuration.middleware.use Browser::Middleware do
285
- redirect_to upgrade_path unless browser.modern?
331
+ use Browser::Middleware do
332
+ redirect_to "/upgrade" if browser.ie?
286
333
  end
287
334
  ```
288
335
 
289
- Notice that you can have multiple conditionals.
336
+ If you're using Rails, you can use the route helper methods. Just add something
337
+ like the following to a initializer file (`config/initializers/browser.rb`).
290
338
 
291
339
  ```ruby
292
340
  Rails.configuration.middleware.use Browser::Middleware do
293
- next if browser.bot.search_engine?
294
- redirect_to upgrade_path(browser: "oldie") if browser.ie? && !browser.modern?
295
- redirect_to upgrade_path(browser: "oldfx") if browser.firefox? && !browser.modern?
341
+ redirect_to upgrade_path if browser.ie?
296
342
  end
297
343
  ```
298
344
 
299
- If you need access to the `Rack::Request` object (e.g. to exclude a path), you can do so with `request`.
345
+ If you need access to the `Rack::Request` object (e.g. to exclude a path), you
346
+ can do so with `request`.
300
347
 
301
348
  ```ruby
302
349
  Rails.configuration.middleware.use Browser::Middleware do
303
- redirect_to upgrade_path unless browser.modern? || request.env["PATH_INFO"] == "/exclude_me"
350
+ redirect_to upgrade_path if browser.ie? && request.env["PATH_INFO"] != "/exclude_me"
304
351
  end
305
352
  ```
306
353
 
307
- ### Migrating to v2
308
-
309
- #### Troubleshooting
310
-
311
- ##### `TypeError: no implicit conversion of Hash into String`
354
+ ### Restrictions
312
355
 
313
- The class constructor now has a different signature. Change the instantiation from `Browser.new(options)` to `Browser.new(ua, options)`, where:
356
+ - User agent has a size limit of 2048 bytes. This can be customized through
357
+ `Browser.user_agent_size_limit=(size)`.
358
+ - Accept-Language has a size limit of 2048 bytes. This can be customized through
359
+ `Browser.accept_language_size_limit=(size)`.
314
360
 
315
- - `ua`: must be a string representing the user agent.
316
- - `options`: must be a hash (for now it only accepts the `accept_language` option).
361
+ If size is not respected, then `Browser::Error` is raised.
317
362
 
318
- ##### `NoMethodError: undefined method 'user_agent'`
319
-
320
- `.ua` can now be used to retrieve the full User Agent string.
363
+ ```ruby
364
+ Browser.user_agent_size_limit = 4096
365
+ Browser.accept_language_size_limit = 4096
366
+ ```
321
367
 
322
368
  ## Development
323
369
 
@@ -335,25 +381,28 @@ Once you've made your great commits (include tests, please):
335
381
  4. Create a pull request
336
382
  5. That's it!
337
383
 
338
- Please respect the indentation rules and code style.
339
- And use 2 spaces, not tabs. And don't touch the version thing.
384
+ Please respect the indentation rules and code style. And use 2 spaces, not tabs.
385
+ And don't touch the version thing.
340
386
 
341
387
  ## Configuring environment
342
388
 
343
- To configure your environment, you must have Ruby and bundler installed. Then run `bundle install` to install all dependencies.
389
+ To configure your environment, you must have Ruby and bundler installed. Then
390
+ run `bundle install` to install all dependencies.
344
391
 
345
392
  To run tests, execute `./bin/rake`.
346
393
 
347
394
  ### Adding new features
348
395
 
349
- Before using your time to code a new feature, open a ticket asking if it makes sense and if it's on this project's scope.
396
+ Before using your time to code a new feature, open a ticket asking if it makes
397
+ sense and if it's on this project's scope.
350
398
 
351
399
  Don't forget to add a new entry to `CHANGELOG.md`.
352
400
 
353
401
  #### Adding a new bot
354
402
 
355
403
  1. Add the user agent to `test/ua_bots.yml`.
356
- 2. Add the readable name to `bots.yml`. The key must be something that matches the user agent, in lowercased text.
404
+ 2. Add the readable name to `bots.yml`. The key must be something that matches
405
+ the user agent, in lowercased text.
357
406
  3. Run tests.
358
407
 
359
408
  Don't forget to add a new entry to `CHANGELOG.md`.
@@ -362,42 +411,43 @@ Don't forget to add a new entry to `CHANGELOG.md`.
362
411
 
363
412
  1. Add the user agent to `test/ua_search_engines.yml`.
364
413
  2. Add the same user agent to `test/ua_bots.yml`.
365
- 3. Add the readable name to `search_engines.yml`. The key must be something that matches the user agent, in lowercased text.
414
+ 3. Add the readable name to `search_engines.yml`. The key must be something that
415
+ matches the user agent, in lowercased text.
366
416
  4. Run tests.
367
417
 
368
418
  Don't forget to add a new entry to `CHANGELOG.md`.
369
419
 
370
420
  #### Wrong browser/platform/device detection
371
421
 
372
- If you know how to fix it, follow the "Writing code" above. Open an issue otherwise; make sure you fill in the issue template with all the required information.
422
+ If you know how to fix it, follow the "Writing code" above. Open an issue
423
+ otherwise; make sure you fill in the issue template with all the required
424
+ information.
373
425
 
374
426
  ## Maintainer
375
427
 
376
- * Nando Vieira - http://nandovieira.com
428
+ - Nando Vieira - http://nandovieira.com
377
429
 
378
430
  ## Contributors
379
431
 
380
- * https://github.com/fnando/browser/contributors
432
+ - https://github.com/fnando/browser/contributors
381
433
 
382
434
  ## License
383
435
 
384
436
  (The MIT License)
385
437
 
386
- Permission is hereby granted, free of charge, to any person obtaining
387
- a copy of this software and associated documentation files (the
388
- 'Software'), to deal in the Software without restriction, including
389
- without limitation the rights to use, copy, modify, merge, publish,
390
- distribute, sublicense, and/or sell copies of the Software, and to
391
- permit persons to whom the Software is furnished to do so, subject to
392
- the following conditions:
393
-
394
- The above copyright notice and this permission notice shall be
395
- included in all copies or substantial portions of the Software.
396
-
397
- THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
398
- EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
399
- MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
400
- IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
401
- CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
402
- TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
403
- SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
438
+ Permission is hereby granted, free of charge, to any person obtaining a copy of
439
+ this software and associated documentation files (the 'Software'), to deal in
440
+ the Software without restriction, including without limitation the rights to
441
+ use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
442
+ the Software, and to permit persons to whom the Software is furnished to do so,
443
+ subject to the following conditions:
444
+
445
+ The above copyright notice and this permission notice shall be included in all
446
+ copies or substantial portions of the Software.
447
+
448
+ THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
449
+ IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
450
+ FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
451
+ COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
452
+ IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
453
+ CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
data/Rakefile CHANGED
@@ -1,3 +1,5 @@
1
+ # frozen_string_literal: true
2
+
1
3
  require "bundler"
2
4
  require "bundler/setup"
3
5
  Bundler::GemHelper.install_tasks
@@ -14,7 +16,14 @@ end
14
16
  require "rubocop/rake_task"
15
17
  desc "Run rubocop"
16
18
  task :rubocop do
17
- RuboCop::RakeTask.new
19
+ RuboCop::RakeTask.new do |t|
20
+ t.options += %w[
21
+ --display-style-guide
22
+ --display-cop-names
23
+ --extra-details
24
+ --auto-correct
25
+ ]
26
+ end
18
27
  end
19
28
 
20
29
  desc "Run specs against all gemfiles"
@@ -28,4 +37,4 @@ task "test:all" do
28
37
  end
29
38
  end
30
39
 
31
- task default: [:test, :rubocop]
40
+ task default: %i[test rubocop]
data/bot_exceptions.yml CHANGED
@@ -1,3 +1,5 @@
1
+ ---
1
2
  - ruby build
2
3
  - pinterest/android
3
4
  - pinterest/ios
5
+ - yandexsearchbrowser