Skip to content

Clear DID cache entry on PDS after identity.updateHandle#3393

Open
char wants to merge 1 commit intobluesky-social:mainfrom
char:char/update-handle-bust-cache
Open

Clear DID cache entry on PDS after identity.updateHandle#3393
char wants to merge 1 commit intobluesky-social:mainfrom
char:char/update-handle-bust-cache

Conversation

@char
Copy link
Copy Markdown

@char char commented Jan 18, 2025

On a self-hosted PDS, updating an account's handle via com.atproto.identity.updateHandle won't change the reported handle value in com.atproto.repo.describeRepo, since the did cache still has stale data.

To solve this, we just clear the cache after the updateHandle endpoint gets hit.


what do we think chat

@char char changed the title Clear DID cache for did:plc: after identity.updateHandle Clear DID cache for after identity.updateHandle Jan 18, 2025
@char char force-pushed the char/update-handle-bust-cache branch from 439062c to 9b51a5c Compare January 18, 2025 06:11
On a self-hosted PDS, updating an account's handle via
`com.atproto.identity.updateHandle` won't change the reported `handle`
value in `com.atproto.repo.describeRepo`, since the did cache still has
stale data.

To solve this, we just clear the cache after the updateHandle endpoint
gets hit.
@char char force-pushed the char/update-handle-bust-cache branch from 9b51a5c to 64a006b Compare January 18, 2025 06:15
@char char changed the title Clear DID cache for after identity.updateHandle Clear DID cache entry on PDS after identity.updateHandle Jan 18, 2025
char added a commit to char/cocoon that referenced this pull request Apr 30, 2025
this ensures that describeRepo gives us the right did doc after we make
an updateHandle call -- we emit an #identity guy on the event stream so we
might as well bust our own cache as well as everybody else's

ref bluesky-social/atproto#3393
char added a commit to char/cocoon that referenced this pull request Apr 30, 2025
this ensures that describeRepo gives us the right did doc after we make
an updateHandle call -- we emit an #identity guy on the event stream so we
might as well bust our own cache as well as everybody else's

ref bluesky-social/atproto#3393
haileyok pushed a commit to haileyok/cocoon that referenced this pull request Apr 30, 2025
this ensures that describeRepo gives us the right did doc after we make
an updateHandle call -- we emit an #identity guy on the event stream so we
might as well bust our own cache as well as everybody else's

ref bluesky-social/atproto#3393
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