{"action":"created","issue":{"url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/1642","repository_url":"https://api.github.com/repos/FreeRADIUS/freeradius-server","labels_url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/1642/labels{/name}","comments_url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/1642/comments","events_url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/1642/events","html_url":"https://github.com/FreeRADIUS/freeradius-server/issues/1642","id":164129484,"number":1642,"title":"LDAP-Group et al need to trigger a load of rlm_ldap","user":{"login":"arr2036","id":791758,"avatar_url":"https://avatars.githubusercontent.com/u/791758?v=3","gravatar_id":"","url":"https://api.github.com/users/arr2036","html_url":"https://github.com/arr2036","followers_url":"https://api.github.com/users/arr2036/followers","following_url":"https://api.github.com/users/arr2036/following{/other_user}","gists_url":"https://api.github.com/users/arr2036/gists{/gist_id}","starred_url":"https://api.github.com/users/arr2036/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arr2036/subscriptions","organizations_url":"https://api.github.com/users/arr2036/orgs","repos_url":"https://api.github.com/users/arr2036/repos","events_url":"https://api.github.com/users/arr2036/events{/privacy}","received_events_url":"https://api.github.com/users/arr2036/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-07-06T17:23:05Z","updated_at":"2016-07-06T21:16:31Z","closed_at":null,"body":"# Issue type\r\n-  [x ] Defect - Unexpected behaviour (obvious or verified by project member).\r\n\r\n# Defect/Feature description\r\n## How to reproduce issue\r\nUse LDAP-Group in a condition, without listing ldap anywhere in the server.\r\n\r\n## Technical issue\r\nBecause the ldap module is not listed anywhere its bootstrap method is never executed.\r\n\r\n## Resolution\r\nThere's a few ways that this could be resolved.\r\n\r\nThe easiest would be to always run bootstrap for every module, so the pair comparisons are registered, and then add instantiation callbacks for paircmps,  as we do with xlats and maps already.\r\n\r\nThen a paircmp is instantiated it could trigger full instantiation of the module.\r\n\r\nAnother way would be to add proper functions and remove paircmps completely."},"comment":{"url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/comments/230909495","html_url":"https://github.com/FreeRADIUS/freeradius-server/issues/1642#issuecomment-230909495","issue_url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/1642","id":230909495,"user":{"login":"mcnewton","id":1319836,"avatar_url":"https://avatars.githubusercontent.com/u/1319836?v=3","gravatar_id":"","url":"https://api.github.com/users/mcnewton","html_url":"https://github.com/mcnewton","followers_url":"https://api.github.com/users/mcnewton/followers","following_url":"https://api.github.com/users/mcnewton/following{/other_user}","gists_url":"https://api.github.com/users/mcnewton/gists{/gist_id}","starred_url":"https://api.github.com/users/mcnewton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mcnewton/subscriptions","organizations_url":"https://api.github.com/users/mcnewton/orgs","repos_url":"https://api.github.com/users/mcnewton/repos","events_url":"https://api.github.com/users/mcnewton/events{/privacy}","received_events_url":"https://api.github.com/users/mcnewton/received_events","type":"User","site_admin":false},"created_at":"2016-07-06T21:16:31Z","updated_at":"2016-07-06T21:16:31Z","body":"Not sure how this can be an issue. If `mods-enabled/ldap` exists, then `ldap` is instantiated with that config, and `LDAP-Group` will work fine.\r\n\r\nIf it hasn't been instantiated, then there is no config. So `LDAP-Group` is meaningless because even if you called bootstrap there would be no ldap server configuration to use, so it wouldn't know what to do.\r\n\r\nOr am I missing something?"}}
{"actor":{"display_login":"mcnewton"}}