Skip to content

Add plaintext_keys_file option - #11

Closed
astrawan wants to merge 3 commits into
paviro:mainfrom
astrawan:feature/plaintext-keys-file
Closed

Add plaintext_keys_file option#11
astrawan wants to merge 3 commits into
paviro:mainfrom
astrawan:feature/plaintext-keys-file

Conversation

@astrawan

@astrawan astrawan commented Aug 1, 2026

Copy link
Copy Markdown

Support loading an independent plaintext keys file, enabling users to keep key material separate from the main configuration and update it without modifying other files.

paviro and others added 3 commits July 31, 2026 15:50
Reflow a watcher call and a multi-arg args() list to match rustfmt.
event-listener 5.4.1 unconditionally implements Send/Sync for its
stack-allocated StackSlot listener, letting !Send tags cross thread
boundaries. It comes in transitively via zbus. 5.4.2 is the patched
release and a semver-compatible bump.
Support loading an independent plaintext keys file, enabling users to
keep key material separate from the main configuration and update it
without modifying other files.
@paviro

paviro commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Hey! I appreciate you took the time to work on this unfortunately I will have to decline this pull request although I do understand the goal to separate the key material more which is something I want to look into.

Here are my reasons:

  1. identity.toml isn't the main configuration. It already only contains device_name and the key
    material. If you do want to have the key file in a separate folder you can already do that today by simply doing:
ln -s /run/secrets/notema-identity ~/.config/notema/identity.toml
  1. This pr currently wont work correctly and break when the user notema encryption passphrase or rotates their encryption key. That would cause the identity to be rewritten back to the config directory for example.

@paviro paviro closed this Aug 3, 2026
@paviro

paviro commented Aug 4, 2026

Copy link
Copy Markdown
Owner

See #13

@paviro

paviro commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Supported now in the latest release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants