Skip to content
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

Merged
merged 25 commits into from
Dec 9, 2024
Merged

Check Sighash Flag #2807

merged 25 commits into from
Dec 9, 2024

Conversation

ouziel-slama
Copy link
Contributor

@ouziel-slama ouziel-slama commented Dec 7, 2024

  • 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

if util.is_utxo_format(message_data["source"]):
return "detach"
return "attach"
except Exception:

Check warning

Code scanning / pylint

Catching too general exception Exception. Warning

Catching too general exception Exception.
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

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it.
@@ -2163,7 +2163,7 @@
return holders


def holders(db, asset, exclude_empty_holders=False):
def holders(db, asset, exclude_empty_holders=False, block_index=None):

Check warning

Code scanning / pylint

Unused argument 'block_index'. Warning

Unused argument 'block_index'.
@@ -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

Too many return statements (8/6).
if len(der) != 6 + rlength + slength:
return False
return True
except Exception:

Check warning

Code scanning / pylint

Catching too general exception Exception. Warning

Catching too general exception Exception.
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

Catching too general exception Exception.
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

Either all return statements in a function should return an expression, or none of them should.
@ouziel-slama ouziel-slama merged commit ece96b5 into develop Dec 9, 2024
18 checks passed
@ouziel-slama ouziel-slama deleted the sighash branch December 9, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants