Skip to content

fix: send query params for NewFromArchive#27

Merged
hiroTamada merged 1 commit intomainfrom
fix/new-from-archive-query-params
Feb 18, 2026
Merged

fix: send query params for NewFromArchive#27
hiroTamada merged 1 commit intomainfrom
fix/new-from-archive-query-params

Conversation

@hiroTamada
Copy link
Contributor

@hiroTamada hiroTamada commented Feb 18, 2026

Summary

  • Fix VolumeService.NewFromArchive to pass VolumeNewFromArchiveParams into ExecuteNewRequest.
  • Ensure required query params (name, size_gb) are serialized for POST /volumes/from-archive.
  • Prevent server-side validation failures caused by missing query parameters.

Test plan

  • cd /Users/hirotamada/kernel/kernel/hypeman-go && go test ./...

Made with Cursor


Note

Low Risk
Small client-side fix to include missing query parameters; minimal blast radius beyond the NewFromArchive call path.

Overview
VolumeService.NewFromArchive now passes VolumeNewFromArchiveParams to requestconfig.ExecuteNewRequest instead of nil, ensuring required query parameters (e.g., name, size_gb) are serialized for POST /volumes/from-archive while still streaming the tarball as the request body.

Written by Cursor Bugbot for commit a8c45a6. This will update automatically on new commits. Configure here.

Pass VolumeNewFromArchiveParams to request execution so required query fields like name and size_gb are serialized for /volumes/from-archive requests.

Co-authored-by: Cursor <cursoragent@cursor.com>
@hiroTamada hiroTamada merged commit fae578c into main Feb 18, 2026
5 checks passed
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.

1 participant

Comments