Passbolt recently contacted us about doing a code review so we thought now would be a great time to share more about our code review process with you.
While in-house and peer reviews are important to maximise code quality, it is still incredibly important to get an independent third party to review your code - that is where CakeDC can step in.
Passbolt is free, open-source, self hosted password manager for teams which makes collaboration and sharing company account credentials within a team much easier. It's based on open security standards and uses OpenPGP to authenticate users and verify secrets server side. Passbolt consists of server side web app built in CakePHP providing web interface and API, and Chrome extension for client side.
The overall aspects that are reviewed in our code review include a review of quality, implementation, security, performance, documentation and test coverage.
When looking into quality, the team reviews aspects concerning the code following CakePHP conventions, coding standards and coding quality. Overall, passbolt’s code review revealed that CakePHP conventions and coding standards are largely followed, no concerns were detected.
Implementation outlines key issues with framework use and approach. It includes reviewing the code for framework usage, separation of concerns as well as code reuse and modularity. Key recommendations are outlined at this point and guidance is given into how to solve any issues. For the Passbolt review, bigger or concerning issues were uncovered, but improvements were recommended and outlined within the closing documentation.
The security portion of the code review deals with how secure the code is in terms of CakePHP usage. No security flaws were found in the passbolt code review.
Our in depth code review focuses on performance, specifically investigating any bottlenecks in the code base and database as well as indexes optimization.
For the full passbolt code review results, check out the Code review results. Passbolt has also posted about their review, check out their post here.
If you or your company has a CakePHP application and you aren’t sure if its running at the optimum, then get in touch - Code reviews can offer insights and learning into how to improve your application.