Skip to content

feat: Add Medical Data Breach Checker & Fix CI/CD Pipeline#177

Open
harsh-hak wants to merge 466 commits intoGopher-Industries:masterfrom
harsh-hak:master
Open

feat: Add Medical Data Breach Checker & Fix CI/CD Pipeline#177
harsh-hak wants to merge 466 commits intoGopher-Industries:masterfrom
harsh-hak:master

Conversation

@harsh-hak
Copy link

@harsh-hak harsh-hak commented Jan 25, 2026

Description

Exposes the API endpoint for the Medical Breach Checker.

  • Registers Medical_Breach module in routes/index.js.
  • Endpoint: /api/security/medical-breach/breach-check.

zeyu-linn and others added 30 commits May 2, 2025 13:06
…tries#89)

* init new route and refactor code for customized rce

* update option for excluding ingredients in RCE

* completed testing and fixed bugs for excluding ingredients in rce feature

* complated refactor rce api: update controller, swagger ui, testing file
…act us fields (Gopher-Industries#90)

* input validation implemented for login, signup and appointments

* input validation implemented for contact us endpoint

* corrected some errors in input validation

* corrected some errors in input validation

* implemented input validation for feedback endpoint

* implemented input validation for meal plan and recipe
…nt_substitution

Optimize Model part of Ingredient Substitution API
…ltharticle-api

Health article search function
…e/update-user-profile

Feature/update user profile
…-yaml-file

fixing the bug in index.yaml file
Added global and route-specific rate limiting
Conflicts removed from rate-limiting and validating logic
wenyupeng and others added 27 commits December 7, 2025 21:36
… and token management, and add initial health news tests.
…and token management, and refactor server to be exportable.
…imize

home service feature and appointment api update
Updated login, signup controllers and authentication middleware
* add data classification table for sensitive data management

* reduce upload limit to 3 per 10 minutes in rate limiter

* Add log inventory table and update data classification table

review & approve by King Hei
@elanlaw1206
Copy link
Contributor

Hi Harsh,

Thanks for the work on the Medical Data Breach Checker.

However, I can’t proceed with this PR in its current form.

This PR is based on a force-pushed master branch and attempts to merge
harsh-hak:master into Gopher-Industries:master. Rewriting the master
branch history causes significant risk in a shared, multi-trimester
project and has introduced a large number of conflicts across core files
(e.g. CI workflow, auth middleware, server.js, package.json).

Because of this:
• The PR scope is too broad (hundreds of commits and unrelated changes)
• The conflicts make it unsafe to review or merge
• We cannot accept force-pushed master history into the main repository

To move forward, please:

  1. Restore your local master to match Gopher-Industries:master
  2. Create a new feature branch from the current upstream master
  3. Isolate only the Medical Data Breach Checker–related changes
  4. Open a fresh PR with a clean, focused diff

Once the changes are scoped and based on a proper feature branch, I’m happy to review again.

Thanks for understanding.

King Hei

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.