Fix Rubocop Security/Open

[rhymes]

What type of PR is this? (check all applicable)

• Refactor
• Feature
• Bug Fix
• Documentation Update

Description

The biggest change here is the removal of Kernel#open in favor of HTTParty which is a gem that's used elsewhere.

A couple of reasons:

• Kernel#open is potentially a security flaw, see rubocop's rule about it
• Two places out of three don't need to download the whole body: in the first we're only checking the status and in the second one we're only using the HTTP request to get the last URI in a redirect. Both can be accomplished with a HTTP HEAD, which is going to be faster (no body, no reason for Ruby to build unnecessary objects for it)

I've also fixed a couple of cosmetic issues.

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Added to documentation?

• docs.dev.to
• readme
• no documentation needed

[maestromac]

Thanks for this @rhymes ! LGTM!