Epix-Incorporated / Adonis

Roblox Server Administration System
https://adonis.dev
MIT License
308 stars 178 forks source link

Encrypt (and compress) oldlogs #1693

Open ccuser44 opened 1 week ago

ccuser44 commented 1 week ago

What part of Adonis is this related to?

Other

What are you suggesting?

  1. Compress oldlogs
  2. Encrypt oldlogs
  3. Base64 encode the encrypted oldlogs because the datastore doesn't accept the full ANSI character range

Why?

btt-t commented 1 week ago

I don’t think Adonis needs to be nor should it be bloated any more than it already it is

ccuser44 commented 1 week ago

I don’t think Adonis needs to be nor should it be bloated any more than it already it is

I don't think that's really bloat. Users expect datastores to be encrypted anyways so it just does what users expect to happen. Besides I'm trying to remove as much bloat from Adonis as possible so I think the amount of bloat at least balances out

Just an FYI that the encryption + base64 literslly only takes a few more characters. The code would go from:

service.HttpService.JSONDecode(data)

to:

service.HttpService.JSONDecode(Remote.Decrypt(Functions.Base64Decode(data), Settings.DataStoreKey))
btt-t commented 1 week ago

so I think the amount of bloat at least balances out

I don't think we need to 'balance' bloat so much as we need to eliminate it where we can.

Just an FYI that the encryption + base64 literslly only takes a few more characters. The code would go from:

I get that, but I also feel that encrypting datastores is unnecessary (at least nowadays). It's not so much about it further complicating the codebase so much as it is about the state of Adonis at the moment; it's already bogged down a lot by unnecessary features like the anti system that today stand more as ancient relics of the past than what's beneficial and of value today.

While I understand and appreciate that consistency is important, I feel that we should be aiming to unwind legacy systems, rather than continue to carry them over when they no longer useful. Imo encrypting datastores on Roblox is a novelty that ultimately doesn't achieve much and only serves to complicate the system, and I'm more in favour of doing away with it than extending it.