[action] artifactory: Add a read timeout option

[raymondhoagland]

The Artifactory gem supports a configurable option for the read timeout. The Fastlane action can be extended to pass the argument through to the client.

The default read timeout for the client is 120 so I see no reason to set a different default.

Checklist

• I've run bundle exec rspec from the root directory to see all new and existing tests pass
• I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
• I've read the Contribution Guidelines
• I've updated the documentation if necessary.

Motivation and Context

Allow configuration of a timeout in case there is a slow connection or other blocker.

Description

This change just adds a new option to set the read timeout for the Artifactory client. This is already a configurable in the underlying gem and will just be passed when initializing the client.

[janpio]

The default read timeout for the client is 120 so I see no reason to set a different default.

Couldn't we not define a default to use the gem default?

[raymondhoagland]

Based on this? Yeah that looks right.

[joshdholtz]

@raymondhoagland This looks good to me! @janpio thanks again for the review while I was away 😍

[fastlane-bot]

Hey @raymondhoagland 👋

Thank you for your contribution to fastlane and congrats on getting this pull request merged 🎉
The code change now lives in the master branch, however it wasn't released to RubyGems yet.
We usually ship about once a week, and your PR will be included in the next one.

Please let us know if this change requires an immediate release by adding a comment here 👍
We'll notify you once we shipped a new release with your changes 🚀

[fastlane-bot]

Congratulations! 🎉 This was released as part of fastlane 2.121.0 🚀

[zlobotron]

Hi
it seems you have missed is_string: false in ConfigItem.new