Skip to content

fix: add verify false to fix ssl error#9

Merged
bglar merged 5 commits intomainfrom
fix/minio-ssl-issue
Mar 4, 2026
Merged

fix: add verify false to fix ssl error#9
bglar merged 5 commits intomainfrom
fix/minio-ssl-issue

Conversation

@fMurugi
Copy link
Contributor

@fMurugi fMurugi commented Feb 26, 2026

Summary

  • Refactor file upload to use binary put instead of multipart mechanism.
  • Have ibm_boto3 as default mechanism for handling file uploads

Checklist

  • This PR targets the main branch
  • I have added or updated relevant docs.
  • I have not included any secrets or credentials.
  • Linting and formatting checks pass.

fredotieno
fredotieno previously approved these changes Feb 26, 2026
cwachira
cwachira previously approved these changes Feb 27, 2026
Copy link
Contributor

@cwachira cwachira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@fredotieno fredotieno dismissed stale reviews from cwachira and themself via 7f9f0f9 March 2, 2026 11:33
cwachira
cwachira previously approved these changes Mar 2, 2026
Copy link
Contributor

@cwachira cwachira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@bglar
Copy link
Contributor

bglar commented Mar 2, 2026

@fMurugi, @fredotieno to test with the alpha release:

pip install geostudio==0.1.4a1 --no-cache

@bglar bglar force-pushed the fix/minio-ssl-issue branch from 8fe344c to c9e9542 Compare March 2, 2026 15:59
Copy link
Contributor

@cwachira cwachira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@bglar bglar merged commit 8a7fc90 into main Mar 4, 2026
@bglar bglar deleted the fix/minio-ssl-issue branch March 4, 2026 11:47
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.

4 participants