Skip to content

fix: Patient registration password complexity and blocklist check (#1348)#1503

Open
RishiByte wants to merge 8 commits into
gopaljilab:mainfrom
RishiByte:codex-fix-patient-password-complexity
Open

fix: Patient registration password complexity and blocklist check (#1348)#1503
RishiByte wants to merge 8 commits into
gopaljilab:mainfrom
RishiByte:codex-fix-patient-password-complexity

Conversation

@RishiByte

Copy link
Copy Markdown
Contributor

This PR introduces strict password complexity requirements on the patient registration endpoint (8+ characters, uppercase, lowercase, number, and special character) and implements checks against the 10,000 most common passwords list. It adds validation on both the client and server side, displays visual cues on the UI, and provides integration testing coverage.

@vercel

vercel Bot commented Jun 19, 2026

Copy link
Copy Markdown

@RishiByte is attempting to deploy a commit to the gopaljilab's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the type:bug Something isn't working or throwing errors. label Jun 19, 2026
@RishiByte

Copy link
Copy Markdown
Contributor Author

@gopaljilab Please Merge this too

@github-actions github-actions Bot added the type:security Vulnerability fixes or security-related enhancements. label Jun 20, 2026
@RishiByte

Copy link
Copy Markdown
Contributor Author

@gopaljilab No issue here as well. Please merge all my 5 pr now i worked on them for a long time atp

@gopaljilab

gopaljilab commented Jun 27, 2026

Copy link
Copy Markdown
Owner

Hi @RishiByte,

Thanks for your patience. I took a look at the branch, and it currently has active merge conflicts across multiple files.

Please address these issues so we can green-light the builds and move forward:

Resolve Merge Conflicts

There are quite a few files currently blocking the build due to conflicts. Please resolve them (either locally or via GitHub's web editor), including:

  • .eslintrc.json
  • client/src/components/ui/form.tsx
  • client/src/lib/queryClient.ts
  • server/index.ts
  • server/middleware/rlsContext.ts
  • server/middleware/validateDTO.ts
  • server/routes/assessments.routes.ts
  • server/routes/ml.routes.ts
  • server/routes/patient.routes.ts
  • server/routes/upload.routes.ts
  • server/services/fhirParser.ts
  • server/types/express-session.d.ts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Something isn't working or throwing errors. type:security Vulnerability fixes or security-related enhancements.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants