Differential D13636

[keyserver] redis cache
ClosedPublic
Actions

Authored by varun on Oct 8 2024, 1:23 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Nov 9, 3:56 AM

	Unknown Object (File)
	Sat, Nov 9, 2:23 AM

	Unknown Object (File)
	Mon, Nov 4, 12:34 AM

	Unknown Object (File)
	Sat, Nov 2, 7:37 PM

	Unknown Object (File)
	Fri, Nov 1, 2:31 AM

	Unknown Object (File)
	Wed, Oct 30, 8:05 PM

	Unknown Object (File)
	Wed, Oct 30, 1:58 AM

	Unknown Object (File)
	Mon, Oct 28, 11:34 AM

View All 36 Files

Subscribers

Details

Reviewers

Commits

rCOMM757a38090721: [keyserver] redis cache

Summary

this cache is used in the next diff to try to retrieve info about a particular fc channel without making a neynar API call.

Depends on D13635

Test Plan

tested in next diff

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Oct 8 2024, 1:23 PM

Herald added a subscriber: tomek. · View Herald TranscriptOct 8 2024, 1:23 PM

varun added a child revision: D13637: [keyserver][lib] replace neynar client cache with redis cache.Oct 8 2024, 1:27 PM

Harbormaster completed remote builds in B32110: Diff 44975.Oct 8 2024, 1:44 PM

varun requested review of this revision.Oct 8 2024, 1:44 PM

ashoat accepted this revision.Oct 8 2024, 6:44 PM

This revision is now accepted and ready to land.Oct 8 2024, 6:44 PM

Closed by commit rCOMM757a38090721: [keyserver] redis cache (authored by varun). · Explain WhyOct 8 2024, 9:17 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM757a38090721: [keyserver] redis cache.

in case someone in the future wonders why i created a new Redis client here... it's because once a client is in pub/sub mode, it can't be used for regular Redis operations like managing a cache

Revision Contents
Changeset List

Path

Size

keyserver/

src/

utils/

51 lines

Diff 44985

keyserver/src/utils/redis-cache.js

Loading...