Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Encrypt (and compress) oldlogs #1693

Open
ccuser44 opened this issue Nov 9, 2024 · 3 comments
Open

Encrypt (and compress) oldlogs #1693

ccuser44 opened this issue Nov 9, 2024 · 3 comments
Labels
✨ enhancement Enhancing or improving existing functionality

Comments

@ccuser44
Copy link
Contributor

ccuser44 commented Nov 9, 2024

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?

  • Prevents users from tampering with the code
  • Respects privacy. Many developers may not want Roblox to (easily) see all the commands all users have ran.
  • Not encrypting them likely violates GDPR laws due to there being no easy way to remove them from specific users without nuking the whole datastore.
@ccuser44 ccuser44 added the ✨ enhancement Enhancing or improving existing functionality label Nov 9, 2024
@btt-t
Copy link
Contributor

btt-t commented Nov 10, 2024

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

@ccuser44
Copy link
Contributor Author

ccuser44 commented Nov 10, 2024

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
Copy link
Contributor

btt-t commented Nov 11, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement Enhancing or improving existing functionality
Projects
None yet
Development

No branches or pull requests

2 participants