Skip to content

fix: add max length validation on signature text input#296

Merged
Durgeshwar-AI merged 1 commit into
Durgeshwar-AI:mainfrom
pericharlabindhumadhavi-data:fix/272-signature-length-validation
Jun 13, 2026
Merged

fix: add max length validation on signature text input#296
Durgeshwar-AI merged 1 commit into
Durgeshwar-AI:mainfrom
pericharlabindhumadhavi-data:fix/272-signature-length-validation

Conversation

@pericharlabindhumadhavi-data

@pericharlabindhumadhavi-data pericharlabindhumadhavi-data commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

🔀 Pull Request

📌 Issue Reference

Closes #272


📝 Summary

Problem

The PDF Sign feature accepted signature text of any length without validation.

This caused:

  • Users to submit excessively long signature text.
  • Potential rendering and processing issues when generating signatures.
  • No consistent input validation between frontend and backend.

Solution

This PR introduces a maximum signature length limit of 500 characters and enforces the restriction on both the frontend and backend.

The frontend prevents users from entering more than 500 characters, while the backend validates incoming requests and rejects oversized inputs with an appropriate error message.

Changes Made

  • Added backend validation for signature text length.
  • Added a 500-character maximum limit.
  • Added frontend maxLength={500} validation.
  • Added clear error handling for oversized signature inputs.

✅ Checklist

  • My code follows the project's coding conventions
  • I have tested all impacted features
  • I have updated or added necessary documentation

🔗 Related Issues / PRs

🏅 Open Source Program Participation

Program Name: GSSoC 2026 (GirlScript Summer of Code)

💬 Additional Notes

Testing Performed

  • Verified signatures under 500 characters are accepted.
  • Verified signatures exceeding 500 characters are rejected.
  • Verified frontend prevents entering more than 500 characters.
  • Verified backend validation works even when requests bypass the frontend.

@vercel

vercel Bot commented Jun 12, 2026

Copy link
Copy Markdown

@pericharlabindhumadhavi-data is attempting to deploy a commit to the Durgeshwar's projects Team on Vercel.

A member of the Team first needs to authorize it.

@pericharlabindhumadhavi-data pericharlabindhumadhavi-data changed the title fix(#272): add max length validation on signature text input fix: add max length validation on signature text input Jun 12, 2026
@Durgeshwar-AI Durgeshwar-AI merged commit c7a314e into Durgeshwar-AI:main Jun 13, 2026
1 check failed
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.

[BUG] PDF Sign — no maximum length validation on signature text input

2 participants