Hello 👋, when this will be fixed on Appwrite Sites? It's draining our quotas for Sentry, it must be a problem with the default logger which is always on for Appwrite Sites and cannot be turned off.
#Converting circularstructure to JSON
clever lark
- When disabling the logger, it should stay disabled. Appwrite Sites have the logger always on, even if disabled. Access log should be disabled as well.
- Converting circular structure to JSON must be fixed and handled properly, for example using https://www.npmjs.com/package/flatted:
// ESM
import {parse, stringify, toJSON, fromJSON} from 'flatted';
// CJS
const {parse, stringify, toJSON, fromJSON} = require('flatted');
const a = [{}];
a[0].a = a;
a.push(a);
stringify(a); // [["1","0"],{"a":"0"}]
sonic fjord
not sure why logging isnt being disabled. and yes we should handle circular jsons too here, cc @gray cloud
gray cloud
1. When disabling the logger, it should stay disabled. Appwrite Sites have the l...
Heyy 👋
Right now, disabled logging only disables trackng console.log and console.error. Network logs are still created, so you see them in "Logs" tab.
Ill talk to Chen about possible re-wording to make this clearer. Is it troubling for you to see request logs - if so, what's the benefit from disabling it?