René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"RFE: Support for add and delete operations in ldif.LDIFParser.parse_change_records()","articleBody":"# Issue description:\r\n\r\nLDIF files with multiple operations are not fully processed by ldif.LDIFParser.parse_change_records().\r\n\r\nOperations of the change type \"modify\" are processed via handle_modify() but operations of the change type \"add\" or \"delete\" are counted but otherwise simply ignored.\r\n\r\nExample of such an LDIF file:\r\n\r\n```\r\nversion: 1\r\n\r\ndn: cn=demo1,ou=groups,dc=example,dc=com\r\nchangetype: add\r\nobjectClass: groupOfNames\r\nobjectClass: nsMemberOf\r\nobjectClass: posixGroup\r\nobjectClass: top\r\ncn: demo1\r\ngidNumber: 99997\r\n\r\n\r\ndn: cn=demo2,ou=groups,dc=example,dc=com\r\nchangetype: add\r\nobjectClass: groupOfNames\r\nobjectClass: nsMemberOf\r\nobjectClass: posixGroup\r\nobjectClass: top\r\ncn: demo2\r\ngidNumber: 99996\r\n\r\n\r\ndn: cn=demo2,ou=groups,dc=example,dc=com\r\nchangetype: modify\r\nadd: description\r\ndescription: Eine zweite Testgruppe\r\n\r\n\r\ndn: cn=demo1,ou=groups,dc=example,dc=com\r\nchangetype: delete\r\n\r\n\r\n```\r\n\r\n\r\n# Steps to reproduce: \r\n\r\nCreate your own LDIF parser and an import for the test file shown:\r\n\r\n```\r\nfrom pathlib import Path\r\nfrom ldif import LDIFParser\r\n\r\nclass MyLDIFParser(LDIFParser):\r\n    \r\n    def __init__(self, input_file, ignored_attr_types=None, max_entries=0, process_url_schemes=None, line_sep='\\n'):\r\n        super().__init__(input_file, ignored_attr_types, max_entries, process_url_schemes, line_sep)\r\n\r\n    def handle(self, dn, entry):\r\n        print(f\"MyLDIFParser::handle({dn})\")\r\n    \r\n    def handle_modify(self, dn, modops, controls=None):\r\n        print(f\"MyLDIFParser::handle_modify({dn})\")\r\n\r\ndef main():\r\n    p_change = Path(\"test_change.ldif\")\r\n    ldifparser_change = MyLDIFParser(open(p_change), \"r\")\r\n    ldifparser_change.parse_change_records()\r\n    print(f\"Changetype counter: {ldifparser_change.changetype_counter}\")\r\n    \r\n    \r\n\r\nif __name__ == '__main__':\r\n    main()\r\n```\r\n\r\nStart this test:\r\n\r\n```\r\n$ ./demo.py \r\nMyLDIFParser::handle_modify(cn=demo2,ou=groups,dc=example,dc=com)\r\nChangetype counter: {'add': 2, 'delete': 1, 'modify': 1, 'modrdn': 0}\r\n```\r\n\r\nOperating system: \r\nFedora 39\r\n\r\nPython version: \r\n- Python 3.6\r\n- Python 3.12\r\n\r\npython-ldap version: \r\n3.4.4","author":{"url":"https://github.com/fager","@type":"Person","name":"fager"},"datePublished":"2024-05-15T19:23:37.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":4},"url":"https://github.com/567/python-ldap/issues/567"}