aptly icon indicating copy to clipboard operation
aptly copied to clipboard

API: Set Batch automatically?

Open WilliamDEdwards opened this issue 3 years ago • 2 comments

Detailed Description

https://www.aptly.info/doc/api/publish/ documents the signing option Batch as:

should be set if passing passphrase

Wouldn't it make more sense to set Batch automatically (and internally) depending on whether Passphrase and PassphraseFile is set?

Context

I can imagine that requiring users to set Batch when a passphrase is set requires extra, unwanted validation on the API side.

Possible Implementation

N/A

Your Environment

N/A

WilliamDEdwards avatar Aug 29 '22 13:08 WilliamDEdwards

I had a look at it and you're right it seems like Batch is kind of a useless option, since it would need to be set to true anyway because the api will time out when it tries to get the passphrase from stdin. As it happens, this isn't even validated, so a missing passphrase with Batch = false will result in the timeout I described earlier : /

I'll make a branch to set it to true by default.

reglim avatar Aug 17 '23 12:08 reglim

I think this would actually be a good idea...

neolynx avatar Feb 14 '25 18:02 neolynx