Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-20346

Remove encryptDB.php from codebase

    Details

    • Type: Bug
    • Status: Done/Fixed
    • Priority: Trivial
    • Resolution: Fixed/Completed
    • Affects Version/s: 4.7.16
    • Fix Version/s: 4.7.19
    • Component/s: None
    • Labels:
      None
    • Versioning Impact:
      Patch (backwards-compatible bug fixes)
    • Documentation Required?:
      None
    • Funding Source:
      Contributed Code
    • Verified?:
      No

      Description

      Not critical, but I wrote up some thoughts on anonymising DBs today and remembered Eileen offering to merge a PR that removed this file.

      I think it should go because:

      • It is old and outdated
      • We'd be better off rid of scripts in bin/*.php which permit direct access without going through core Civi router
      • MD5 is not encryption, so what changes it does make are reversible.
      • It targets a limited number of fields, so a DB will contain identifying data even if not deanonymised
      • My main concern here is that people might think encryptDB.php really encrypts your DB.

      See also:

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              xurizaemon Chris Burgess
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: