Skip to content

[SC-REP-051] Implement profile blacklisting logic for malicious actors#593

Merged
soomtochukwu merged 2 commits into
DXmakers:mainfrom
Trovic1:feat/sc-rep-051-profile-blacklisting
May 27, 2026
Merged

[SC-REP-051] Implement profile blacklisting logic for malicious actors#593
soomtochukwu merged 2 commits into
DXmakers:mainfrom
Trovic1:feat/sc-rep-051-profile-blacklisting

Conversation

@Trovic1
Copy link
Copy Markdown
Contributor

@Trovic1 Trovic1 commented May 27, 2026

Closes #405


Summary

  • implement blacklist-aware reputation profile storage in the Soroban reputation contract
  • add custom profile metrics for completed jobs, review aggregates, and active badge levels
  • add fixed-point arithmetic for rating averages and decay-based score reductions
  • restrict score adjustment and blacklist flows to an authorized contract address
  • add contract tests for empty profile loads, badge upgrades, blacklist behavior, and unauthorized review rejection

What Changed

  • refactored reputation storage to use structured Profile, RoleMetrics, and ReviewAggregate models
  • added blacklist state and public getters exposing blacklist and badge metadata
  • updated review submission logic to safely compute average rating basis points
  • added admin configuration for an authorized updater contract
  • added blacklist and decay logic for malicious actor handling
  • expanded unit coverage for security-sensitive reputation flows

Acceptance Criteria Coverage

  • reputation profiles load and save correctly without panicking on empty accounts
  • badge upgrades trigger and level changes reflect immediately in public getters
  • vulnerability tests prove arbitrary direct reviews from unverified public keys are rejected

Testing

  • cargo test -p reputation

@Trovic1 Trovic1 requested a review from soomtochukwu as a code owner May 27, 2026 15:52
@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2026

@Trovic1 is attempting to deploy a commit to the mAzI's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 27, 2026

@Trovic1 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Trovic1
Copy link
Copy Markdown
Contributor Author

Trovic1 commented May 27, 2026

@soomtochukwu done

@soomtochukwu soomtochukwu merged commit d114b5c into DXmakers:main May 27, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SC-REP-051] Reputation System Robustness Auditing - Step 51

2 participants