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

Make the Adonis encryption cipher support the full 0-255 character range #1701

Closed
ccuser44 opened this issue Nov 10, 2024 · 3 comments · Fixed by #1704
Closed

Make the Adonis encryption cipher support the full 0-255 character range #1701

ccuser44 opened this issue Nov 10, 2024 · 3 comments · Fixed by #1704
Labels
✨ enhancement Enhancing or improving existing functionality

Comments

@ccuser44
Copy link
Contributor

What part of Adonis is this related to?

Other

What are you suggesting?

Make the Adonis encryption cipher support the full 0-255 character range

If someone does this however you should keep note that datastores depends on the encryption algorithm and thus you would have to use something else for them and have backwards compatibility.

@ccuser44 ccuser44 added the ✨ enhancement Enhancing or improving existing functionality label Nov 10, 2024
@EasternBloxxer
Copy link
Contributor

Remove remote encryption entirely. its a waste of time

@ccuser44
Copy link
Contributor Author

Remove remote encryption entirely. its a waste of time

No :P

@ccuser44
Copy link
Contributor Author

Also new one should use bit32.bxor because it works just better.

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

Successfully merging a pull request may close this issue.

2 participants