)]}'
{"keystone/conf/security_compliance.py":[{"author":{"_account_id":16465,"name":"Kristi Nikolla","email":"knikolla@bu.edu","username":"knikolla"},"change_message_id":"d4d6c3d2f883444283db9e8fffdeb7bd208241fb","unresolved":false,"context_lines":[{"line_number":59,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":60,"context_line":"The number of days for which a password will be considered valid before"},{"line_number":61,"context_line":"requiring it to be changed. This feature is considered disabled if not defined"},{"line_number":62,"context_line":"in configuration, which is the default behavior. This feature can be enabled by"},{"line_number":63,"context_line":"setting the value to any number greater than the minimum value of 1. If"},{"line_number":64,"context_line":"enabled, new password changes will have an expiration date, however existing"},{"line_number":65,"context_line":"passwords would not be impacted. This feature depends on the `sql` backend for"},{"line_number":66,"context_line":"the `[identity] driver`."}],"source_content_type":"text/x-python","patch_set":2,"id":"5ff73747_cf334062","line":63,"range":{"start_line":62,"start_character":49,"end_line":63,"end_character":67},"updated":"2017-05-03 15:40:00.000000000","message":"So setting this to 1 it would still be disabled?","commit_id":"dea317dfaa7951d91f0c35f726bb3f892201bd1a"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"91a8b0f6bc818f80785b442d5ae995055ce42be5","unresolved":false,"context_lines":[{"line_number":60,"context_line":"The number of days for which a password will be considered valid before"},{"line_number":61,"context_line":"requiring it to be changed. This feature is considered disabled if not defined"},{"line_number":62,"context_line":"in configuration, which is the default behavior. This feature can be enabled by"},{"line_number":63,"context_line":"setting the value to any number greater than the minimum value of 1. If"},{"line_number":64,"context_line":"enabled, new password changes will have an expiration date, however existing"},{"line_number":65,"context_line":"passwords would not be impacted. This feature depends on the `sql` backend for"},{"line_number":66,"context_line":"the `[identity] driver`."}],"source_content_type":"text/x-python","patch_set":2,"id":"1f013ff3_d7563b07","line":63,"in_reply_to":"1f013ff3_3c1e64ba","updated":"2017-05-15 21:54:13.000000000","message":"Good point - I can take another stab at fixing this up.","commit_id":"dea317dfaa7951d91f0c35f726bb3f892201bd1a"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"7a3e286617f0f045285de4c9c236e4db5c245ba4","unresolved":false,"context_lines":[{"line_number":60,"context_line":"The number of days for which a password will be considered valid before"},{"line_number":61,"context_line":"requiring it to be changed. This feature is considered disabled if not defined"},{"line_number":62,"context_line":"in configuration, which is the default behavior. This feature can be enabled by"},{"line_number":63,"context_line":"setting the value to any number greater than the minimum value of 1. If"},{"line_number":64,"context_line":"enabled, new password changes will have an expiration date, however existing"},{"line_number":65,"context_line":"passwords would not be impacted. This feature depends on the `sql` backend for"},{"line_number":66,"context_line":"the `[identity] driver`."}],"source_content_type":"text/x-python","patch_set":2,"id":"1f013ff3_510b568d","line":63,"in_reply_to":"1f013ff3_d7563b07","updated":"2017-05-16 00:37:34.000000000","message":"++","commit_id":"dea317dfaa7951d91f0c35f726bb3f892201bd1a"},{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"b4bee460a6a1f444b608bc56becafaedaf446663","unresolved":false,"context_lines":[{"line_number":59,"context_line":"    help\u003dutils.fmt(\"\"\""},{"line_number":60,"context_line":"The number of days for which a password will be considered valid before"},{"line_number":61,"context_line":"requiring it to be changed. This feature is considered disabled if not defined"},{"line_number":62,"context_line":"in configuration, which is the default behavior. This feature can be enabled by"},{"line_number":63,"context_line":"setting the value to any number greater than the minimum value of 1. If"},{"line_number":64,"context_line":"enabled, new password changes will have an expiration date, however existing"},{"line_number":65,"context_line":"passwords would not be impacted. This feature depends on the `sql` backend for"},{"line_number":66,"context_line":"the `[identity] driver`."}],"source_content_type":"text/x-python","patch_set":2,"id":"1f013ff3_3c1e64ba","line":63,"range":{"start_line":62,"start_character":49,"end_line":63,"end_character":67},"in_reply_to":"5ff73747_cf334062","updated":"2017-05-15 21:43:20.000000000","message":"Yeah this part confuses me. It looks like \u0027None\u0027 and \u00271\u0027 are disabled, with anything \u003e 1 enabled?","commit_id":"dea317dfaa7951d91f0c35f726bb3f892201bd1a"}],"keystone/tests/unit/test_cli.py":[{"author":{"_account_id":9796,"name":"ChangBo Guo","email":"glongwave@gmail.com","username":"gcb"},"change_message_id":"7c20e359fe68e205d2b2ebae68a248aea9ce634c","unresolved":false,"context_lines":[{"line_number":1184,"context_line":"        # option undefined to show that it is actually disabled since `None` is"},{"line_number":1185,"context_line":"        # returned for options that are registered without a default but not"},{"line_number":1186,"context_line":"        # defined in a configuration file."},{"line_number":1187,"context_line":"        self.config_fixture.config(group\u003d\u0027security_compliance\u0027,"},{"line_number":1188,"context_line":"                                   minimum_password_age\u003d0)"},{"line_number":1189,"context_line":"        self.assertFalse("},{"line_number":1190,"context_line":"            security_compliance."},{"line_number":1191,"context_line":"            symptom_minimum_password_age_greater_than_expires_days())"}],"source_content_type":"text/x-python","patch_set":1,"id":"5ff73747_a3a38300","line":1188,"range":{"start_line":1187,"start_character":0,"end_line":1188,"end_character":58},"updated":"2017-05-03 06:34:11.000000000","message":"0 is the default value of config option \u0027minimum_password_age\u003d0\u0027, so do need this line.","commit_id":"b2ea2ca80d8e5e54227e956d00edd4298125988f"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"16b70d03cb90af88ad7719d92558d67f3a211549","unresolved":false,"context_lines":[{"line_number":1185,"context_line":"        # returned for options that are registered without a default but not"},{"line_number":1186,"context_line":"        # defined in a configuration file."},{"line_number":1187,"context_line":"        self.config_fixture.config(group\u003d\u0027security_compliance\u0027,"},{"line_number":1188,"context_line":"                                   minimum_password_age\u003d0)"},{"line_number":1189,"context_line":"        self.assertFalse("},{"line_number":1190,"context_line":"            security_compliance."},{"line_number":1191,"context_line":"            symptom_minimum_password_age_greater_than_expires_days())"}],"source_content_type":"text/x-python","patch_set":1,"id":"5ff73747_c3e8c17d","line":1188,"in_reply_to":"5ff73747_a3a38300","updated":"2017-05-03 13:49:22.000000000","message":"Good catch - I\u0027ve cherry-picked this fix on top of yours so that we can merge the comment.","commit_id":"b2ea2ca80d8e5e54227e956d00edd4298125988f"}]}