Skip to content

chore: add instruction file support to TestServer#56

Merged
kessplas merged 22 commits into
fireegg-test-serversfrom
kessplas/inst-file
Nov 6, 2025
Merged

chore: add instruction file support to TestServer#56
kessplas merged 22 commits into
fireegg-test-serversfrom
kessplas/inst-file

Conversation

@kessplas

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

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

@kessplas kessplas marked this pull request as ready for review November 4, 2025 21:50

@seebees seebees left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Small nit, otherwise LGTM

/// This also requires a way to create non-encrypted clients which we don't have yet.
clientId: String,
enableInstructionFilePutObject: Boolean = false,
disableInstructionFile: Boolean = false

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

are we adding custom suffix?
Otherwise, how do I decrypt something that was ReEncrypted?
(Also, ruby supports writing with a custom suffix)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this would need to be done as a parameter to the get object request,
and is out of scope for this PR,
though we should do it at some point

@kessplas kessplas merged commit be6dd9a into fireegg-test-servers Nov 6, 2025
3 checks passed
@kessplas kessplas deleted the kessplas/inst-file branch November 6, 2025 22:12
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