-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove all history in ccf-app-samples repo #43
Comments
Some PRs were squashed due to microsoft#43 Copy banking-app from https://github.com/takuro-sato/ccf-app-template/tree/example_apps (microsoft#7) The repo was originally forked from https://github.com/microsoft/ccf-app-template Co-authored-by: Takuro Sato <takurosato@microsoft.com> Remove unused codes (microsoft#9) * Remove unused codes * Fix failed CI Co-authored-by: Takuro Sato <takurosato@microsoft.com> Update README.md (microsoft#10) * Update README.md * Fix README.md Co-authored-by: Takuro Sato <takurosato@microsoft.com> Update why ccf + Implement application claim for transfer (microsoft#12) * Add 'why ccf' section * Add application claim for transfer API * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Add 'Get claim' endpoint * Update /receipt so that the response includes the expanded claim * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Remove /claim Co-authored-by: Takuro Sato <takurosato@microsoft.com> Co-authored-by: Amaury Chamayou <amaury@xargs.fr> Clean up (microsoft#13) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Add features to devcontainer (microsoft#16) * Add features to devcontainer * npm install at container creation Run tests in CI with virtual mode (microsoft#19) * Run tests in CI * Tidy up Co-authored-by: Takuro Sato <takurosato@microsoft.com> Added guidelines for running tests (microsoft#21) Add linting (microsoft#20) * Add linting * Fix 'detected dubious ownership in repository' error * See if CI fails as expected * Revert "See if CI fails as expected" This reverts commit 4570982. Co-authored-by: Takuro Sato <takurosato@microsoft.com> Add back 'Open in GitHub Codespaces' button (microsoft#22) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Use docker image that can use virtual (microsoft#25) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Create the user certs as part of the demo (microsoft#34) Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> GitHub Collaboration (microsoft#36) * Added guide about board * Typos Added make file to hide scripts (microsoft#31) * Add simple make commands * Add initial devcontainer build * Adding workflow-dispatch to test * workflow_dispatch * Add Safe Directory * Pass GITHUB_WORKSPACE environment variable * Added image that can use virtual * Makefile per sample * Amended CI * Removed the banking-app clean from the root Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> Created set_js_app.json for governance (microsoft#33) Split demo.sh into demo_governance.sh and demo_application.sh (microsoft#40) * Split demo.sh into demo_governance.sh and demo_application.sh * add more comments * remove the extra echo line * resolve review comments
@ross-p-smith |
Unfortunately, I think we have to be a bit more brutal: - https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/removing-sensitive-data-from-a-repository |
We could try
|
Both of the link and the way suggested sound good to try. I tried:
But I could simply import it. Do I need some setting? |
Maybe it's an AzureDevOps linked to the MSFT tenant as credscan is setup on it. I've just tried it again and it fails. Was the Azure DevOps that you created a personal one? |
I see. |
All PRs were squashed due to #43 Copy banking-app from https://github.com/takuro-sato/ccf-app-template/tree/example_apps (#7) The repo was originally forked from https://github.com/microsoft/ccf-app-template Co-authored-by: Takuro Sato <takurosato@microsoft.com> Remove unused codes (#9) * Remove unused codes * Fix failed CI Co-authored-by: Takuro Sato <takurosato@microsoft.com> Update README.md (#10) * Update README.md * Fix README.md Co-authored-by: Takuro Sato <takurosato@microsoft.com> Update why ccf + Implement application claim for transfer (#12) * Add 'why ccf' section * Add application claim for transfer API * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Add 'Get claim' endpoint * Update /receipt so that the response includes the expanded claim * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Remove /claim Co-authored-by: Takuro Sato <takurosato@microsoft.com> Co-authored-by: Amaury Chamayou <amaury@xargs.fr> Clean up (#13) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Add features to devcontainer (#16) * Add features to devcontainer * npm install at container creation Run tests in CI with virtual mode (#19) * Run tests in CI * Tidy up Co-authored-by: Takuro Sato <takurosato@microsoft.com> Added guidelines for running tests (#21) Add linting (#20) * Add linting * Fix 'detected dubious ownership in repository' error * See if CI fails as expected * Revert "See if CI fails as expected" This reverts commit 4570982. Co-authored-by: Takuro Sato <takurosato@microsoft.com> Add back 'Open in GitHub Codespaces' button (#22) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Use docker image that can use virtual (#25) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Create the user certs as part of the demo (#34) Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> GitHub Collaboration (#36) * Added guide about board * Typos Added make file to hide scripts (#31) * Add simple make commands * Add initial devcontainer build * Adding workflow-dispatch to test * workflow_dispatch * Add Safe Directory * Pass GITHUB_WORKSPACE environment variable * Added image that can use virtual * Makefile per sample * Amended CI * Removed the banking-app clean from the root Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> Created set_js_app.json for governance (#33) Split demo.sh into demo_governance.sh and demo_application.sh (#40) * Split demo.sh into demo_governance.sh and demo_application.sh * add more comments * remove the extra echo line * resolve review comments
I tried bfc and ten force pushed. I got errors for updating PR refs ( takurosato@LAPTOP-5I4F12NJ:~/Projects/CCF/repo/ccf-app-samples.git$ java -jar ~/bfg.jar --delete-files '{set_user0.json,set_user1.json,user0_cert.pem,user0_privk.pem,user1_cert.pem,user1_privk.pem}'
Using repo : /home/takurosato/Projects/CCF/repo/ccf-app-samples.git
Found 39 objects to protect
Found 23 commit-pointing refs : HEAD, refs/heads/main, refs/pull/10/head, ...
Protected commits
-----------------
These are your protected commits, and so their contents will NOT be altered:
* commit 5817aca7 (protected by 'HEAD')
Cleaning
--------
Found 130 commits
Cleaning commits: 100% (130/130)
Cleaning commits completed in 264 ms.
Updating 21 Refs
----------------
Ref Before After
---------------------------------------
refs/pull/10/head | e9fad6c6 | bbedbb18
refs/pull/11/head | ac99e48f | 6ce8b7fd
refs/pull/12/head | 0c75d0c6 | 3869fcc4
refs/pull/13/head | ba37374c | 0b1d4f9b
refs/pull/16/head | a2b350b2 | c930fcd2
refs/pull/19/head | 20ecc552 | b7f623e1
refs/pull/20/head | 60eae65c | ee37bfdb
refs/pull/21/head | 5a2ae178 | 519ab49b
refs/pull/22/head | 6f90abee | fe7aa39a
refs/pull/24/head | 5c8782b0 | 1a381c6f
refs/pull/25/head | 47b30668 | f956f7f0
refs/pull/31/head | 0cd9e995 | d3b75dfc
refs/pull/33/head | 6f2ca657 | ef9d1cd7
refs/pull/34/head | 0cd28ed2 | 25715c75
refs/pull/36/head | b9481981 | d95f7cac
...
Updating references: 100% (21/21)
...Ref update completed in 43 ms.
Commit Tree-Dirt History
------------------------
Earliest Latest
| |
..DDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDmDmmmmmmmmmmmmmmmmmmDmmmmmm
D = dirty commits (file tree fixed)
m = modified commits (commit message or parents changed)
. = clean commits (no changes to file tree)
Before After
-------------------------------------------
First modified commit | 5ef802e9 | 66f5f5b4
Last dirty commit | 4600c79a | 593dcfa1
Deleted files
-------------
Filename Git id
------------------------------------------------------
set_user0.json | 681452b6 (764 B ), 19e32c4e (802 B )
set_user1.json | eb77e737 (802 B ), f9e086ec (764 B )
user0_cert.pem | 0dfd546c (648 B )
user0_privk.pem | 08307963 (359 B )
user1_cert.pem | 374255bd (648 B )
user1_privk.pem | 843163e6 (359 B )
In total, 183 object ids were changed. Full details are logged here:
/home/takurosato/Projects/CCF/repo/ccf-app-samples.git.bfg-report/2022-11-21/13-57-51
BFG run is complete! When ready, run: git reflog expire --expire=now --all && git gc --prune=now --aggressive
|
According to rtyley/bfg-repo-cleaner#36 (comment), |
All PRs were squashed due to #43 Copy banking-app from https://github.com/takuro-sato/ccf-app-template/tree/example_apps (#7) The repo was originally forked from https://github.com/microsoft/ccf-app-template Co-authored-by: Takuro Sato <takurosato@microsoft.com> Remove unused codes (#9) * Remove unused codes * Fix failed CI Co-authored-by: Takuro Sato <takurosato@microsoft.com> Update README.md (#10) * Update README.md * Fix README.md Co-authored-by: Takuro Sato <takurosato@microsoft.com> Update why ccf + Implement application claim for transfer (#12) * Add 'why ccf' section * Add application claim for transfer API * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Add 'Get claim' endpoint * Update /receipt so that the response includes the expanded claim * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Remove /claim Co-authored-by: Takuro Sato <takurosato@microsoft.com> Co-authored-by: Amaury Chamayou <amaury@xargs.fr> Clean up (#13) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Add features to devcontainer (#16) * Add features to devcontainer * npm install at container creation Run tests in CI with virtual mode (#19) * Run tests in CI * Tidy up Co-authored-by: Takuro Sato <takurosato@microsoft.com> Added guidelines for running tests (#21) Add linting (#20) * Add linting * Fix 'detected dubious ownership in repository' error * See if CI fails as expected * Revert "See if CI fails as expected" This reverts commit 4570982. Co-authored-by: Takuro Sato <takurosato@microsoft.com> Add back 'Open in GitHub Codespaces' button (#22) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Use docker image that can use virtual (#25) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Create the user certs as part of the demo (#34) Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> GitHub Collaboration (#36) * Added guide about board * Typos Added make file to hide scripts (#31) * Add simple make commands * Add initial devcontainer build * Adding workflow-dispatch to test * workflow_dispatch * Add Safe Directory * Pass GITHUB_WORKSPACE environment variable * Added image that can use virtual * Makefile per sample * Amended CI * Removed the banking-app clean from the root Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> Created set_js_app.json for governance (#33) Split demo.sh into demo_governance.sh and demo_application.sh (#40) * Split demo.sh into demo_governance.sh and demo_application.sh * add more comments * remove the extra echo line * resolve review comments
All PRs were squashed due to #43 Copy banking-app from https://github.com/takuro-sato/ccf-app-template/tree/example_apps (#7) The repo was originally forked from https://github.com/microsoft/ccf-app-template Co-authored-by: Takuro Sato <takurosato@microsoft.com> Remove unused codes (#9) * Remove unused codes * Fix failed CI Co-authored-by: Takuro Sato <takurosato@microsoft.com> Update README.md (#10) * Update README.md * Fix README.md Co-authored-by: Takuro Sato <takurosato@microsoft.com> Update why ccf + Implement application claim for transfer (#12) * Add 'why ccf' section * Add application claim for transfer API * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Add 'Get claim' endpoint * Update /receipt so that the response includes the expanded claim * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Remove /claim Co-authored-by: Takuro Sato <takurosato@microsoft.com> Co-authored-by: Amaury Chamayou <amaury@xargs.fr> Clean up (#13) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Add features to devcontainer (#16) * Add features to devcontainer * npm install at container creation Run tests in CI with virtual mode (#19) * Run tests in CI * Tidy up Co-authored-by: Takuro Sato <takurosato@microsoft.com> Added guidelines for running tests (#21) Add linting (#20) * Add linting * Fix 'detected dubious ownership in repository' error * See if CI fails as expected * Revert "See if CI fails as expected" This reverts commit 4570982. Co-authored-by: Takuro Sato <takurosato@microsoft.com> Add back 'Open in GitHub Codespaces' button (#22) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Use docker image that can use virtual (#25) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Create the user certs as part of the demo (#34) Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> GitHub Collaboration (#36) * Added guide about board * Typos Added make file to hide scripts (#31) * Add simple make commands * Add initial devcontainer build * Adding workflow-dispatch to test * workflow_dispatch * Add Safe Directory * Pass GITHUB_WORKSPACE environment variable * Added image that can use virtual * Makefile per sample * Amended CI * Removed the banking-app clean from the root Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> Created set_js_app.json for governance (#33) Split demo.sh into demo_governance.sh and demo_application.sh (#40) * Split demo.sh into demo_governance.sh and demo_application.sh * add more comments * remove the extra echo line * resolve review comments
We confirmed that the current main (18bf65d) has no problem with CredScan . The current main was made using bfg tool. |
* Copy banking-app from https://github.com/takuro-sato/ccf-app-template/tree/example_apps (#7) The repo was originally forked from https://github.com/microsoft/ccf-app-template Co-authored-by: Takuro Sato <takurosato@microsoft.com> * Remove unused codes (#9) * Remove unused codes * Fix failed CI Co-authored-by: Takuro Sato <takurosato@microsoft.com> * Update README.md (#10) * Update README.md * Fix README.md Co-authored-by: Takuro Sato <takurosato@microsoft.com> * Update why ccf + Implement application claim for transfer (#12) * Add 'why ccf' section * Add application claim for transfer API * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Add 'Get claim' endpoint * Update /receipt so that the response includes the expanded claim * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Remove /claim Co-authored-by: Takuro Sato <takurosato@microsoft.com> Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Clean up (#13) Co-authored-by: Takuro Sato <takurosato@microsoft.com> * Add features to devcontainer (#16) * Add features to devcontainer * npm install at container creation * Run tests in CI with virtual mode (#19) * Run tests in CI * Tidy up Co-authored-by: Takuro Sato <takurosato@microsoft.com> * Added guidelines for running tests (#21) * Add linting (#20) * Add linting * Fix 'detected dubious ownership in repository' error * See if CI fails as expected * Revert "See if CI fails as expected" This reverts commit 4570982. Co-authored-by: Takuro Sato <takurosato@microsoft.com> * Add back 'Open in GitHub Codespaces' button (#22) Co-authored-by: Takuro Sato <takurosato@microsoft.com> * Use docker image that can use virtual (#25) Co-authored-by: Takuro Sato <takurosato@microsoft.com> * Create the user certs as part of the demo (#34) Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> * GitHub Collaboration (#36) * Added guide about board * Typos * Added make file to hide scripts (#31) * Add simple make commands * Add initial devcontainer build * Adding workflow-dispatch to test * workflow_dispatch * Add Safe Directory * Pass GITHUB_WORKSPACE environment variable * Added image that can use virtual * Makefile per sample * Amended CI * Removed the banking-app clean from the root Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> * initial steps for moving certs from docker image * adjusting build_image script and docker files * Created set_js_app.json for governance (#33) * updating script, readme and makefile * updating ci.yml file to reflect new way of building containers * prettifying README.md * fixing typo on README.md * Split demo.sh into demo_governance.sh and demo_application.sh (#40) * Split demo.sh into demo_governance.sh and demo_application.sh * add more comments * remove the extra echo line * resolve review comments * changing folder structure for certificates to be the same as in sandbox flow * Add banking-app Some PRs were squashed due to #43 Copy banking-app from https://github.com/takuro-sato/ccf-app-template/tree/example_apps (#7) The repo was originally forked from https://github.com/microsoft/ccf-app-template Co-authored-by: Takuro Sato <takurosato@microsoft.com> Remove unused codes (#9) * Remove unused codes * Fix failed CI Co-authored-by: Takuro Sato <takurosato@microsoft.com> Update README.md (#10) * Update README.md * Fix README.md Co-authored-by: Takuro Sato <takurosato@microsoft.com> Update why ccf + Implement application claim for transfer (#12) * Add 'why ccf' section * Add application claim for transfer API * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/README.md Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/test.sh Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Add 'Get claim' endpoint * Update /receipt so that the response includes the expanded claim * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Update banking-app/src/endpoints/banking.ts Co-authored-by: Amaury Chamayou <amaury@xargs.fr> * Remove /claim Co-authored-by: Takuro Sato <takurosato@microsoft.com> Co-authored-by: Amaury Chamayou <amaury@xargs.fr> Clean up (#13) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Add features to devcontainer (#16) * Add features to devcontainer * npm install at container creation Run tests in CI with virtual mode (#19) * Run tests in CI * Tidy up Co-authored-by: Takuro Sato <takurosato@microsoft.com> Added guidelines for running tests (#21) Add linting (#20) * Add linting * Fix 'detected dubious ownership in repository' error * See if CI fails as expected * Revert "See if CI fails as expected" This reverts commit 4570982. Co-authored-by: Takuro Sato <takurosato@microsoft.com> Add back 'Open in GitHub Codespaces' button (#22) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Use docker image that can use virtual (#25) Co-authored-by: Takuro Sato <takurosato@microsoft.com> Create the user certs as part of the demo (#34) Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> GitHub Collaboration (#36) * Added guide about board * Typos Added make file to hide scripts (#31) * Add simple make commands * Add initial devcontainer build * Adding workflow-dispatch to test * workflow_dispatch * Add Safe Directory * Pass GITHUB_WORKSPACE environment variable * Added image that can use virtual * Makefile per sample * Amended CI * Removed the banking-app clean from the root Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> Created set_js_app.json for governance (#33) Split demo.sh into demo_governance.sh and demo_application.sh (#40) * Split demo.sh into demo_governance.sh and demo_application.sh * add more comments * remove the extra echo line * resolve review comments Co-authored-by: Takuro Sato <79583855+takuro-sato@users.noreply.github.com> Co-authored-by: Takuro Sato <takurosato@microsoft.com> Co-authored-by: Amaury Chamayou <amaury@xargs.fr> Co-authored-by: Ross Smith <ross-p-smith@users.noreply.github.com> Co-authored-by: Ayman <Aymalla@outlook.com>
The https://github.com/microsoft/ccf-app-samples/ repository has previously had certificates stored inside it for the demo. These were replaced by dynamically creating the certificates as part of the demo.
However this means the certs are in the git history and this means it is problematic to fork this repository to Azure DevOps.
I think we should squash the commits before and after the certs being added now before we iterate further.
The text was updated successfully, but these errors were encountered: