Reviews and Mentoring for RavindraSingh

Events happening in the community are now at Drupal community events on www.drupal.org.
You are viewing a wiki page. You are welcome to join the group and then edit it. Be bold!

RavindraSingh reviews for various new project applications. Having hands on experience in reviewing/auditing contributed modules also. This is a part of mentoring to become a code review administrator eventually.

My main focus area is to check security issues/thridparty dependency in application. I take help from http://pareview.sh/ and some contrib modules like:
https://www.drupal.org/project/xhprof
https://www.drupal.org/project/coder
https://www.drupal.org/project/security_check
https://www.drupal.org/project/security_review
https://www.drupal.org/project/dcq

I represented all these things to Drupalers in a presentation (http://www.slideshare.net/ravindrasingh251/checklist-for-drupal-site-dev...)
AND after all these thing I do manual test.

Below are the projects application I reviewed/following.

[D7] Notify log - https://www.drupal.org/node/2457903#comment-9840995

[d7] Trrybe - https://www.drupal.org/node/2467397#comment-9804919

[D7] Secure Services - https://www.drupal.org/node/2202055#comment-8510603

[D7] Age Verification Module - https://www.drupal.org/node/2013591#comment-7509853
https://www.drupal.org/node/2013591#comment-7518731
(I have continually followed this project)

[D8] FoLogin - https://www.drupal.org/node/2167397#comment-8345155

[D7] Project QA - https://www.drupal.org/node/2166707#comment-8345177

[D7] Webcam field widget - https://www.drupal.org/node/1982010#comment-7519749

Security Vulnerability Findings

[D7] Environment Libraries - https://www.drupal.org/node/2504207#comment-10275519

[D7] Chitika Ads - https://www.drupal.org/node/2014627#comment-7512831

[D7] Environment Libraries - https://www.drupal.org/node/2504207#comment-10275615

Comments

Hi and thanks for starting

klausi's picture

Hi and thanks for starting this!

You get bonus extra points for finding security issues, which is THE most important part of project application reviews. See https://www.drupal.org/project/issues/search/projectapplications?project... for examples what we usually find. Please create a section about your security vulnerability findings once you identify those in an application.

Hi Klausi,I have found some

RavindraSingh's picture

Hi Klausi,

I have found some security issues an application - https://www.drupal.org/node/2504207#comment-10275519 and https://www.drupal.org/node/2504207#comment-10275615. Please have a look and let me if it is really valuable.

Thanks,
Ravindra

Code review for security advisory coverage applications

Group organizers

Group notifications

This group offers an RSS feed. Or subscribe to these personalized, sitewide feeds: