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
Create codeql-analysis.yml #3975
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3975 +/- ##
==========================================
+ Coverage 37.06% 37.10% +0.04%
==========================================
Files 325 325
Lines 9109 9115 +6
Branches 1468 1470 +2
==========================================
+ Hits 3376 3382 +6
Misses 5542 5542
Partials 191 191 |
Performance Report✔️ no performance regression detected Full benchmark results
|
return passphrase; | ||
} catch (e) { | ||
console.log(`Imported passphrase ${passphraseFile}, but it's invalid: ${(e as Error).message}`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Logging the error is very important, does codeql complain about that too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll check if the error message does not contain any information regarding the file path. Wanted to be on the safe side by removing it, but I'll confirm and if its safe and include.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated to Include the error message
…r contain info relateing to location of passphrase file
Motivation
Recommended by Github for code analysis