Skip to content

fix(crypto): resolve the invalid witness#6368

Merged
kuny0707 merged 15 commits intotronprotocol:release_v4.8.1from
Federico2014:fix/local_witness
Aug 19, 2025
Merged

fix(crypto): resolve the invalid witness#6368
kuny0707 merged 15 commits intotronprotocol:release_v4.8.1from
Federico2014:fix/local_witness

Conversation

@Federico2014
Copy link
Contributor

What does this PR do?
Closes #6281

@Federico2014 Federico2014 changed the title fix(crypto): optimize the null witness fix(crypto): resolve the invalid witness Jun 30, 2025
@kuny0707 kuny0707 linked an issue Jun 30, 2025 that may be closed by this pull request
@kuny0707 kuny0707 moved this to In Review in java-tron Jun 30, 2025
…_witness

# Conflicts:
#	framework/src/test/java/org/tron/core/zksnark/ShieldedReceiveTest.java
# Conflicts:
#	framework/src/main/java/org/tron/core/config/args/Args.java
#	framework/src/test/java/org/tron/core/config/args/ArgsTest.java
#	plugins/build.gradle
@Federico2014 Federico2014 force-pushed the fix/local_witness branch 2 times, most recently from 7bc1f4b to e0f7edd Compare August 15, 2025 11:16
@Federico2014 Federico2014 force-pushed the fix/local_witness branch 2 times, most recently from fcdfeb5 to 493786b Compare August 19, 2025 07:14
@kuny0707 kuny0707 merged commit f7a6748 into tronprotocol:release_v4.8.1 Aug 19, 2025
7 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in java-tron Aug 19, 2025
warku123 pushed a commit to warku123/java-tron that referenced this pull request Feb 6, 2026
* feat(crypto): upgrade the bouncycastle denpendency

* fix(crypto): optimize the null localwitness

* fix(metric): process the null witness address

* feat(crypto): optimize the code

* feat(doc): add the comment

* feat(test): add the nullWitnessAddress test

* add witness config test

* strengthen the private key validation

* increase the test coverage

* optimiz the witnessAddress initialization

* simplify the localWitness

* refactor the code

* add the test unit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Invalid witness address when the localwitness is null