Skip to content

Conversation

@peanutenthusiast
Copy link
Collaborator

@peanutenthusiast peanutenthusiast commented Jul 9, 2025

Description

This PR implements

  • validation for Twilio Messages, generic OpenedX event notifications
  • unit tests for custom function validatePhoneNumber to validate US phone numbers starting with +1
  • unit tests for custom function validateSafeString to validate against potential unsafe strings for SQL injection, XSS attacks, etc.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor
  • Other (please describe):

How Has This Been Tested?

Added unit tests to check for the above custom functions

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code where necessary
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have updated the documentation (if applicable)

Related Issues

Fixes #11 (if applicable)

@peanutenthusiast peanutenthusiast marked this pull request as draft July 9, 2025 04:19
@peanutenthusiast peanutenthusiast marked this pull request as ready for review July 14, 2025 00:20
@ramakanth98 ramakanth98 self-requested a review July 26, 2025 18:22
@ramakanth98 ramakanth98 merged commit d2595be into main Jul 26, 2025
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 webhook endpoint to receive incoming messages.

4 participants