Fix the enclave for deployment#387
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request refactors the enclave registration logic to interact with the EspressoTEEVerifier contract instead of the EspressoNitroTEEVerifier directly. Key changes include renaming getNitroVerifier to getVerifier, updating the return types to use the base verifier bindings, and introducing constants for teeTypeNitro and serviceTypeBatchPoster to support the updated contract method signatures for registering and checking enclave hashes. I have no feedback to provide.
jjeangal
left a comment
jjeangal
left a comment
There was a problem hiding this comment.
Looks good to me! Tested this PR with the production code and it works out just fine. I would simply recommend removing ports 9000 and 9001 and using 8337 and 8338 as it would make things more straightforward.
* Fix registration * Add readiness * Fix enclave retries * Simplify ports
2 participants
Closes https://app.asana.com/1/1208976916964769/project/1209976130071762/task/1213873022354661?focus=true.
This PR:
How to test this PR: