Ecosyste.ms: OpenCollective

An open API service for software projects hosted on Open Collective.

github.com/QubesOS/qubes-app-linux-split-gpg

Qubes component: app-linux-split-gpg
https://github.com/QubesOS/qubes-app-linux-split-gpg

`--output` is more than just stdout redirection

GPG will behave differently if `--output=-` or `-o-` is passed than if
no `-o` or `--output` fla...

40a1d3b60ddb8b8f396ad3a04ce87a6997f89c12 authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Use set_output for -o as well as --output

Handling -o and --output differently is inconsistent.

5b53dd38634b32cc3af1c3b093b9844d0f55d984 authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Recognize -o in addition to --output

The short version is, well, shorter.

5aa66e64810847012298d61e717d94a7c22a1451 authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Reject non-UTF-8 compatible display charsets

The check is done in the wrapper script, which ignores the option if it
is valid. The C code fo...

655bc129782456f7cdc71b42d871e4fc29feb000 authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Allow `--utf8-strings`

It is used by Mailpile and is safe.

9d7b24683b0c47019442b1c1dbbc2662e11cc216 authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Ignore various options used by Mailpile

Fixes QubesOS/qubes-issues#3485

99f19b05733874d6699bced3025eea12a38d47d7 authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Ditch pipecat and fix hangs

Instead of using named pipes and an extra subprocess, just use the stdin
and stdout of qrexec-cl...

54a010e3e28d9d2152c81e82fdb4cdd3791efebb authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Merge remote-tracking branch 'origin/pr/65'

* origin/pr/65:
More robust file descriptor handling

9fb4bfb1cd7f8f1ad8a4d333323c0b0407bbaef9 authored almost 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/63'

* origin/pr/63:
Allow --with-secret
Ignore several options used by kmail

643177123162c47a7892d5d352a27cf18b3b5dcd authored almost 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/60'

* origin/pr/60:
Forcibly disable dirmngr

ca973c4df7b60a4249cb8a1d2d31f3dc3fc0c186 authored almost 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Fix the Arch build

The Arch build was broken because there is no install-vm target in the
Makefile.

af96037d4875d3025ed1b1bc23f5c957ea59daec authored almost 3 years ago by Demi Marie Obenour <[email protected]>
More robust file descriptor handling

Set file descriptors to -1 after closing them, and assert that they are
in range before adding t...

850e19d1d68ccbafa6d3156f4c66a21c127df27e authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Merge remote-tracking branch 'origin/pr/59'

* origin/pr/59:
Allow '--clear-sign' as well as '--clearsign'

98492067f1b2af045e2654c65c475338af82ac7e authored almost 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/58'

* origin/pr/58:
Translate ‘--detach-sig’ to ‘--detach-sign’

d35751ff2965b58c666a8f5f5a2c9f51629081a0 authored almost 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/57'

* origin/pr/57:
tests: reduce whonix boot clock rand during testing

30794b26f7df0b6f39de0bc8cd63539989f6169e authored almost 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/53'

* origin/pr/53:
undo removal of resetting dogtail search count
merge timeout approaches
pr...

91534ab2f99244c946a74ee673c11cedea0692dd authored almost 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/51'

* origin/pr/51:
tests: increase search attempts for thunderbird start

4e8ff5e9987dc5458418a51e30e90110fd65b100 authored almost 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Allow --with-secret

It is used by kmail; see
https://github.com/QubesOS/qubes-issues/issues/3326#issuecomment-956257914

f39380923041ed7c949226fa5f78afa5d740bbda authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Ignore several options used by kmail

Add '--lc-ctype', '--lc-messages', '--xauthority', and
'--input-size-hint' to ignored list

Thos...

ff31021a7516494a6ed8392ef52402e5cb6a63ac authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Forcibly disable dirmngr

Using dirmngr in a backend qube makes no sense, so forcibly disable it
with '--disable-dirmngr'.

aaec233e60924fe2df30824c94c532f3590c92d9 authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Allow '--clear-sign' as well as '--clearsign'

The two are synonyms and both should be accepted. Although
'--clear-sign' could be mapped to '-...

a9f57befad0e55e3809ab96386ab2a3e9d34c0b6 authored almost 3 years ago by Demi Marie Obenour <[email protected]>
Translate ‘--detach-sig’ to ‘--detach-sign’

Some code apparently relies on ‘--detach-sig’ working.

Reported-by: Frédéric Pierret <frederic....

abb7c47e7213a3e56ea85c7843851f23e5e04d6b authored almost 3 years ago by Demi Marie Obenour <[email protected]>
undo removal of resetting dogtail search count

7f7211c35b83a40b0cb13326aff4c0625814db6e authored almost 3 years ago by deeplow <[email protected]>
merge timeout approaches

64ec8984e7cd8c0d0ccf2ed67e574232a58ff422 authored almost 3 years ago by deeplow <[email protected]>
print failed to send error

ec7c1160d7f336f79321e95b7322e036d82d767b authored almost 3 years ago by deeplow <[email protected]>
tests: earlier detection of failed message sending

In Thunderbird when for various reasons would fail to send a
message (e.g. no thaving properly c...

935293322732b47908fb8e31420a6f9868c5dd17 authored almost 3 years ago by deeplow <[email protected]>
tests: reduce whonix boot clock rand during testing

7ecdcf7acaa549c3a73bbf9ff4cca1efaf1bbfdd authored almost 3 years ago by deeplow <[email protected]>
version 2.0.55

3ad9aeedf5da192e3b1e6fa1174dab3ec62a600b authored about 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/50'

* origin/pr/50:
Do not use fancy unicode quotes
Avoid mishandling empty strings
Consolidat...

3fee9166fc70b289fbcf0168baf0585ad10a9f08 authored about 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/55'

* origin/pr/55:
tests: remove deprecated enigmail HACK & tb restart
tests: tb add PGP key to...

062f3726d5b26ef61c6ab158a214baf74239ae70 authored about 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Do not use fancy unicode quotes

They make code harder to review for malicious unicode sequences.

5fe4133461aa1368fff39dec91f76785287b43ea authored about 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Avoid mishandling empty strings

They would cause a premature loop exit.

e280dc0eb1686a7dc06dbdfd939f30865c7421e4 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Consolidate redundant case branches

Shrinks the code with no disadvantages.

47ac25887c5c7aab0e0f92ef9fa5abbd2839b9aa authored about 3 years ago by Demi Marie Obenour <[email protected]>
Use bash's [[ consistently

bash only supports [ for POSIX compatibility.

0cbea15819e588ab19ef50748691ec2d034726c5 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Fix some bugs in the wrapper script

Hopefully this will fix openQA.

10f715670dc18beb281874f0e1de7178e735a3b7 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Require arguments to be printable UTF-8

This is a defense in depth measure.

38e808101b19562953bcf4fbfe450a2a73aefeeb authored about 3 years ago by Demi Marie Obenour <[email protected]>
Convert --detach into --detach-sign

The former is an abbreviated version of the latter and is rejected, but
Thunderbird uses it :(

df81022cbd061ba6351e89ccd6d1b3ee2cae43a7 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Never permute arguments and options

GPG doesn’t consider arguments following a non-option argument to be
options, even if they begin...

92a4e6a30b2b1c330e3f51a46331fbc903cf0f2c authored about 3 years ago by Demi Marie Obenour <[email protected]>
Server: reduce getopt_long(3)’s attack surface

In normal use, the client should only invoke the server if the arguments
are correct, so the ser...

a0b541da1d787e5fb05d62793ec2a80a23442957 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Fix short option handling in wrapper script

Short options were not handled correctly by the wrapper script,
potentially causing incorrect op...

30c61d31c260bc9777c3424bcb16ec22ac3ae1c8 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Reject abbreviated long options

getopt_long(3) allows long options to be abbreviated if they are
unambiguous or if they match th...

5de8cde67f8544187b1a2cc8aff229147fd4aa0d authored about 3 years ago by Demi Marie Obenour <[email protected]>
Better error message if GPG tries to read a password

“Inappropriate ioctl for device” is rather user-unfriendly. ‘--no-tty’
causes GPG to emit a bet...

358e89743c7fac9f14d6259d02249e0822893584 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Remove options already ignored in wrapper script

No need to allow them.

8806da62bbfecdfc9a1ac0e59d922e387a4fd746 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Do not allow remote qube to control argv[0] of gpg

It is useless extra attack surface.

efc437ba18f19dc13f9a4bdf477e266fd27f6c65 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Remove ‘--rfc1991’

It isn’t even supported by gpg.

54c9e65dd2116ff1bd8dbf013cffe15e1356a6b5 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Add trailing newline in error message

f614220b62b165c21a83e5a74d6474a7c9afff31 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Fix some bugs in the wrapper script

There were several places where options that took arguments were
misparsed. Also ignore some de...

ed876f48ad111e4cec8eb30dcd7a14822d222ce2 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Trivial client cleanup

No change in behavior.

1cf98b76934b8671950e46ede300fcd6c94c4fc4 authored about 3 years ago by Demi Marie Obenour <[email protected]>
Fix quoting bug

No change in behavior for reasonable usernames.

8c3a4a0027ade3eb67b4fa14bbda5c6652235f9c authored about 3 years ago by Demi Marie Obenour <[email protected]>
version 2.0.54

353e82713673ea286759480c5518970c376eb8bf authored about 3 years ago by Marek Marczykowski-Górecki <[email protected]>
tests: remove deprecated enigmail HACK & tb restart

Removes workaround for enigmail extension bug (no longer in use)
which was introduced in commit ...

13a9caafd4db758b90a919204b0b318e8d6f0875 authored about 3 years ago by deeplow <[email protected]>
tests: tb add PGP key to acct via user.js

setup user's PGP via the user.js instead of doing it via the UI
(less prone to breaking when UI ...

3458bc5d7842bb4265cd6b2d88bcce91e9ecca23 authored about 3 years ago by deeplow <[email protected]>
Merge remote-tracking branch 'origin/pr/52'

* origin/pr/52:
tests: retry "enter imap password" on fail
tests: support windows with role ...

a1b46d266b25fadae1271151ffab8c5d4ae085d3 authored about 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/49'

* origin/pr/49:
Tighten up parsing of file descriptors

7dacdd3492eaeb49b2c0b9309d49d0a719b4348f authored about 3 years ago by Marek Marczykowski-Górecki <[email protected]>
tests: retry "enter imap password" on fail

Sometimes the IMAP password prompt would not show up when we get
the messages on thunderbird 78....

b3b72ed4959b063a0233c619bba858bbd3bf57df authored about 3 years ago by deeplow <[email protected]>
tests: support windows with role 'frame' or 'dialog'

For some odd reason, when thunderbird creates windows, the window
role is non-deterministically ...

5f71ba8871f36a794cd3b581bcde63754fd3fcac authored about 3 years ago by deeplow <[email protected]>
Make fd path replacement length check clearer

The previous length checking code was confusing. It used two variables,
untrusted_sig_path_size...

f48ef300b860c4ae60b510c486dbe82bf01f281d authored about 3 years ago by Alyssa Ross <[email protected]>
Fix off-by-one in fd path replacement

If I had a GPG signature file at /tmp/sig that I wanted to verify,
qubes-gpg-client --verify /tm...

089d1fbad4f3850182263487a15ad722dc72af61 authored about 3 years ago by Alyssa Ross <[email protected]>
tests: improve resilience of OpenPGP button click

OpenPGP button sometimes was not being detected even after numerous
attemps and time. For these ...

52cc3cc43d479a99830ed9e75835902133189b89 authored about 3 years ago by deeplow <[email protected]>
tests: check mail manually instead

Thunderbird 78.14.0 was having an issue where it would set the
user preference of "mail.server.s...

f03ce50d69c36ed5e42087e8913f21a7e18ffe6c authored about 3 years ago by deeplow <[email protected]>
tests: make thunderbird user.js dynamically generated

splitgpg2 will not need certain components from user.js so this
creates a need to dynamically ge...

cb2ba73a057bc701c45524342e143d2f308416a1 authored about 3 years ago by deeplow <[email protected]>
tests: tb fix OpenPGP opening in tb91+

471dd95bd7410ae608f620492c8faf1a283bfd68 authored about 3 years ago by deeplow <[email protected]>
tests: remove deprecated tb OpenPGP message code

Would show up as a dialog box which is no longer the case so we're
removing it to simplify the code

c15b563dd8f7c6969daf0fb9947ea652ece2f6f1 authored about 3 years ago by deeplow <[email protected]>
tests: more fixes for tb 91+

bc23eeba56a3ad1285db84a36e9e501d87173508 authored about 3 years ago by deeplow <[email protected]>
tests: fix bug where arguments were missing a space

b5a34462efd91da5b741728c77913a75b95c7d8b authored about 3 years ago by deeplow <[email protected]>
tests: autotype IMAP password

autotypes the IMAP password as set while configuring dovecot-imap

19cbfe1038e9d7485a97276b7030f07daba79ff6 authored about 3 years ago by deeplow <[email protected]>
tests: remove autoconf and local account setup code

Autoconfiguration and local account setup code is no longer needed
since that thunderbird is now...

8709c2de3bfa87109ab421d67309feafa3178b9d authored about 3 years ago by deeplow <[email protected]>
tests: thunderbird use user.js for default profile

Initial refacto in order to setup the autoconfiguration for
thunderbird without relying on inter...

0069ad385b91bbac2973f7f0d72e441aad8d9258 authored about 3 years ago by deeplow <[email protected]>
test: add IMAP server as tb 91+ removed movemail

Dovecot server is used

3c1b9acc2e09f1f0e8a9a4191ff383c04ba93ef8 authored about 3 years ago by deeplow <[email protected]>
tests: increase search attempts for thunderbird start

Fedora-34-xfce was taking quite a bit longer. See:
https://openqa.qubes-os.org/tests/21405#step/...

e624dab5ae8ba6fe491abb0fa5acff79bcbe04f6 authored about 3 years ago by deeplow <[email protected]>
version 2.0.53

6ddc1dbcbb9a42367a6591f4a65d67c0a2cf5bee authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
tests: option arguments parsing

Check if gpg and split-gpg agrees whether an option requires an argument
or not.

f3f72b9ae79dae206f789fcb226669c6cb465f44 authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
tests: include stderr in the failure message

cd4e229a4be975c672dec9cf7a2f94a8fccdc5b5 authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Add --display option to the parser

This avoids confusing it with --display-charset.
Do not allow to use the option.

c3301ff7dd8775d3d37f87282618c9243a0117f2 authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Fix forbidden option reporting

If an option has an argument, optind-1 points at that argument, not the
option itself. Fix this,...

819f3bb821439dc8ad4633745fdafe78b7ebbf1a authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Add --default-recipient option to the parser

This avoids confusion with --default-recipient-self.
But don't add it to allowed options until s...

dd0ae884e8a63d391d8fd0aa2b82f16529fabe3f authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Remove duplicated --fixed-list-mode

678a1b2f6d2ed624c8db48cc17362e2441ac6e47 authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Fix --with-colons declaration

It isn't the same as -k. This is especially important, since -k
(--list-keys) is a command optio...

3c8da0dd05ac7ac91f5e018e769b9080dd1d5c4e authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Properly handle gpg's command options

Command option define how positional arguments should be interpreted.
Implement proper check for...

d0d8a42188bdcc8cbca3bf913fe34bb9ee23ac64 authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Correctly mark --s2k-* options as requiring an argument

--s2k-* options take an argument, so mark them as such. Split-gpg and
the actual gpg disagreeing...

c1b0e147bc13ae05235465686e0c0b9688aa963a authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Tighten up parsing of file descriptors

strtol() is a bit too loose with its parsing. Only accept the form that
we actually want.

92835aa5521cb04877e6b2d07c9554e08bd7fc04 authored over 3 years ago by Demi Marie Obenour <[email protected]>
version 2.0.52

6a004564133865dc6bd10eff516829bd7323842d authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/48'

* origin/pr/48:
tests: avoid redirect to dogtail repo

a1459fe0098d4ab5a7bf22c8e1b9135a7c993af4 authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
use functools wraps in decorator

ff8d3ec5e2e456fb29ca6a778d61d6e3eb9a1c66 authored over 3 years ago by deeplow <[email protected]>
retry_if_failed add_local_account (now idempotent)

Made the function add_local_account generally idempotent to
support the retry_if_failed decorator.

cdb1f131040ff48a1681e48683163a4b9fddee87 authored over 3 years ago by deeplow <[email protected]>
add retry_if_failed wrapper to setup function

this is to stabilize the tests during the setup phase. Many were
failing due to random errors fr...

e9c1fd0ebe48f2845709d4b2691a56911cdcba3e authored over 3 years ago by deeplow <[email protected]>
tests: manage thunderbird through dedicated class

previously thunderbird was started, closed ad-hoc in various
places. This made it cumbersome to ...

74cce39c5d11a9252d02c2bc0d4dece13fec310d authored over 3 years ago by deeplow <[email protected]>
Merge remote-tracking branch 'origin/pr/46'

* origin/pr/46:
tests: remove creation of .gnupg (enigmail req.)
tests: remove deprecated en...

4f5677284e27cccac7dd041dcf79bace88e58a31 authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
tests: avoid redirect to dogtail repo

The redirect page is blocked via Tor, while the .git URL should work.
See https://gitlab.com/git...

7b259e92dbd8f2342b8df12541d15d0165a01b42 authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
tests: remove creation of .gnupg (enigmail req.)

2ce256380baa6f74ea7b680ebaa2e900e13f3557 authored over 3 years ago by deeplow <[email protected]>
tests: remove deprecated enigmail and tb<78 code

done given all default templates all have thunderbird > 78, where
enigmail is no longer supported.

9db91b9ddd1503ec002409439aa39eb070387df2 authored over 3 years ago by deeplow <[email protected]>
version 2.0.51

9e8baeaff38a6c465d8b1f3d85363d157d8d49dd authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/45'

* origin/pr/45:
test: fix missed clicks on "open file" dialogue

dcaaaf40d6b298829cdae1386a3dfe782093bad8 authored over 3 years ago by Marek Marczykowski-Górecki <[email protected]>
test: fix missed clicks on "open file" dialogue

Tests like
https://openqa.qubes-os.org/tests/19954#step/TC_10_Thunderbird_fedora-34-xfce/1
were ...

e08614310d6371209de542d76afa3c1f93c2677e authored over 3 years ago by deeplow <[email protected]>
tests: generate w/ subkeys in remaining tests

Make remaining tests generate subkeys as well. This way all tests
generate keys the same way (cl...

4eddc753fcd1b63394a5dcc5d37a7d6520af02a0 authored over 3 years ago by deeplow <[email protected]>
tests: master key sign-only + subkey

Version 78.10.2 started supporting offline master keys. This change
broke the configuration wher...

8ee3294f92eab0884e20034285176e7e0da12487 authored over 3 years ago by deeplow <[email protected]>
spec: add BR make

eae1af18f251b9001381fdbccb2928a8a3a46d56 authored over 3 years ago by Frédéric Pierret (fepitre) <[email protected]>
version 2.0.50

aa5f3dc34ceaf9d1254de03207e813d567a893c2 authored about 4 years ago by Marek Marczykowski-Górecki <[email protected]>
tests: adjust key import dialog in debian-10

2d3a0929328f20ddc0b16388bc0a417ca2ae290f authored about 4 years ago by Marek Marczykowski-Górecki <[email protected]>
Merge remote-tracking branch 'origin/pr/39'

* origin/pr/39:
Allow to override defined CFLAGS
Improve reproducibility

6639b71575f145d2c08a94dd24ee776cf025265a authored about 4 years ago by Marek Marczykowski-Górecki <[email protected]>