S3::URI

[dvirtz]

Issue #1507

Description of changes:

• refactor Http::URI to a generic Net::URI.
• add S3::S3URI class.
• both Http::URI and S3::S3URI derive from Net::URI.

Check all that applies:

• [x] Did a review by yourself.
• [x] Added proper tests to cover this PR. (If tests are not applicable, explain.)
• [ ] Checked if this PR is a breaking (APIs have been changed) change.
• [ ] Checked if this PR will not introduce cross-platform inconsistent behavior.
• [ ] Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

• [ ] Linux
• [ ] Windows
• [ ] Android
• [x] MacOS
• [ ] IOS
• [ ] Other Platforms

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[dvirtz]

I'll be happy to get some hints about if and how to update documentation.

[KaibaLopez]

Hi @dvirtz , Can you provide some background on why this change is necessary? Like, what problem are you having that this fixes?

[dvirtz]

The background is shortly described in the attached issue, but to reiterate I'm getting an S3 URI from my library's clients and in order to call the S3 API I need to parse that URI into bucket, key etc. The added S3URI class solves that.

[github-actions[bot]]

Greetings! Sorry to say but this is a very old issue that is probably not getting as much attention as it deservers. We encourage you to check if this is still an issue in the latest release and if you find that this is still a problem, please feel free to open a new one.

[jmklix]

Closing this because this is change is not backwards compatible