Skip to content

feat: implement Stellar wallet address validation - #207

Merged
SudiptaPaul-31 merged 1 commit into
SudiptaPaul-31:mainfrom
AugistineCreates:fix/stellar-wallet-validation
Oct 3, 2026
Merged

SudiptaPaul-31 merged 1 commit into
SudiptaPaul-31:mainfrom
AugistineCreates:fix/stellar-wallet-validation

Conversation

@AugistineCreates

Copy link
Copy Markdown
Contributor

Closes #196

Adds robust validation for Stellar wallet addresses before they are stored or used in blockchain operations. This prevents malformed or incompatible addresses from reaching transaction and ownership workflows.

Tasks Completed

✅ Implemented validateStellarAddress utility leveraging stellar-sdk's StrKey.isValidEd25519PublicKey for robust validation and checksum checking.
✅ Integrated the validation utility into authentication (lib/stellar-auth.ts), transaction (lib/stellar.ts), and ownership workflows.
✅ Handled valid, invalid, and edge cases gracefully by returning clear error feedback to avoid silent failures.
✅ Added a comprehensive suite of unit tests covering the validation utility.
Acceptance Criteria Met

Only valid Stellar addresses are accepted; malformed addresses are immediately rejected with standardized errors.
Checks are highly reusable and enforce consistent wallet address requirements across all services.
Database integrity is completely preserved by intercepting invalid inputs before storage or on-chain submission.

@drips-wave

drips-wave Bot commented Sep 27, 2026

Copy link
Copy Markdown

@AugistineCreates 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

@vercel

vercel Bot commented Sep 27, 2026

Copy link
Copy Markdown

@AugistineCreates is attempting to deploy a commit to the Sudipta 's projects Team on Vercel.

A member of the Team first needs to authorize it.

@SudiptaPaul-31
SudiptaPaul-31 merged commit 888a939 into SudiptaPaul-31:main Oct 3, 2026
0 of 2 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.

Implement Stellar Wallet Address Validation

2 participants