Ci/apply zizmor security audit
Description
This PR fixes #638. It fixes persist credentials and template injection (environment variable was used).
Notes to the reviewers
Changelog notice
Checklists
All Submissions:
- [x] I've signed all my commits
- [x] I followed the contribution guidelines
- [x] I'm linking the issue being fixed by this PR
This PR has been signed and rebased. Similar fix has been done in bdk. Thanks.
Just a quick update: I'll wait for https://github.com/bitcoindevkit/bdk_wallet/pull/8 to get merged before looking at this.
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.
@aagbotemi great job! if you can just rebase this once more I will merge it
Thank you @reez, I have rebased