unsend icon indicating copy to clipboard operation
unsend copied to clipboard
verified publisher icon unsend-dev

add api logs

Open KMKoushik opened this issue 10 months ago • 5 comments

KMKoushik avatar Feb 08 '25 21:02 KMKoushik

@KMKoushik I saw this feature PR is still pending. Is there anything that needs to be fixed or implemented on this?

shreekrishnalamichhane avatar Jul 22 '25 17:07 shreekrishnalamichhane

@KMKoushik this is nice aswell :)

enemyrr avatar Aug 22 '25 13:08 enemyrr

sorry, i'm thinking a lot about this pr. this might easily fill up the db and create bunch of issues. i'm not even sure if it;s wise to store logs in postgres atleast directly without any timeseries extension.

will not be merging this pr anytime soon.

KMKoushik avatar Aug 23 '25 21:08 KMKoushik

@KMKoushik What about a seperate clickhouse database to store the logs keeping it seperate from the main database and can be optional. Controlled by the env variables.

shreekrishnalamichhane avatar Oct 26 '25 10:10 shreekrishnalamichhane

I can send an update to the PR with the new approach.

shreekrishnalamichhane avatar Oct 26 '25 10:10 shreekrishnalamichhane