-
Notifications
You must be signed in to change notification settings - Fork 458
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
API Refactor: Fix node handler Attest
RPC to handle the new banned status
#1689
API Refactor: Fix node handler Attest
RPC to handle the new banned status
#1689
Conversation
…status Signed-off-by: Agustín Martínez Fayó <amartinezfayo@gmail.com>
…status Signed-off-by: Agustín Martínez Fayó <amartinezfayo@gmail.com>
I think we also need code in |
Signed-off-by: Agustín Martínez Fayó <amartinezfayo@gmail.com>
Signed-off-by: Agustín Martínez Fayó <amartinezfayo@gmail.com>
Signed-off-by: Agustín Martínez Fayó <amartinezfayo@gmail.com>
Signed-off-by: Agustín Martínez Fayó <amartinezfayo@gmail.com>
Signed-off-by: Agustín Martínez Fayó <amartinezfayo@gmail.com>
Signed-off-by: Agustín Martínez Fayó <amartinezfayo@gmail.com>
I've realised that some of the changes were already done in master at this point. This is ready for a new review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
\o/
Pull Request check list
Affected functionality
Attest method in Node handler.
Description of change
The Attest method in the node handler is updated to reject attestation if the agent is determined to be banned.
Which issue this PR fixes
Fixes #1680.