Skip to content

copy_blob between containers? #23

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

Closed
eric-simpliphy opened this issue May 9, 2025 · 4 comments · Fixed by #24
Closed

copy_blob between containers? #23

eric-simpliphy opened this issue May 9, 2025 · 4 comments · Fixed by #24

Comments

@eric-simpliphy
Copy link

I see there is a copy_blob method in the client. But it is limited to copying blobs within a container. I have a need to copy between containers (and I could see someone having a need to copy between accounts which I believe is supported by the API).

Is there any desire to expand that function to allow that? Or does the fact that the container being part of the constructor means from an architecture perspective this lib is just meant to operate with a single container and multi-container operations are out of scope?

If out of scope I'm wondering if there is a way to make it easier to "extend" this library for more custom needs. I'm working around this by basically using the HTTP class directly similar to what the Client class is doing but just more directly specifying the headers, etc. The main bit of jankyness is I have to call the private signer method on Client to provide it to HTTP. Maybe making that public would provide a relief valve for these sorts of requests. People could make more custom API calls by operating at that lower level but still build on the work you have done (signing, generating the uri, etc).

@JoeDupuis
Copy link
Member

Hmm, true, the api with the container in the initializer is a bit restrictive. If it becomes too restrictive in the future I might consider changing it with a new major version. Hopefully keeping backward compatibility.

Would something like #24 work?

@eric-simpliphy
Copy link
Author

#24 looks amazing!

@JoeDupuis
Copy link
Member

Perfect! I'll release 0.5.8

@JoeDupuis
Copy link
Member

Done!

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 a pull request may close this issue.

2 participants