Skip to content

feat: add input validation/sanitization and error handling/logging in external bridge service endpoints#35

Open
slopefields wants to merge 4 commits intodevfrom
justin_yoo/feature/endpoint_sanitization
Open

feat: add input validation/sanitization and error handling/logging in external bridge service endpoints#35
slopefields wants to merge 4 commits intodevfrom
justin_yoo/feature/endpoint_sanitization

Conversation

@slopefields
Copy link
Copy Markdown
Member

@slopefields slopefields commented Mar 17, 2026

Description

Adds input (body, header, query) validation/sanitization and error handling/logging across external bridge service endpoints

  • Plaky feature name: "Add comprehensive, Add input validation and sanitization for all API endpoints, error handling and logging middleware across all API endpoints"
  • Component, feature, or system affected: external bridge service routes and input validation middleware
  • Purpose of change: improve endpoint reliability and request handling

Changes

  • Added standardized validation for request body, header, and query parameters
  • Added standardized sanitization of incoming request data before processing
  • Added/improved error logging for invalid requests and failures

Related

Related PRs (not dependent):
deepiri-platform: Team-Deepiri/deepiri-platform#145
deepiri-auth-service: Team-Deepiri/deepiri-auth-service#31
deepiri-api-gateway: Team-Deepiri/deepiri-api-gateway#40


Testing

  • Ran endpoint checks with dummy requests and confirmed validation/sanitization behavior

Workflow Checklist

  • Branch is up to date with dev
  • PR is from branch → dev
  • PR title follows convention (feat:, fix:, refactor:, etc.)
  • Plaky feature/bug name included above
  • Tagged @Team-Deepiri/support-team
  • Plaky feature moved to "Needs QA"

Review Requests

@Team-Deepiri/support-team

@austinm2h35-sketch
Copy link
Copy Markdown
Contributor

@Todd838 This is related to your assigned PR in deepiri-platform

@jrb00013
Copy link
Copy Markdown
Member

@Todd838 What's going on with this one

@jrb00013
Copy link
Copy Markdown
Member

jrb00013 commented Apr 1, 2026

@Todd838 let's finish this my man, you have to be proactive and reach out for help if you need help

@jrb00013
Copy link
Copy Markdown
Member

@Todd838

@asheenhameeda8-cpu
Copy link
Copy Markdown

@Todd838 it's been a while finish this up

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.

4 participants