aboutsummaryrefslogtreecommitdiffstats
path: root/dirmngr/ks-engine-ldap.c
diff options
context:
space:
mode:
authorWerner Koch <[email protected]>2020-12-14 14:18:01 +0000
committerWerner Koch <[email protected]>2020-12-14 18:29:49 +0000
commitcc056eb534c1b8f7d1a90af3b9ecb9d6b2f322fa (patch)
treefe95dcd9ff5ae786516005551415ad0086245f56 /dirmngr/ks-engine-ldap.c
parentdirmngr: Fix adding keys to an LDAP server. (diff)
downloadgnupg-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 'dirmngr/ks-engine-ldap.c')
-rw-r--r--dirmngr/ks-engine-ldap.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/dirmngr/ks-engine-ldap.c b/dirmngr/ks-engine-ldap.c
index 11c922eac..7d61313d9 100644
--- a/dirmngr/ks-engine-ldap.c
+++ b/dirmngr/ks-engine-ldap.c
@@ -1737,9 +1737,6 @@ extract_attributes (LDAPMod ***modlist, char *line)
if (is_sub)
modlist_add (modlist, "pgpSubKeyID", keyid);
-
- if (is_sig)
- modlist_add (modlist, "pgpSignerID", keyid);
}
if (is_pub)
@@ -1967,7 +1964,6 @@ ks_ldap_put (ctrl_t ctrl, parsed_uri_t uri,
modlist_add (&modlist, "pgpKeyType", NULL);
modlist_add (&modlist, "pgpUserID", NULL);
modlist_add (&modlist, "pgpKeyCreateTime", NULL);
- modlist_add (&modlist, "pgpSignerID", NULL);
modlist_add (&modlist, "pgpRevoked", NULL);
modlist_add (&modlist, "pgpSubKeyID", NULL);
modlist_add (&modlist, "pgpKeySize", NULL);