diff options
| author | Werner Koch <[email protected]> | 2020-12-14 14:18:01 +0000 |
|---|---|---|
| committer | Werner Koch <[email protected]> | 2020-12-14 18:29:49 +0000 |
| commit | cc056eb534c1b8f7d1a90af3b9ecb9d6b2f322fa (patch) | |
| tree | fe95dcd9ff5ae786516005551415ad0086245f56 /g10/call-dirmngr.c | |
| parent | dirmngr: Fix adding keys to an LDAP server. (diff) | |
| download | gnupg-cc056eb534c1b8f7d1a90af3b9ecb9d6b2f322fa.tar.gz gnupg-cc056eb534c1b8f7d1a90af3b9ecb9d6b2f322fa.zip | |
dirmngr: Do not store the useless pgpSignerID in the LDAP.
* dirmngr/ks-engine-ldap.c (extract_attributes): Do not store the
pgpSignerID.
* g10/call-dirmngr.c (ks_put_inq_cb): Do not emit sig records.
--
The pgpSignerID has no use in the LDAP and thus don't store it.
David's idea back in 2004 was
/* This bit is really for the benefit of people who
store their keys in LDAP servers. It makes it easy
to do queries for things like "all keys signed by
Isabella". */
See-commit: 3ddd4410aef928827e1c8d4fb02c1ccd3f8eaea5
I consider this dangerous because such a query is not able to validate
the signature, does not get revocation signatures, and also has no
information about the validity of the signatures. Further many keys
are spammed tehse days with faked signatures and it does not make
sense to blow up the LDAP with such garbage.
Signed-off-by: Werner Koch <[email protected]>
Diffstat (limited to 'g10/call-dirmngr.c')
| -rw-r--r-- | g10/call-dirmngr.c | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/g10/call-dirmngr.c b/g10/call-dirmngr.c index 1282ae192..1a4d896c6 100644 --- a/g10/call-dirmngr.c +++ b/g10/call-dirmngr.c @@ -1015,21 +1015,6 @@ ks_put_inq_cb (void *opaque, const char *line) } break; - /* This bit is really for the benefit of people who - store their keys in LDAP servers. It makes it easy - to do queries for things like "all keys signed by - Isabella". */ - case PKT_SIGNATURE: - { - PKT_signature *sig = node->pkt->pkt.signature; - - if (IS_UID_SIG (sig)) - record_output (fp, node->pkt->pkttype, NULL, - -1, -1, sig->keyid, - sig->timestamp, sig->expiredate, NULL); - } - break; - default: continue; } |
