)]}'
{"metadata.rb":[{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"9ac23fea78ca2c6867bb352eea8c11cb96686a36","unresolved":false,"context_lines":[{"line_number":3,"context_line":"maintainer_email \u0027opscode-chef-openstack@googlegroups.com\u0027"},{"line_number":4,"context_line":"license          \u0027Apache 2.0\u0027"},{"line_number":5,"context_line":"description      \u0027Installs/Configures the OpenStack Dashboard (Horizon)\u0027"},{"line_number":6,"context_line":"long_description IO.read(File.join(File.dirname(__FILE__), \u0027README.md\u0027))"},{"line_number":7,"context_line":"version          \u002711.0.0\u0027"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"recipe           \u0027openstack-dashboard::horizon\u0027, \u0027Sets up the Horizon dashboard.\u0027"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"baa041b7_ea3411fc","line":6,"updated":"2015-04-03 11:23:26.000000000","message":"Should there be a version change with the removal of a couple attributes?","commit_id":"66c38982a40bfe1f7dcdfd3f99269d93b4005d13"},{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"b43545b41cdf787e83e64ee55a4a95b8cfb2b524","unresolved":false,"context_lines":[{"line_number":3,"context_line":"maintainer_email \u0027opscode-chef-openstack@googlegroups.com\u0027"},{"line_number":4,"context_line":"license          \u0027Apache 2.0\u0027"},{"line_number":5,"context_line":"description      \u0027Installs/Configures the OpenStack Dashboard (Horizon)\u0027"},{"line_number":6,"context_line":"long_description IO.read(File.join(File.dirname(__FILE__), \u0027README.md\u0027))"},{"line_number":7,"context_line":"version          \u002711.0.0\u0027"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"recipe           \u0027openstack-dashboard::horizon\u0027, \u0027Sets up the Horizon dashboard.\u0027"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"baa041b7_a797578f","line":6,"in_reply_to":"baa041b7_ea3411fc","updated":"2015-04-03 15:20:33.000000000","message":"yeah, maybe.  We are trying to reduce the number of version changes during master development as it tends to slow us down.\nSince this change does not effect other cookbook dependencies within master, I don\u0027t think a version bump is necessary, the new version 11 is good enough for this type of release work.","commit_id":"66c38982a40bfe1f7dcdfd3f99269d93b4005d13"}],"spec/apache2-server_spec.rb":[{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"9ac23fea78ca2c6867bb352eea8c11cb96686a36","unresolved":false,"context_lines":[{"line_number":171,"context_line":"              node.set[\u0027openstack\u0027][\u0027endpoints\u0027][\u0027dashboard-http-bind\u0027][\u0027port\u0027] \u003d 81"},{"line_number":172,"context_line":"              node.set[\u0027openstack\u0027][\u0027endpoints\u0027][\u0027dashboard-https-bind\u0027][\u0027port\u0027] \u003d https_port_value"},{"line_number":173,"context_line":"              expect(chef_run).to render_file(file.name)"},{"line_number":174,"context_line":"                .with_content(%r(^\\s*RewriteRule \\^\\(\\.\\*\\)\\$ https://%\\{SERVER_NAME\\}:#{https_port_value}%\\{REQUEST_URI\\} \\[L,R\\]$))"},{"line_number":175,"context_line":"            end"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"            it \u0027shows the parameterized rewrite rule when https_port is different from 443\u0027 do"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"baa041b7_4a82dd95","line":174,"updated":"2015-04-03 11:23:26.000000000","message":"Shouldn\u0027t this example verify the http port and not https?","commit_id":"66c38982a40bfe1f7dcdfd3f99269d93b4005d13"},{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"b43545b41cdf787e83e64ee55a4a95b8cfb2b524","unresolved":false,"context_lines":[{"line_number":171,"context_line":"              node.set[\u0027openstack\u0027][\u0027endpoints\u0027][\u0027dashboard-http-bind\u0027][\u0027port\u0027] \u003d 81"},{"line_number":172,"context_line":"              node.set[\u0027openstack\u0027][\u0027endpoints\u0027][\u0027dashboard-https-bind\u0027][\u0027port\u0027] \u003d https_port_value"},{"line_number":173,"context_line":"              expect(chef_run).to render_file(file.name)"},{"line_number":174,"context_line":"                .with_content(%r(^\\s*RewriteRule \\^\\(\\.\\*\\)\\$ https://%\\{SERVER_NAME\\}:#{https_port_value}%\\{REQUEST_URI\\} \\[L,R\\]$))"},{"line_number":175,"context_line":"            end"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"            it \u0027shows the parameterized rewrite rule when https_port is different from 443\u0027 do"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"baa041b7_87ca136d","line":174,"in_reply_to":"baa041b7_4a82dd95","updated":"2015-04-03 15:20:33.000000000","message":"No, this logic is in the SSL section and effects the ssl port. See line 15 in the dash-site template.  \nBut I\u0027ll update the description a bit in that regard.","commit_id":"66c38982a40bfe1f7dcdfd3f99269d93b4005d13"}]}