{"action":"created","issue":{"url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/1540","repository_url":"https://api.github.com/repos/FreeRADIUS/freeradius-server","labels_url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/1540/labels{/name}","comments_url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/1540/comments","events_url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/1540/events","html_url":"https://github.com/FreeRADIUS/freeradius-server/pull/1540","id":133895470,"number":1540,"title":"Make do_python to return the indicated code even when there is no request","user":{"login":"alejandro-perez","id":2082536,"avatar_url":"https://avatars.githubusercontent.com/u/2082536?v=3","gravatar_id":"","url":"https://api.github.com/users/alejandro-perez","html_url":"https://github.com/alejandro-perez","followers_url":"https://api.github.com/users/alejandro-perez/followers","following_url":"https://api.github.com/users/alejandro-perez/following{/other_user}","gists_url":"https://api.github.com/users/alejandro-perez/gists{/gist_id}","starred_url":"https://api.github.com/users/alejandro-perez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alejandro-perez/subscriptions","organizations_url":"https://api.github.com/users/alejandro-perez/orgs","repos_url":"https://api.github.com/users/alejandro-perez/repos","events_url":"https://api.github.com/users/alejandro-perez/events{/privacy}","received_events_url":"https://api.github.com/users/alejandro-perez/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-02-16T06:19:53Z","updated_at":"2016-02-16T16:30:27Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/pulls/1540","html_url":"https://github.com/FreeRADIUS/freeradius-server/pull/1540","diff_url":"https://github.com/FreeRADIUS/freeradius-server/pull/1540.diff","patch_url":"https://github.com/FreeRADIUS/freeradius-server/pull/1540.patch"},"body":"current implementation of do_python() always returns SUCESS when there is no request. Such is the case of instantiate(). Since the\r\n```\r\nif (!request)\t\t\r\n\t\tgoto finish;\r\n```\r\ncode seems to be there to protect the access to request-> elements later, we add the protection in \r\n```\r\nif (PyTuple_CheckExact(pRet) && request != NULL) {\r\n \t\tPyObject *pTupleInt;\t\t \t\tPyObject *pTupleInt;\r\n```\r\nthat is the only place in the remaining code of the method request is used. "},"comment":{"url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/comments/184755445","html_url":"https://github.com/FreeRADIUS/freeradius-server/pull/1540#issuecomment-184755445","issue_url":"https://api.github.com/repos/FreeRADIUS/freeradius-server/issues/1540","id":184755445,"user":{"login":"alejandro-perez","id":2082536,"avatar_url":"https://avatars.githubusercontent.com/u/2082536?v=3","gravatar_id":"","url":"https://api.github.com/users/alejandro-perez","html_url":"https://github.com/alejandro-perez","followers_url":"https://api.github.com/users/alejandro-perez/followers","following_url":"https://api.github.com/users/alejandro-perez/following{/other_user}","gists_url":"https://api.github.com/users/alejandro-perez/gists{/gist_id}","starred_url":"https://api.github.com/users/alejandro-perez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alejandro-perez/subscriptions","organizations_url":"https://api.github.com/users/alejandro-perez/orgs","repos_url":"https://api.github.com/users/alejandro-perez/repos","events_url":"https://api.github.com/users/alejandro-perez/events{/privacy}","received_events_url":"https://api.github.com/users/alejandro-perez/received_events","type":"User","site_admin":false},"created_at":"2016-02-16T16:30:26Z","updated_at":"2016-02-16T16:30:26Z","body":"https://github.com/FreeRADIUS/freeradius-server/pull/1541"}}