Skip to content

Add netrc export option - #13381

Closed
Millak wants to merge 1 commit into
keepassxreboot:developfrom
Millak:add-netrc-export
Closed

Add netrc export option#13381
Millak wants to merge 1 commit into
keepassxreboot:developfrom
Millak:add-netrc-export

Conversation

@Millak

@Millak Millak commented May 27, 2026

Copy link
Copy Markdown

This adds a netrc exporter type to keepassxc-cli and tries to create a netrc file which can be read by other programs.

Fixes #13380

Testing strategy

I copied tests/TestCsvExporter.* to tests/TestNetrcExporter.* and modified and removed tests as necessary.

Type of change

  • ✅ New feature (change that adds functionality)

@@ -0,0 +1,100 @@
/*
* Copyright (C) 2015 Florian Geyer <blueice@fobos.de>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm pretty sure these persons did not have anything to do with your source and header files. Check them all.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not always sure which copyright headers to keep and which to toss when I've started by copying another file and then modifying it.

@droidmonkey

droidmonkey commented May 27, 2026

Copy link
Copy Markdown
Member

I dont necessarily like that this is an export option and not a format option against the show command. You are essentially dumping the entire contents of your database into another program so it may use one of the entries. Why use KeePassXC in the first place? netrc is the 1990s way to manage credentials.

@Millak

Millak commented May 28, 2026

Copy link
Copy Markdown
Author

I dont necessarily like that this is an export option and not a format option against the show command. You are essentially dumping the entire contents of your database into another program so it may use one of the entries.

That's a good way of phrasing it. The output is meant to be ephemeral, so show would be better, and it would only return the actual entry desired.

@varjolintu

Copy link
Copy Markdown
Member

Closed because you made a duplicate: #13387

@varjolintu varjolintu closed this May 29, 2026
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.

Add netrc export option

3 participants