http://code.google.com/p/google-highly-open-participation-drupal/issues/...
The Snippets section of the Drupal.org handbook is an invaluable resource
that the community uses to share code. Proper security and style reviews
are needed to make sure Drupal remains secure and stable.
Review mini-modules for Drupal 5.x in the handbook to verify that they work
as a module and they comply with code and security standards.
Correct/expand those that need it. Mark acceptable and newly corrected
ones as reviewed on x date.
This task involves setting up a local web server and selecting 4
mini-modules (see list below) to review by recreating the module and
testing locally to verify it actually works. Non-working modules will have
the page unpublished. For working mini-modules, perform a code review to
ensure the code adheres to security and code style best practices. If they
do not, make the corrections needed. Mark reviewed mini-module handbook
pages with the date of the passing review.
The deliverable on this task is to mark 4 mini-modules either as reviewed
and compliant (after a review by member of the security team) or create an
issue to have the page unpublished by a site admin.
Resources:
* Mini modules: http://drupal.org/node/126743
* See "Coding resources" section in the Wiki
Estimated time:
5 days
Comments
Comment #1
webchickThis task has been claimed by kourge. Work is available for review at http://code.google.com/p/google-highly-open-participation-drupal/issues/...
Comment #2
webchickComment #3
add1sun commentedLooks good kourge! I've had security team give them a look over and so I'm marking this task complete.
Comment #4
kourge commentedOMG, I feel sorry now because I've failed to notice this issue up until now.
Comment #5
webchickOh hey. Please don't feel sorry. We're still kind of learning how best to manage this program so there are a few kinks we're working out. We'll try and communicate better in the future. :)
Comment #6
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.