To checks from forthcoming Security Review 1.1

CommentFileSizeAuthor
#4 2049669.D6.do-not-test.patch14.07 KBgcassie
#2 2049669-acquia_connector-security-review-1-1.patch16.08 KBcoltrane
#1 2049669-acquia_connector-security-review-1-1.patch14.9 KBcoltrane
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

coltrane’s picture

coltrane
he/him
CreditAttribution: coltrane commented
Status: Active » Needs review
FileSize
2049669-acquia_connector-security-review-1-1.patch14.9 KB
coltrane’s picture

coltrane
he/him
CreditAttribution: coltrane commented
FileSize
2049669-acquia_connector-security-review-1-1.patch16.08 KB

Updated patch

gcassie’s picture

gcassie CreditAttribution: gcassie commented
Status: Needs review » Reviewed & tested by the community

Looks good.

gcassie’s picture

gcassie CreditAttribution: gcassie commented
FileSize
2049669.D6.do-not-test.patch14.07 KB

Here's the D6 version.

coltrane’s picture

coltrane
he/him
CreditAttribution: coltrane commented
Status: Reviewed & tested by the community » Fixed

Thanks for the review and backport @gcassie!! Committed http://drupalcode.org/project/acquia_connector.git/commit/be1218e and http://drupalcode.org/project/acquia_connector.git/commit/5b8377b

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.