rtyley / bfg-repo-cleaner

Removes large or troublesome blobs like git-filter-branch does, but faster. And written in Scala
https://rtyley.github.io/bfg-repo-cleaner/
GNU General Public License v3.0
10.84k stars 535 forks source link

--replace-text missed a file #437

Closed garin1000 closed 3 years ago

garin1000 commented 3 years ago

Hi all,

after following the instructions in https://rtyley.github.io/bfg-repo-cleaner/ with a custom pw.txt as input for -rt, the passwords were still present in a single commit. Running bfg again on a fresh bare git repo still did not find the commit, and thus did not change anything. The pw.txt file is correct, the passwords did not change and they were found on all other commits (even to the same file). Any help will be appreciated ;-)

In the tree below, the passwords were committed in 27bddd5 in a single file in a specific subdirectory of the working tree. bfg was called on commit f8478bc7 but did not find the password committed in 27bddd5.

* commit cac362a (newest)
|
* commit 4f5f33f
|
* commit 5c682ce
|
* commit 75d8189
|
* commit f847bc7
|     Removed passwords
|
*   commit 78408fb
|\  Merge: 770afb0 1633fd1
| |     Merge branch 'master' of [remote/origin] into master
| |
| * commit 1633fd1
| |
| ... several commits on the right side here
| |
| * commit caace12
| |
| * commit 27bddd5
| |     Accidentally commited passwords
| |
| * commit 815f2e0
| |
* | commit 770afb0
|/      Removed passwords from HEAD
| 
* commit 80cf384
|
* commit 3994b3c (oldest)
|
garin1000 commented 3 years ago

Solved, file was larger than the default file content threshold of 1M. Would have been helpful if this threshold would have been mentioned on the BFG page ;-)