feat(disk): add custom cipher selection for LUKS2 encryption#4581
feat(disk): add custom cipher selection for LUKS2 encryption#4581Alperen42v wants to merge 2 commits into
Conversation
|
You have clearly not tested any of this code as archinstall doesn't even start up |
|
hii thank you for the feedback You are completely right. The reason I opened this PR early is that this is still a work-in-progress (WIP). As a beginner who is contributing to FOSS for the very first time, I wanted to share the core idea and the basic implementation early, hoping to get feedback and collaboration from more experienced developers to help me finish and fix it. I didn't mean to push a broken build as a final version. I will run it locally with |
|
That's fine, given that the PR was raised in non-draft state I was under the impression this is the final state. Ideally if you want to ask for feedback it'd be better to raise an issue first and then discuss it there to avoid unnecessary throw away work. I think the feature is fine to be added as long as we keep the current algorithm as the default setting. Some pointers
|
|
Update!!!!!!!!!!!!!!!!!!!!!!!!!!!! Changes implemented:
Testing Status
|
|
These things are generally fine if it's tested in a VM, hardware installation is more important with driver/firmware related things like GPU drivers and DE environments. But I'm in the process of testing stuff for a release, and could test this while I'm at it. Not sure it will be tested enough to make it into the release, but if everything looks fine I might sneak it in ^^ |
This PR introduces the ability for users to select a custom encryption cipher (such as
chacha20-poly1305) directly from the disk encryption menu during the installation process.Changes:
luks.py: Added acipherattribute to theLuks2dataclass and dynamically injected the--cipherflag into thecryptsetup luksFormatcommand if specified.device_handler.py: Updatedencrypt()andformat_encrypted()functions to accept and forward the selected cipher configuration fromDiskEncryption.This enhances the flexibility of the installer, enabling advanced users to deploy forensics-resistant setups easily.
Tests and Checks
(Note: Code has been statically verified and peer-reviewed, but not yet tested on a live installation environment.)