Skip to content
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

Fix Enhance Input Validation #68 #69

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Sachin-NK
Copy link

Related to #68

This PR enhances the API serializers by addressing multiple issues to improve data integrity, security, and error prevention:

  • Email Validation: Added email checks using validate_email for user-related serializers to ensure valid email formats.
  • Password Handling: The password field in UserRegisterSerializer is now set to write_only to prevent exposure in API responses.
  • ModelSerializer Update: Converted UserSerializer to a ModelSerializer for better integration with the model.

These improvements help catch errors early, provide clearer error messages, and make the API more reliable.

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.

1 participant