bnewbold
bnewbold
I could be wrong about this, but I think we have a problem with lexgen-erated client functions. For example: https://github.com/bluesky-social/indigo/blob/main/api/atproto/admingetModerationReports.go#L20 ``` func AdminGetModerationReports(ctx context.Context, c *xrpc.Client, cursor string, limit int64,...
Currently, if you do something like `getRecord` on a record that doesn't exist, or an unknown XRPC endpoint, you get an HTTP 500 error, without a properly formatted XRPC error...
This issue is a list of things i'm intending to review and test in the indigo MST and repo/repomgr implementations. I'll also be cross-referencing and expanding from the `atproto` (Typescript)...
Using ephemeral/parallel postgresql databases (eg, ramdisk). Goal is to get additional confidence that we are testing the same configuration that we deploy in production and staging. Eg, should be easy...
With `fakermaker`, got an error: ``` HANDLER ERROR: (/xrpc/app.bsky.notification.list) attempted to hydrate unknown notif kind: 2 ``` The "mentioned" notification kind is commented out and un-implemented in `notifs/notifs.go`.
Not all endpoints and features are implemented in the current PDS. Some notable missing things (turned up by `fakermaker` test; you can all search for `panic` under `pds/handlers.go`). Some notable/priority...
Running `fakermaker` `gen-profile`, got an error: ``` mst.Add failed: value already set at key: app.bsky.actor.profile/self ``` Fix should include a regression test.
We are planning to tear down the legacy `search.bsky.social` service immanently. As noted in https://github.com/jcsalterego/Contrails/issues/26, that index has not been getting updates for about a week. This migration was hinted...
Another one breaking up https://github.com/bluesky-social/atproto/pull/2376 in to smaller chunks. In a previous review on the old PR, @dholms that the client hadn't been updated to use `q` instead of `term`....
Part of splitting up https://github.com/bluesky-social/atproto/pull/2376 in to smaller chunks for easier merging. These have been marked deprecated for a long time. Removes from client and API-side at the same time....