Skip to content

feat(test-server): Add Go V4 test server#12

Merged
lucasmcdonald3 merged 36 commits into
fireegg-test-serversfrom
go-v4-server
Oct 9, 2025
Merged

feat(test-server): Add Go V4 test server#12
lucasmcdonald3 merged 36 commits into
fireegg-test-serversfrom
go-v4-server

Conversation

@lucasmcdonald3

@lucasmcdonald3 lucasmcdonald3 commented Sep 22, 2025

Copy link
Copy Markdown

Issue #, if available:

Description of changes:

  • Add test server for Go S3EC V4. (Go S3EC V4 isn't published, this is just a submodule with pending changes.)

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

Lucas McDonald added 16 commits September 16, 2025 11:31
Comment thread test-server/go-v4-server/main.go Outdated
response := PutObjectOutput{
Bucket: bucket,
Key: key,
Metadata: []string{}, // Return empty metadata list as per the model

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we not want to assert metadata?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I don't think we ever want to but the server certainly should forward the metadata on -- will update

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I couldn't get this working and this is blocking other changes, added a TODO

@lucasmcdonald3 lucasmcdonald3 merged commit cb01d31 into fireegg-test-servers Oct 9, 2025
3 checks passed
@lucasmcdonald3 lucasmcdonald3 deleted the go-v4-server branch October 9, 2025 16:09
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