Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Azure Fog Gems #4195

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Update Azure Fog Gems #4195

wants to merge 3 commits into from

Conversation

jochenehret
Copy link
Contributor

@jochenehret jochenehret commented Jan 30, 2025

  • A short explanation of the proposed change:

We want to update the Fog Azure gems to meet security requirements. Currently the gems are resolved from (outdated) feature branches because they contain CF-specific patches. This PR updates the gems to the latest available versions:

The other dependencies of "fog-azure-rm" were updated to a set of versions that worked.

  • An explanation of the use cases your change solves

Updates outdated dependencies to the latest available versions (which are also outdated because the gems are not maintained anymore). Starting with this PR, we want to discuss how to move forward.

  • Links to any other associated PRs

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests
    CATs found no issues.

* update "fog-azure-rm" to v0.6.0 (use fork which contains fixes from https://github.com/fog/fog-azure-rm/tree/fog-arm-cf)
* this version references v2.0 of "azure-storage-common" and "azure-storage-blob" gems
* the other dependencies of "fog-azure-rm" were updated to a set of versions that worked...
@sethboyles
Copy link
Member

sethboyles commented Jan 31, 2025

It doesn't look like to me that azure-storage-ruby has fixed the issue we were experiencing: https://github.com/Azure/azure-storage-ruby/blob/v2.0.2-blob/blob/lib/azure/storage/blob/block.rb#L414

The change I think we still need: sethboyles/azure-storage-ruby@d4737cd

@sethboyles
Copy link
Member

Apparently Thom and I worked on this spike in the fog library to avoid the issue entirely, but I don't recall the specifics: sethboyles/fog-azure-rm@00945ea

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants