-
Notifications
You must be signed in to change notification settings - Fork 204
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
Check Sighash Flag #2807
Check Sighash Flag #2807
Conversation
ouziel-slama
commented
Dec 7, 2024
•
edited
Loading
edited
- Double-check the spelling and grammar of all strings, code comments, etc.
- Double-check that all code is deterministic that needs to be
- Add tests to cover any new or revised logic
- Ensure that the test suite passes
- Update the project release notes
- Update the project documentation, as appropriate, with a corresponding Pull Request in the Documentation repository
Fix ParentProcessChecker Thread
logger.info("API Server stopped.") | ||
|
||
|
||
def is_process_alive(pid): | ||
"""Check For the existence of a unix pid.""" | ||
try: |
Check warning
Code scanning / pylint
Unnecessary "else" after "return", remove the "else" and de-indent the code inside it. Warning
@@ -155,11 +157,152 @@ | |||
return vout["value"], vout["script_pub_key"], is_segwit | |||
|
|||
|
|||
def is_valid_der(der): |
Check warning
Code scanning / pylint
Too many return statements (8/6). Warning
if len(der) != 6 + rlength + slength: | ||
return False | ||
return True | ||
except Exception: |
Check warning
Code scanning / pylint
Catching too general exception Exception. Warning
try: | ||
r = int.from_bytes(schnorr[0:32], byteorder="big") | ||
s = int.from_bytes(schnorr[32:64], byteorder="big") | ||
except Exception: |
Check warning
Code scanning / pylint
Catching too general exception Exception. Warning
return None | ||
|
||
|
||
def get_schnorr_signature_sighash_flag(value): |
Check warning
Code scanning / pylint
Either all return statements in a function should return an expression, or none of them should. Warning