Skip to content

Conversation

@jatZama
Copy link
Member

@jatZama jatZama commented Oct 8, 2025

No description provided.

@cla-bot cla-bot bot added the cla-signed label Oct 8, 2025
@jatZama jatZama changed the title Host cleaning feat(host-contracts): cleanup ACL Oct 8, 2025
@jatZama jatZama marked this pull request as ready for review October 8, 2025 21:33
Copy link
Contributor

@isaacdecoded isaacdecoded left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why these changes are needed right now. Also, this will have lot of conflicts with the ongoing PR for delegated decryption 🤔

@jatZama
Copy link
Member Author

jatZama commented Oct 9, 2025

Not sure why these changes are needed right now. Also, this will have lot of conflicts with the ongoing PR for delegated decryption 🤔

Yes we need it if we decide to redeploy contracts. Conflicts are minimal in this case with my PR, ill update it later.

Copy link
Contributor

@isaacdecoded isaacdecoded left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see the point of this PR of taking the chance for removing potential to-be-deprecated state. Maybe we should notify coprocessor team about the change in the host-listener service.

Copy link
Contributor

@melanciani melanciani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks !

@jatZama jatZama merged commit c909f0f into main Oct 9, 2025
108 of 109 checks passed
@jatZama jatZama deleted the hostCleaning branch October 9, 2025 10:06
jatZama added a commit that referenced this pull request Oct 9, 2025
* feat(host-contracts): cleanup ACL

* chore(host-contracts): updated tests and selectors

* chore(coprocessor): removed deprecated events

chore(coprocessor): fix typo

* chore(host-contracts): update readme
jatZama added a commit that referenced this pull request Oct 9, 2025
* feat(gateway-contracts): remove useless url storing and cleanup (#1034)

* feat(gateway-contracts): remove useless url storing and cleanup

* chore(gateway-contracts): fixed typos

* test(gateway-contracts): removed some of the deprecated tests

* chore(gateway-contracts): fixed getters

* chore(gateway-contracts): conformance

chore(gateway-contracts): conformance check

* feat(host-contracts): cleanup ACL (#1036)

* feat(host-contracts): cleanup ACL

* chore(host-contracts): updated tests and selectors

* chore(coprocessor): removed deprecated events

chore(coprocessor): fix typo

* chore(host-contracts): update readme

* feat(kms-connector): re-estimate gas on out-of-gas failed tx and resend (#1037)

* feat(kms-connector): retry on out-of-gas

* chore(kms-connector): quickfix to update gas on kmsgeneration contract

* chore(kms-connector): cargo linting

* chore(kms-connector): remove useless code
dartdart26 pushed a commit that referenced this pull request Oct 9, 2025
* feat(host-contracts): cleanup ACL

* chore(host-contracts): updated tests and selectors

* chore(coprocessor): removed deprecated events

chore(coprocessor): fix typo

* chore(host-contracts): update readme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants