René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Feb 10 change to ldap_attribute_store.py broke 'no ldap record found'.","articleBody":"Previously there was a section of code that started on line 586 which handles whether or not an LDAP record was found. It takes the following format.\n```\nif record:\n    ...\nelse:\n   ...\n```\n\nOn the commit found here: https://github.com/IdentityPython/SATOSA/commit/4b9c83b966392c3f732a6a85c63994e6f832643e \nThat got nested under a for loop in attempt to process multiple records like such:\n```\nfor record in responses:\n    if record:\n        ...\n    else:\n        ...\n```\n\nThe problem is that if no record exists in responses, the else clause will never be executed, because the for loop will be entirely skipped. (That is of course after you get passed the error of 'responses' not being assigned: \"UnboundLocalError: local variable 'responses' referenced before\").\n\nIt's likely that it was intended to be something like below instead.\n```\n# Address responses being an Unboundlocal when no record is found, then...\n\nif responses:\n    for record in responses:\n        ...\nelse:\n    ...\n```\n\nI've posted the Pull Request to resolve this: [Fix for issue 488](https://github.com/IdentityPython/SATOSA/pull/489)","author":{"url":"https://github.com/aaschenbrener","@type":"Person","name":"aaschenbrener"},"datePublished":"2025-04-24T15:00:57.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/488/SATOSA/issues/488"}