feat: add fx modules for memory and aws stores - #10
Merged
Conversation
This was referenced Mar 17, 2026
Merged
frrist
reviewed
Mar 18, 2026
| // UploadsTable is the table name for uploads. | ||
| UploadsTable string `mapstructure:"uploads_table"` | ||
|
|
||
| AgentIndexTable string `mapstructure:"agent_index_table"` |
Member
There was a problem hiding this comment.
Are the names of these tables something an operator should ever configure? Or would it make more sense to hard code them as constants that we can configure later if the need arises?
Member
Author
There was a problem hiding this comment.
Well we have multiple deployments of the upload service and they all have different table names, but actually it's just a prefix change - the names of the tables are always the same (minus the prefix).
frrist
approved these changes
Mar 18, 2026
Uses the agent store to index and log incoming/outgoing messages and uses it in the receipts endpoint to retrieve receipts for tasks. * feat: dependency injected handlers (#12) Alters the handlers so that they are dependency injected. Follows the same pattern as in Piri. This avoids having to create a huge object that has all the dependencies for all handlers and also avoids having to have an interface per handler that defines the required deps. Moved config module out of the app module since in tests we want to supply the config directly, not write a config file and then load it from disk. * feat: access implementation (#15) Adds the following handlers: * `access/authorize` * `access/claim` * `access/confirm` * `access/delegate` * `provider/add` * chore: disable windows and 32bit testing (#13) * feat: provider add implementation * chore: appease linter * chore: appease linter * feat: add `upload/add` impl (#16) * feat: add `space/blob/add` and `ucan/conclude` implementation (#17) * feat: add `space/index/add` implementation (#18) * feat: add `space/blob/replicate` implementation (#19) * feat: storage provider admin handlers and CLI commands (#20) Adds CLI commands: ``` sprue client admin provider register sprue client admin provider deregister sprue client admin provider list sprue client admin provider weight set ``` * chore: remove state store and other cleanup (#21) Final task to remove the old state store (now unused). * feat: add upload and blob list handlers (#22) Adds missing handlers for listing uploads, upload shards and blobs. * refactor: remove provider DID from register command (#24) This is unnecessary since it is the issuer of the proof. * feat: add SMTP mailer (#25) This PR adds an SMTP mailer and fixes a bunch of other issues.
alanshaw
added a commit
that referenced
this pull request
Apr 9, 2026
Adds stores that _should_ be compatible with w3infra. * feat: add fx modules for memory and aws stores (#10) * feat: log agent messages (#11) Uses the agent store to index and log incoming/outgoing messages and uses it in the receipts endpoint to retrieve receipts for tasks. * feat: dependency injected handlers (#12) Alters the handlers so that they are dependency injected. Follows the same pattern as in Piri. This avoids having to create a huge object that has all the dependencies for all handlers and also avoids having to have an interface per handler that defines the required deps. Moved config module out of the app module since in tests we want to supply the config directly, not write a config file and then load it from disk. * feat: access implementation (#15) Adds the following handlers: * `access/authorize` * `access/claim` * `access/confirm` * `access/delegate` * `provider/add` * chore: disable windows and 32bit testing (#13) * feat: provider add implementation * chore: appease linter * chore: appease linter * feat: add `upload/add` impl (#16) * feat: add `space/blob/add` and `ucan/conclude` implementation (#17) * feat: add `space/index/add` implementation (#18) * feat: add `space/blob/replicate` implementation (#19) * feat: storage provider admin handlers and CLI commands (#20) Adds CLI commands: ``` sprue client admin provider register sprue client admin provider deregister sprue client admin provider list sprue client admin provider weight set ``` * chore: remove state store and other cleanup (#21) Final task to remove the old state store (now unused). * feat: add upload and blob list handlers (#22) Adds missing handlers for listing uploads, upload shards and blobs. * refactor: remove provider DID from register command (#24) This is unnecessary since it is the issuer of the proof. * feat: add SMTP mailer (#25) This PR adds an SMTP mailer and fixes a bunch of other issues.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Dependency Tree