{"action":"created","issue":{"url":"https://api.github.com/repos/SSSD/sssd/issues/33","repository_url":"https://api.github.com/repos/SSSD/sssd","labels_url":"https://api.github.com/repos/SSSD/sssd/issues/33/labels{/name}","comments_url":"https://api.github.com/repos/SSSD/sssd/issues/33/comments","events_url":"https://api.github.com/repos/SSSD/sssd/issues/33/events","html_url":"https://github.com/SSSD/sssd/pull/33","id":179147971,"number":33,"title":"SECRETS: Some small misc fixes + fixing #3168","user":{"login":"fidencio","id":112762,"avatar_url":"https://avatars.githubusercontent.com/u/112762?v=3","gravatar_id":"","url":"https://api.github.com/users/fidencio","html_url":"https://github.com/fidencio","followers_url":"https://api.github.com/users/fidencio/followers","following_url":"https://api.github.com/users/fidencio/following{/other_user}","gists_url":"https://api.github.com/users/fidencio/gists{/gist_id}","starred_url":"https://api.github.com/users/fidencio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fidencio/subscriptions","organizations_url":"https://api.github.com/users/fidencio/orgs","repos_url":"https://api.github.com/users/fidencio/repos","events_url":"https://api.github.com/users/fidencio/events{/privacy}","received_events_url":"https://api.github.com/users/fidencio/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2016-09-26T06:22:11Z","updated_at":"2016-09-30T13:23:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/SSSD/sssd/pulls/33","html_url":"https://github.com/SSSD/sssd/pull/33","diff_url":"https://github.com/SSSD/sssd/pull/33.diff","patch_url":"https://github.com/SSSD/sssd/pull/33.patch"},"body":"This series was made on top of Jakub's patches adding Secrets'\ndocumentation, that is applied on top of my own patches adding\na the update-confdb-process.\n\nThe patches that needs to be reviewed are the last 6 patches, and\nthey're listed below:\n f587a7c (Fabiano Fidêncio, 3 days ago)\n    SECRETS: Add a configurable depth limit for nested containers\n\n    Resolves: https://fedorahosted.org/sssd/ticket/3168\n\n    Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>\n\n e98bd63 (Fabiano Fidêncio, 7 hours ago)\n    SECRETS: Use a tmp_context on local_db_check_containers()\n\n    Otherwise the struct ldb_dn will be hanging on the mem_ctx till it gets\n    freed.\n\n    Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>\n\n 4a90c20 (Fabiano Fidêncio, 7 hours ago)\n    SECRETS: Make functions from local.c static\n\n    There's no reason for those functions to be exposed.\n\n    Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>\n\n 67e1301 (Fabiano Fidêncio, 10 hours ago)\n    CONFIG: Add secrets' provider option\n\n    The provider option specifies where should the secrets be stored.\n\n    Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>\n\n 5de385f (Jakub Hrozek, 6 weeks ago)\n    CONFIG: List allowed secrets responder options\n\n    Co-author: Fabiano Fidêncio <fidencio@redhat.com>\n\n 1f51d50 (Fabiano Fidêncio, 11 hours ago)\n    CONFIG: Add secrets responder to the allowed sections\n\n    Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>\n\nIn the last patch (f587a7c) I'm adding a test for the maximum depth,\nbut I decided to, at least for now, do this test based on the default\nvalue set for this option instead of trying to implemenet something in\npython to read the config entries from the ConfDB file."},"comment":{"url":"https://api.github.com/repos/SSSD/sssd/issues/comments/250743016","html_url":"https://github.com/SSSD/sssd/pull/33#issuecomment-250743016","issue_url":"https://api.github.com/repos/SSSD/sssd/issues/33","id":250743016,"user":{"login":"fidencio","id":112762,"avatar_url":"https://avatars.githubusercontent.com/u/112762?v=3","gravatar_id":"","url":"https://api.github.com/users/fidencio","html_url":"https://github.com/fidencio","followers_url":"https://api.github.com/users/fidencio/followers","following_url":"https://api.github.com/users/fidencio/following{/other_user}","gists_url":"https://api.github.com/users/fidencio/gists{/gist_id}","starred_url":"https://api.github.com/users/fidencio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fidencio/subscriptions","organizations_url":"https://api.github.com/users/fidencio/orgs","repos_url":"https://api.github.com/users/fidencio/repos","events_url":"https://api.github.com/users/fidencio/events{/privacy}","received_events_url":"https://api.github.com/users/fidencio/received_events","type":"User","site_admin":false},"created_at":"2016-09-30T13:23:00Z","updated_at":"2016-09-30T13:23:00Z","body":"Okay, with this last patch series config-check works like a charm for [secrets] and [secrets/users/$uid].\r\nAlso the name of the newly introduced property was renamed to containers_nest_limit (and all the related vars/constants as well)."}}
{"actor":{"display_login":"fidencio"}}