Converting circularstructure to JSON

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.

1. 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.

2. 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"}]

not sure why logging isnt being disabled. and yes we should handle circular jsons too here, cc @Meldiron

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?

Hey 👋@Meldiron, currently the access log is using the same code logic and throwing Circular JSON exceptions, that drained our Quota drastically.

1. There should be an option to disable access log for cases when other apps are already logging it to prevent data duplication. It must be separated from error and warning logs, i.e: 2 checkboxes

2. Definitely fix the Circular JSON exceptions as soon as possible. (Disable access logging can temporarily resolve our issue where we don't need the access logging and might be faster to implement (if condition), but it is not fixing the issue with Circular JSON exceptions for other developers).