{"id":494,"date":"2011-07-29T09:07:06","date_gmt":"2011-07-29T16:07:06","guid":{"rendered":"http:\/\/blog.mozilla.org\/security\/?p=494"},"modified":"2011-07-29T09:08:03","modified_gmt":"2011-07-29T16:08:03","slug":"494","status":"publish","type":"post","link":"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/","title":{"rendered":"Evolving the Security Review and Discussion Process"},"content":{"rendered":"<p><em>&#8220;The journey of a thousand miles begins with one step.&#8221; ~ Lao Tzu<\/em><br \/>\n<em>&#8220;If you do what you&#8217;ve always done, you&#8217;ll get what you&#8217;ve always gotten.&#8221; ~ Anthony Robbins<\/em><\/p>\n<p>We&#8217;ve been thinking about and working and retooling our security review process over the last few months with a set of goals in mind.<\/p>\n<p><strong> Review more items and review them early in the development process.<\/strong><br \/>\nPretty straight forward, find more features\/bugs\/ideas that need to be looked at. Schedule them and find stuff. The challenge here was getting much more plugged into the development process, especially for desktop Firefox which is where we started our focus. As things move along we are branching out into mobile, messaging, and other projects. We also want teams to come to us instead of us having to chase them.<\/p>\n<p><strong>Reviews need to be useful to all involved.<\/strong><br \/>\nThe core idea that security has to be add value, not just some time that people spend in a room talking about theory or speed bump on the ship lane. The outcome of meetings with security should be valuable and should focus on finding a path through, not stopping because we find a &#8220;problem&#8221;. When we had blockers we had to be clear about why it&#8217;s a blocker and what needs to be done. When it&#8217;s not a blocker it had to have a severity and the importance communicated clearly for future prioritization. And if the meeting was done early, it was done early. Time is a resource we cannot replace or replenish and it has to be used wisely. If we wanted to get the first goal this had to happen. And the format of the meeting had to be more firm.<\/p>\n<p><strong>Results of a review should be easy to find and available for review.<\/strong><br \/>\nWe want everyone interested in security and the security of our products to review our work, be critical in feedback and find ways to improve. That means publishing what we did, when we did it and what if any actions were to come from the meetings. We did just that, and you can find it here https:\/\/wiki.mozilla.org\/Security\/Reviews.<\/p>\n<p><strong>Meetings need to be practicable, open, and publicized.<\/strong><br \/>\nWith the previous goals in mind the meeting format changed to something like this:<\/p>\n<ul>\n<li>\u00a0Introduce Feature (5-10 minutes) [can be answered ahead of time to save meeting time]<\/li>\n<li>\u00a0Goal of Feature, what is trying to be achieved (problem solved, use cases, etc)<\/li>\n<ul>\n<li>\u00a0What solutions\/approaches were considered other than the proposed solution?<\/li>\n<li>\u00a0Why was this solution chosen?<\/li>\n<li>\u00a0Any security threats already considered in the design and why?<\/li>\n<\/ul>\n<li>Threat Brainstorming (30-40 minutes)<\/li>\n<li>Conclusions \/ Action Items (10-20 minutes)<\/li>\n<\/ul>\n<p>We also created a public calendar so that anyone who was interested could join in and share their knowledge; available as <a href=\"https:\/\/mail.mozilla.com\/home\/ckoenig@mozilla.com\/Security%20Review.html\">HTML<\/a>and <a href=\"https:\/\/mail.mozilla.com\/home\/ckoenig@mozilla.com\/Security%20Review\">.ICS<\/a>. Every meeting also contains the information on how to dial-in and interact during the meetings.<\/p>\n<p><strong>Results So Far<\/strong><\/p>\n<p>So far this has had some good outcomes, we found and proactively fixed items in both <a href=\"https:\/\/wiki.mozilla.org\/Security\/Reviews\/Firefox5\/ReviewNotes\/CSSAni\">CSS Animations<\/a> and <a href=\"https:\/\/wiki.mozilla.org\/Security\/Reviews\/Firefox6\/ReviewNotes\/ServerDomEvents\">Server Sent DOM Events<\/a> as examples. As well as having a large number of very productive conversations early in the development process with many teams that will yields results as we continue to work with them. By changing our focus to one of early intervention and positive paths we are also building a stronger culture of security focus across the Mozilla culture.<\/p>\n<p><strong>Other Related Changes<\/strong><\/p>\n<p>We also added some <a href=\"https:\/\/spartiates.wordpress.com\/2011\/07\/18\/security-keywords-in-bugzilla\/ \">keywords to Bugzilla<\/a> and some new items to the <a href=\"https:\/\/spartiates.wordpress.com\/2011\/07\/19\/security-feature-pages\/\">feature pages<\/a> to help track our work, identify things that may need to be looked at and and allow others to nominate items for our review. Since I wrote about these on my personal blog I won&#8217;t repeat them here.<\/p>\n<p><strong>Looking Forward<\/strong><\/p>\n<p>We&#8217;ve made what I think are very useful changes that allow us to be more &#8220;fleet of foot&#8221; with the new rapid release process. We&#8217;ve done it in a uniquely Mozilla way while still leveraging from best practices from many in the industry. And it&#8217;s a great starting\u00a0 point for even more positive change. We strive to give our users the most secure browsing experience, with a product they can trust and that puts privacy and security at the forefront.<\/p>\n<p>As we evolve the process we encourage you to get involved by joining in our meetings, the wiki and discussions, as we do I will continue to write about what we are doing and share our journey with you.<\/p>\n<p>-curtis<\/p>\n","protected":false},"excerpt":{"rendered":"<p>&#8220;The journey of a thousand miles begins with one step.&#8221; ~ Lao Tzu &#8220;If you do what you&#8217;ve always done, you&#8217;ll get what you&#8217;ve always gotten.&#8221; ~ Anthony Robbins We&#8217;ve &hellip; <a class=\"go\" href=\"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/\">Read more<\/a><\/p>\n","protected":false},"author":315,"featured_media":0,"comment_status":"closed","ping_status":"closed","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[78,69],"tags":[4462,4461],"coauthors":[],"yoast_head":"<!-- This site is optimized with the Yoast SEO plugin v22.5 - https:\/\/yoast.com\/wordpress\/plugins\/seo\/ -->\n<title>Evolving the Security Review and Discussion Process - Mozilla Security Blog<\/title>\n<meta name=\"robots\" content=\"index, follow, max-snippet:-1, max-image-preview:large, max-video-preview:-1\" \/>\n<link rel=\"canonical\" href=\"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/\" \/>\n<meta name=\"twitter:label1\" content=\"Written by\" \/>\n\t<meta name=\"twitter:data1\" content=\"Curtis Koenig\" \/>\n\t<meta name=\"twitter:label2\" content=\"Est. reading time\" \/>\n\t<meta name=\"twitter:data2\" content=\"4 minutes\" \/>\n<script type=\"application\/ld+json\" class=\"yoast-schema-graph\">{\"@context\":\"https:\/\/schema.org\",\"@graph\":[{\"@type\":\"WebPage\",\"@id\":\"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/\",\"url\":\"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/\",\"name\":\"Evolving the Security Review and Discussion Process - Mozilla Security Blog\",\"isPartOf\":{\"@id\":\"https:\/\/blog.mozilla.org\/security\/#website\"},\"datePublished\":\"2011-07-29T16:07:06+00:00\",\"dateModified\":\"2011-07-29T16:08:03+00:00\",\"author\":{\"@id\":\"https:\/\/blog.mozilla.org\/security\/#\/schema\/person\/792ed382fa382861f5deff578b21429c\"},\"breadcrumb\":{\"@id\":\"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/#breadcrumb\"},\"inLanguage\":\"en-US\",\"potentialAction\":[{\"@type\":\"ReadAction\",\"target\":[\"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/\"]}]},{\"@type\":\"BreadcrumbList\",\"@id\":\"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/#breadcrumb\",\"itemListElement\":[{\"@type\":\"ListItem\",\"position\":1,\"name\":\"Home\",\"item\":\"https:\/\/blog.mozilla.org\/security\/\"},{\"@type\":\"ListItem\",\"position\":2,\"name\":\"Evolving the Security Review and Discussion Process\"}]},{\"@type\":\"WebSite\",\"@id\":\"https:\/\/blog.mozilla.org\/security\/#website\",\"url\":\"https:\/\/blog.mozilla.org\/security\/\",\"name\":\"Mozilla Security Blog\",\"description\":\"\",\"potentialAction\":[{\"@type\":\"SearchAction\",\"target\":{\"@type\":\"EntryPoint\",\"urlTemplate\":\"https:\/\/blog.mozilla.org\/security\/?s={search_term_string}\"},\"query-input\":\"required name=search_term_string\"}],\"inLanguage\":\"en-US\"},{\"@type\":\"Person\",\"@id\":\"https:\/\/blog.mozilla.org\/security\/#\/schema\/person\/792ed382fa382861f5deff578b21429c\",\"name\":\"Curtis Koenig\",\"image\":{\"@type\":\"ImageObject\",\"inLanguage\":\"en-US\",\"@id\":\"https:\/\/blog.mozilla.org\/security\/#\/schema\/person\/image\/f076fe8c668c394e8741f7be518c066a\",\"url\":\"https:\/\/secure.gravatar.com\/avatar\/309fb1423f538a1443ceb8c01e33f651?s=96&d=identicon&r=g\",\"contentUrl\":\"https:\/\/secure.gravatar.com\/avatar\/309fb1423f538a1443ceb8c01e33f651?s=96&d=identicon&r=g\",\"caption\":\"Curtis Koenig\"},\"description\":\"Sr. Security Program Manager\",\"sameAs\":[\"https:\/\/spartiates.wordpress.com\/\"]}]}<\/script>\n<!-- \/ Yoast SEO plugin. -->","yoast_head_json":{"title":"Evolving the Security Review and Discussion Process - Mozilla Security Blog","robots":{"index":"index","follow":"follow","max-snippet":"max-snippet:-1","max-image-preview":"max-image-preview:large","max-video-preview":"max-video-preview:-1"},"canonical":"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/","twitter_misc":{"Written by":"Curtis Koenig","Est. reading time":"4 minutes"},"schema":{"@context":"https:\/\/schema.org","@graph":[{"@type":"WebPage","@id":"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/","url":"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/","name":"Evolving the Security Review and Discussion Process - Mozilla Security Blog","isPartOf":{"@id":"https:\/\/blog.mozilla.org\/security\/#website"},"datePublished":"2011-07-29T16:07:06+00:00","dateModified":"2011-07-29T16:08:03+00:00","author":{"@id":"https:\/\/blog.mozilla.org\/security\/#\/schema\/person\/792ed382fa382861f5deff578b21429c"},"breadcrumb":{"@id":"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/#breadcrumb"},"inLanguage":"en-US","potentialAction":[{"@type":"ReadAction","target":["https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/"]}]},{"@type":"BreadcrumbList","@id":"https:\/\/blog.mozilla.org\/security\/2011\/07\/29\/494\/#breadcrumb","itemListElement":[{"@type":"ListItem","position":1,"name":"Home","item":"https:\/\/blog.mozilla.org\/security\/"},{"@type":"ListItem","position":2,"name":"Evolving the Security Review and Discussion Process"}]},{"@type":"WebSite","@id":"https:\/\/blog.mozilla.org\/security\/#website","url":"https:\/\/blog.mozilla.org\/security\/","name":"Mozilla Security Blog","description":"","potentialAction":[{"@type":"SearchAction","target":{"@type":"EntryPoint","urlTemplate":"https:\/\/blog.mozilla.org\/security\/?s={search_term_string}"},"query-input":"required name=search_term_string"}],"inLanguage":"en-US"},{"@type":"Person","@id":"https:\/\/blog.mozilla.org\/security\/#\/schema\/person\/792ed382fa382861f5deff578b21429c","name":"Curtis Koenig","image":{"@type":"ImageObject","inLanguage":"en-US","@id":"https:\/\/blog.mozilla.org\/security\/#\/schema\/person\/image\/f076fe8c668c394e8741f7be518c066a","url":"https:\/\/secure.gravatar.com\/avatar\/309fb1423f538a1443ceb8c01e33f651?s=96&d=identicon&r=g","contentUrl":"https:\/\/secure.gravatar.com\/avatar\/309fb1423f538a1443ceb8c01e33f651?s=96&d=identicon&r=g","caption":"Curtis Koenig"},"description":"Sr. Security Program Manager","sameAs":["https:\/\/spartiates.wordpress.com\/"]}]}},"_links":{"self":[{"href":"https:\/\/blog.mozilla.org\/security\/wp-json\/wp\/v2\/posts\/494"}],"collection":[{"href":"https:\/\/blog.mozilla.org\/security\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/blog.mozilla.org\/security\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/blog.mozilla.org\/security\/wp-json\/wp\/v2\/users\/315"}],"replies":[{"embeddable":true,"href":"https:\/\/blog.mozilla.org\/security\/wp-json\/wp\/v2\/comments?post=494"}],"version-history":[{"count":0,"href":"https:\/\/blog.mozilla.org\/security\/wp-json\/wp\/v2\/posts\/494\/revisions"}],"wp:attachment":[{"href":"https:\/\/blog.mozilla.org\/security\/wp-json\/wp\/v2\/media?parent=494"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/blog.mozilla.org\/security\/wp-json\/wp\/v2\/categories?post=494"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/blog.mozilla.org\/security\/wp-json\/wp\/v2\/tags?post=494"},{"taxonomy":"author","embeddable":true,"href":"https:\/\/blog.mozilla.org\/security\/wp-json\/wp\/v2\/coauthors?post=494"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}