)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"2520ab2518f27ebeba2dc5e3c68a919592df24db","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Andreas Steil \u003csteil@b1-systems.de\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-07-31 18:26:24 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"OpenStack Security Guide - Chapter 35: Grammatical and Format errors"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Dissolving multiple grammatical errors, inclusive of awkward wording and improper punctuation; recharging \"SSL\" with \"SSL/TLS\""},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baada198_543c59bf","line":7,"updated":"2014-08-01 06:42:42.000000000","message":"\"OpenStack Security Guide - Chapter 35: \" is not necessary. Please give a short summary what you have changed. I think \"Fix grammatical/format errors and restructure syntax\" whould be fine here.","commit_id":"a209424d2563f9b9828e30e5b4a53f2d9c762117"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"2520ab2518f27ebeba2dc5e3c68a919592df24db","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"OpenStack Security Guide - Chapter 35: Grammatical and Format errors"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Dissolving multiple grammatical errors, inclusive of awkward wording and improper punctuation; recharging \"SSL\" with \"SSL/TLS\""},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I45753743c9a88990b2d087ffbbddea6be6d2ff63"},{"line_number":12,"context_line":"Closes-Bug: #1342912"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baada198_f451ed0d","line":9,"updated":"2014-08-01 06:42:42.000000000","message":"Please have a look at https://wiki.openstack.org/wiki/GitCommitMessages how to better format commit messages. I would use a list for the details here.","commit_id":"a209424d2563f9b9828e30e5b4a53f2d9c762117"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"8c83ed6a8042e396f55235aea80a73f59a2be774","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"OpenStack Security Guide - Chapter 35: Grammatical and Format errors"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Dissolving multiple grammatical errors, inclusive of awkward wording and improper punctuation; recharging \"SSL\" with \"SSL/TLS\"; removing whitespaces"},{"line_number":10,"context_line":"Closes-Bug: #1342912"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I45753743c9a88990b2d087ffbbddea6be6d2ff63"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"baada198_3a1b4470","line":9,"updated":"2014-08-01 12:41:20.000000000","message":"Please follow https://wiki.openstack.org/wiki/GitCommitMessages.","commit_id":"cd9aabb27c73d757202d4602eb28ca3f6ee1385e"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"ab59b119e7456d39b5cd4d77487bd0bb7548d66c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Andreas Steil \u003csteil@b1-systems.de\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-08-01 14:44:20 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"OpenStack Security Guide - Chapter 35: Grammatical and Format errors"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Dissolving multiple grammatical errors, inclusive of awkward wording and improper punctuation; recharging \"SSL\" with \"SSL/TLS\"; removing whitespaces; adding insertions"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"baada198_59155b99","line":7,"updated":"2014-08-01 17:57:57.000000000","message":"It\u0027s not anymore chapter 35 after recent rework, so this should be changed if you do another update.","commit_id":"9bb002985a45e2a78a1d42c3ec632ec77ce6c063"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"ab59b119e7456d39b5cd4d77487bd0bb7548d66c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"OpenStack Security Guide - Chapter 35: Grammatical and Format errors"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Dissolving multiple grammatical errors, inclusive of awkward wording and improper punctuation; recharging \"SSL\" with \"SSL/TLS\"; removing whitespaces; adding insertions"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1342912"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"baada198_194043a1","line":9,"updated":"2014-08-01 17:57:57.000000000","message":"This is still a line that is longer than 70 chars, if you need to make additional changes on this patch, please add some line breaks.","commit_id":"9bb002985a45e2a78a1d42c3ec632ec77ce6c063"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"252348f458443708c60d51b1cefaa71bd08e4c46","unresolved":false,"context_lines":[{"line_number":7,"context_line":"OpenStack Security Guide - Chapter 13: Grammatical and format errors"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adapting some wording according to the comments;"},{"line_number":10,"context_line":"removing \": database\" from title;"},{"line_number":11,"context_line":"replacing \"SSL\" by \"SSL/TLS\""},{"line_number":12,"context_line":"Closes-Bug: #1342912"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"baada198_2a28f283","line":10,"updated":"2014-08-05 08:20:30.000000000","message":"Not necessary to mention this. Database was already removed in an other change.","commit_id":"c95e4089272e305ccd1e20d6b948e7f178fe2448"}],"security-guide/section_case-studies-database.xml":[{"author":{"_account_id":2807,"name":"Bryan D. Payne","email":"bdpayne@acm.org","username":"bdpayne"},"change_message_id":"c05c9c1dc79f8a6aa4bb03f6a34e751c6a31d2b9","unresolved":false,"context_lines":[{"line_number":36,"context_line":"to use both passwords and X.509 certificates for authentication."},{"line_number":37,"context_line":"He elects not to use the \u003csystemitem"},{"line_number":38,"context_line":"class\u003d\"service\"\u003enova-conductor\u003c/systemitem\u003e sub-service due to a"},{"line_number":39,"context_line":"desire for fine-grained access control.  \u003c/para\u003e \u003c/section\u003e"},{"line_number":40,"context_line":"\u003c/section\u003e"}],"source_content_type":"application/xml","patch_set":1,"id":"baada198_a8856b5a","line":39,"updated":"2014-07-31 19:58:32.000000000","message":"Extra whitespace here before the \u003c/para\u003e... this is probably why the gate job is failing.  Please fix.","commit_id":"a209424d2563f9b9828e30e5b4a53f2d9c762117"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"ab59b119e7456d39b5cd4d77487bd0bb7548d66c","unresolved":false,"context_lines":[{"line_number":6,"context_line":"  xml:id\u003d\"case-studies-database\"\u003e"},{"line_number":7,"context_line":"  \u003c?dbhtml stop-chunking?\u003e"},{"line_number":8,"context_line":"\u003ctitle\u003eCase studies: database\u003c/title\u003e"},{"line_number":9,"context_line":"\u003cpara\u003eIn this case study we discuss how Alice and Bob would address database"},{"line_number":10,"context_line":"selection and configuration for their respective private and public"},{"line_number":11,"context_line":"clouds.\u003c/para\u003e"},{"line_number":12,"context_line":"\u003csection xml:id\u003d\"case-studies-database-idp38048\"\u003e"}],"source_content_type":"application/xml","patch_set":4,"id":"baada198_3977c709","line":9,"updated":"2014-08-01 17:57:57.000000000","message":"This breaks the intendation as it was before - similar to program code, we add some whitespace to show the structure.","commit_id":"9bb002985a45e2a78a1d42c3ec632ec77ce6c063"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"ab59b119e7456d39b5cd4d77487bd0bb7548d66c","unresolved":false,"context_lines":[{"line_number":14,"context_line":"\u003cpara\u003eAlice\u0027s organization has high availability concerns, thus she has"},{"line_number":15,"context_line":"selected MySQL as the database for the cloud environment. She places the"},{"line_number":16,"context_line":"database on the Management network, utilizing SSL/TLS with mutual"},{"line_number":17,"context_line":"authentication among the services to ensure secure access.  Based on the"},{"line_number":18,"context_line":"assumption that external access of the database will not be facilitated, she"},{"line_number":19,"context_line":"generates self-signed certificates from the organization’s root certificate,"},{"line_number":20,"context_line":"installing the certificates on the database and its access endpoints. Alice"}],"source_content_type":"application/xml","patch_set":4,"id":"baada198_f9470fa2","line":17,"updated":"2014-08-01 17:57:57.000000000","message":"single space after \".\"","commit_id":"9bb002985a45e2a78a1d42c3ec632ec77ce6c063"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"ab59b119e7456d39b5cd4d77487bd0bb7548d66c","unresolved":false,"context_lines":[{"line_number":32,"context_line":"Management network, the database uses certificates signed with the company\u0027s"},{"line_number":33,"context_line":"self-signed root certificate. Bob creates separate user accounts for each"},{"line_number":34,"context_line":"database user, and configures the database to use both passwords and X.509"},{"line_number":35,"context_line":"certificates for authentication.  He elects not to use the \u003csystemitem"},{"line_number":36,"context_line":"class\u003d\"service\"\u003enova-conductor\u003c/systemitem\u003e sub-service due to a desire for"},{"line_number":37,"context_line":"fine-grained access control.\u003c/para\u003e \u003c/section\u003e \u003c/section\u003e"}],"source_content_type":"application/xml","patch_set":4,"id":"baada198_9959b38f","line":35,"updated":"2014-08-01 17:57:57.000000000","message":"Only a single space here, no double ones.","commit_id":"9bb002985a45e2a78a1d42c3ec632ec77ce6c063"},{"author":{"_account_id":2807,"name":"Bryan D. Payne","email":"bdpayne@acm.org","username":"bdpayne"},"change_message_id":"e1f1acef6b1cb93020358613e614facc127553d6","unresolved":false,"context_lines":[{"line_number":5,"context_line":"  version\u003d\"5.0\""},{"line_number":6,"context_line":"  xml:id\u003d\"case-studies-database\"\u003e"},{"line_number":7,"context_line":"    \u003c?dbhtml stop-chunking?\u003e"},{"line_number":8,"context_line":"    \u003ctitle\u003eCase studies: database\u003c/title\u003e"},{"line_number":9,"context_line":"      \u003cpara\u003eIn this case study we discuss how Alice and Bob would address database"},{"line_number":10,"context_line":"      selection and configuration for their respective private and public"},{"line_number":11,"context_line":"      clouds.\u003c/para\u003e"}],"source_content_type":"application/xml","patch_set":5,"id":"baada198_41e0c496","line":8,"updated":"2014-08-04 18:05:18.000000000","message":"We recently removed the \": database\" from the end of this title name.  Please rebase or adjust this patch to not add it back.","commit_id":"776b4027dcb4c05e88648c119bde502d658d5a01"},{"author":{"_account_id":2807,"name":"Bryan D. Payne","email":"bdpayne@acm.org","username":"bdpayne"},"change_message_id":"e1f1acef6b1cb93020358613e614facc127553d6","unresolved":false,"context_lines":[{"line_number":12,"context_line":"  \u003csection xml:id\u003d\"case-studies-database-idp38048\"\u003e"},{"line_number":13,"context_line":"    \u003ctitle\u003eAlice\u0027s private cloud\u003c/title\u003e"},{"line_number":14,"context_line":"      \u003cpara\u003eAlice\u0027s organization has high availability concerns, thus she has"},{"line_number":15,"context_line":"      selected MySQL as the database for the cloud environment. She places the"},{"line_number":16,"context_line":"      database on the Management network, utilizing SSL/TLS with mutual"},{"line_number":17,"context_line":"      authentication among the services to ensure secure access. Based on the"},{"line_number":18,"context_line":"      assumption that external access of the database will not be facilitated, she"}],"source_content_type":"application/xml","patch_set":5,"id":"baada198_b3073316","line":15,"updated":"2014-08-04 18:05:18.000000000","message":"Consider replacing \"thus\" with \"therefore\"?\n\nAlso, \"for the cloud environment\" is a big vague.  The next sentence provides the detail that this goes on the management security domain, so I think that we probably don\u0027t need anything more specific here.  Perhaps just strike \"for the cloud environment\"?","commit_id":"776b4027dcb4c05e88648c119bde502d658d5a01"},{"author":{"_account_id":964,"name":"Anne Gentle","email":"annegentle@justwriteclick.com","username":"annegentle"},"change_message_id":"66bd0292a7dd2f6521afa5548016fb243e40c8ed","unresolved":false,"context_lines":[{"line_number":12,"context_line":"  \u003csection xml:id\u003d\"case-studies-database-idp38048\"\u003e"},{"line_number":13,"context_line":"    \u003ctitle\u003eAlice\u0027s private cloud\u003c/title\u003e"},{"line_number":14,"context_line":"      \u003cpara\u003eAlice\u0027s organization has high availability concerns, thus she has"},{"line_number":15,"context_line":"      selected MySQL as the database for the cloud environment. She places the"},{"line_number":16,"context_line":"      database on the Management network, utilizing SSL/TLS with mutual"},{"line_number":17,"context_line":"      authentication among the services to ensure secure access. Based on the"},{"line_number":18,"context_line":"      assumption that external access of the database will not be facilitated, she"}],"source_content_type":"application/xml","patch_set":5,"id":"baada198_becc538b","line":15,"in_reply_to":"baada198_b3073316","updated":"2014-08-04 22:24:10.000000000","message":"Honestly, I prefer the \"so\" originally used... it\u0027s less formal and easier to read.","commit_id":"776b4027dcb4c05e88648c119bde502d658d5a01"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"84349a25a440ec3cad58e155e1c94bb62d4ab39b","unresolved":false,"context_lines":[{"line_number":16,"context_line":"      the database on the Management network, utilizing SSL/TLS with mutual"},{"line_number":17,"context_line":"      authentication among the services to ensure secure access. Based on the"},{"line_number":18,"context_line":"      assumption that external access of the database will not be facilitated, she"},{"line_number":19,"context_line":"      generates self-signed certificates from the organization’s root certificate,"},{"line_number":20,"context_line":"      installing the certificates on the database and its access endpoints. Alice"},{"line_number":21,"context_line":"      creates separate user accounts for each database user then configures the"},{"line_number":22,"context_line":"      database to use both passwords and X.509 certificates for authentication. She"}],"source_content_type":"application/xml","patch_set":6,"id":"baada198_4a528612","line":19,"updated":"2014-08-05 08:21:36.000000000","message":"Can you please change the ’ (organization’s) into a \u0027.","commit_id":"c95e4089272e305ccd1e20d6b948e7f178fe2448"},{"author":{"_account_id":2807,"name":"Bryan D. Payne","email":"bdpayne@acm.org","username":"bdpayne"},"change_message_id":"d4f76e59119f3735883b98c7788d91c5a426c48f","unresolved":false,"context_lines":[{"line_number":16,"context_line":"      the database on the Management network, utilizing SSL/TLS with mutual"},{"line_number":17,"context_line":"      authentication among the services to ensure secure access. Based on the"},{"line_number":18,"context_line":"      assumption that external access of the database will not be facilitated, she"},{"line_number":19,"context_line":"      generates self-signed certificates from the organization\u0027s root certificate,"},{"line_number":20,"context_line":"      installing the certificates on the database and its access endpoints. Alice"},{"line_number":21,"context_line":"      creates separate user accounts for each database user then configures the"},{"line_number":22,"context_line":"      database to use both passwords and X.509 certificates for authentication. She"}],"source_content_type":"application/xml","patch_set":8,"id":"baada198_67e819e0","line":19,"updated":"2014-08-06 21:12:38.000000000","message":"From a security context, this doesn\u0027t really make sense.  The idea is to sign the certificate with the organization\u0027s root certificate.  This should not be a self-signed certificate.","commit_id":"16f8241277bc49c6868314ad3dff8bcc4a3f52b3"}]}