Skip to content

Commit

Permalink
Merge pull request #670 from brockfanning/foia-50-filter-out-captcha-…
Browse files Browse the repository at this point in the history
…data

FOIA-50: Filter captcha out of email and API submissions.
  • Loading branch information
brockfanning authored Oct 31, 2024
2 parents b7b4a9f + 890922b commit 7511e92
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,9 @@ protected function setSubmissionContents() {
'confirmation_id' => $this->webformSubmission->id(),
];
$submissionContents = array_merge($submissionMetadata, $submissionContents);
if (!empty($submissionContents['captcha'])) {
unset($submissionContents['captcha']);
}
$this->submissionContents = $submissionContents;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,10 @@ protected function assembleRequestData(FoiaRequestInterface $foiaRequest) {
$foiaRequestId = ['request_id' => $foiaRequest->id()];
$submissionValues = array_merge($apiVersion, $agencyInfo, $foiaRequestId, $formValues);

if (!empty($submissionValues['captcha'])) {
unset($submissionValues['captcha']);
}

return $submissionValues;
}

Expand Down

0 comments on commit 7511e92

Please sign in to comment.