DatingDapp
First Flight #33
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Previous Next
Submission Details
Severity: low
Invalid

Missing Mandatory Gender Field in Profile NFT in SoulboundProfileNFT contract.

0xblackadam

Finding Description and Impact

The SoulboundProfileNFT contract lacks a mandatory gender field in the mintProfile function when creating a profile. For a dating app, this is a critical oversight because:

  • Prevents effective matchmaking based on user preferences

  • Makes filtering and recommendations impossible

  • Reduces the utility of the profile NFT for dating purposes

Impact:

  • Cannot implement preference-based matching

  • Reduces platform usability for dating purposes

  • Makes it harder to build meaningful connections

  • Inefficient user discovery process

Lines of Code

https://github.com/CodeHawks-Contests/2025-02-datingdapp/blob/878bd34ef6607afe01f280cd5aedf3184fc4ca7b/src/SoulboundProfileNFT.sol#L30

Recommended Mitigation Steps

  • Gender field role should be make compulsory

These changes provide several benefits:

  • Enables effective matchmaking algorithms

  • Respects user preferences and orientations

  • Improves user experience with better matches

Tools Used

Manual Review

Updates

Appeal created

n0kto Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Design choice
Assigned finding tags:

Informational or Gas

Please read the CodeHawks documentation to know which submissions are valid. If you disagree, provide a coded PoC and explain the real likelyhood and the detailed impact on the mainnet without any supposition (if, it could, etc) to prove your point.

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.