cisagov / gophish-tools

Helpful tools for interacting with a GoPhish phishing instance
Creative Commons Zero v1.0 Universal
42 stars 6 forks source link

Ensure credentials are not being stored or captured #54

Closed JCantu248 closed 3 years ago

JCantu248 commented 3 years ago

๐Ÿ—ฃ Description

Minor revision to configure all campaigns to not allow either credentials or passwords to be captured by Gophish.

๐Ÿ’ญ Motivation and context

Discussing with Kelly Theile and the PCA team, we discussed if passwords are being stored in plaintext (#52) and we were informed that we are not allowed to receive any credentials in any form (plaintext or encrypted) for legal reasons. Credentials must be dropped by the landing pages. This change ensures that no credential or password data is captured.

In addition, setting the capture credential and password boolean values was done in one place, before being repeated on each branch of execution.

Closes #52

๐Ÿงช Testing

This was tested by bringing the code changes into our LiPCA process and checking that the campaigns created afterwards were no longer configured to capture submitted data. After setting capture credentials to false, the checkbox in the Gophish GUI for capture submitted data is no longer set.

โœ… Checklist

dav3r commented 3 years ago

You should also merge in the latest changes from develop to ensure there are no conflicts.