Add FHE.allowPublic as alias for FHE.allowGlobal#48
Merged
roeezolantz merged 8 commits intofeat/tn-chain-sig-verificationfrom Feb 25, 2026
Merged
Add FHE.allowPublic as alias for FHE.allowGlobal#48roeezolantz merged 8 commits intofeat/tn-chain-sig-verificationfrom
FHE.allowPublic as alias for FHE.allowGlobal#48roeezolantz merged 8 commits intofeat/tn-chain-sig-verificationfrom
Conversation
liorbond
approved these changes
Feb 23, 2026
roeezolantz
force-pushed
the
feat/tn-chain-sig-verification
branch
from
b949fe9 to
4694f33
Compare
roeezolantz
force-pushed
the
feat/allow-public
branch
from
df40125 to
d300ff0
Compare
roeezolantz
added a commit
that referenced
this pull request
Feb 25, 2026
* implement contract changes to verify decryption sigs * remove decryption id from sig * improvements * PR comments fixes, tests, ci * fix ci * fix test * fix existing tests * use tryRecover for non-reverting signature verification * add onlyIfEnabled to publish functions * replace require string with LengthMismatch custom error * add safety comment and advance free memory pointer in assembly * use bytes32 for ctHash in Impl and FHE overloads * add verifyDecryptResultSafe typed overloads * add tests for tryRecover, onlyIfEnabled, and edge cases * update changelog * Add `FHE.allowPublic` as alias for `FHE.allowGlobal` (#48) * Add `allowPublic` alias for `allowGlobal` * Add isPubliclyAllowed and isGloballyAllowed to TaskManager * Add tests for isPubliclyAllowed and isGloballyAllowed * update changelog * add FHE.isPubliclyAllowed, remove from DeterministicTM * remove isGloballyAllowed, keep only isPubliclyAllowed * update changelog --------- Co-authored-by: Roee Zolantz <zolantz.roee@gmail.com> --------- Co-authored-by: Architect <83823837+architect-dev@users.noreply.github.com>
Contributor
Author
|
This PR was merged after the CTA compatibility PR was merged. The |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
FHE.allowPublic as a more self-explanatory api for allowing public usage and decryption of a ctHash. Internally uses FHE.allowGlobal to grant public access.
Also adds isPubliclyAllowed / isGloballyAllowed view functions to ITaskManager and TaskManager to query whether a ctHash has been globally allowed. Both delegate to acl.globalAllowed(). Used by the Dispatcher to verify access without a permit.