Skip to content

PII ISSUE FIXES [MOSIP-44379] #1029

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.

Sign up for GitHub

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

Draft
GOKULRAJ136 wants to merge 10 commits into
base: develop
Choose a base branch
from
Draft

PII ISSUE FIXES [MOSIP-44379] #1029

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
0e6db31
PII Issue Fixes (#1024)
GOKULRAJ136 Feb 20, 2026
2ddcb71
Fixed User Encryption PII
GOKULRAJ136 Feb 20, 2026
55d3b45
Fixed User Encryption PII (#1025)
GOKULRAJ136 Feb 20, 2026
8011dd7
Merge remote-tracking branch 'upstream/MOSIP-PII-dev' into demo-PII
GOKULRAJ136 Feb 23, 2026
7391018
Updated user validation paths
GOKULRAJ136 Feb 23, 2026
88a346e
Updated User Validations
GOKULRAJ136 Feb 23, 2026
37549af
Merge pull request #1026 from GOKULRAJ136/demo-PII
ckm007 Feb 23, 2026
7ddd9b3
Revert "Updated user validation paths" (#1027)
GOKULRAJ136 Feb 24, 2026
b8466ce
Added user_details table (#1028)
GOKULRAJ136 Feb 25, 2026
e7fda69
Core Changes PII (#1037)
GOKULRAJ136 Mar 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 9 additions & 9 deletions api-test/src/main/resources/preReg/AddLostUinApplication/AddLostUinApplication.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ AddLostUinApplication:
Prereg_AddLostUinApplication_All_Valid_Smoke_sid:
endPoint: /preregistration/v1/applications/lostuin
uniqueIdentifier: TC_prereg_AddLostUinApplication_01
description: process the lost UIN application with a valid ID
description: validate that the Lost UIN application is successfully processed when a valid ID is provided
role: batch
templateFields: ["langCode"]
restMethod: post
Expand All @@ -21,7 +21,7 @@ AddLostUinApplication:
Prereg_AddLostUinApplication_Invalid_id:
endPoint: /preregistration/v1/applications/lostuin
uniqueIdentifier: TC_prereg_AddLostUinApplication_02
description: process the lost UIN application with an invalid ID
description: validate that the Lost UIN application is not processed when an invalid ID is provided
role: batch
templateFields: ["langCode"]
restMethod: post
Expand All @@ -43,7 +43,7 @@ AddLostUinApplication:
Prereg_AddLostUinApplication_invalid_lang_code:
endPoint: /preregistration/v1/applications/lostuin
uniqueIdentifier: TC_prereg_AddLostUinApplication_03
description: Process the lost UIN application with an invalid language code
description: validate that the Lost UIN application is not processed when an invalid language code is provided
role: batch
restMethod: post
inputTemplate: preReg/AddLostUinApplication/addLostUinApplication
Expand All @@ -64,7 +64,7 @@ AddLostUinApplication:
Prereg_AddLostUinApplication_without_version:
endPoint: /preregistration/v1/applications/lostuin
uniqueIdentifier: TC_prereg_AddLostUinApplication_04
description: Process the lost UIN application without specifying the version
description: validate that the Lost UIN application is not processed when the version is not provided
role: batch
templateFields: ["langCode"]
restMethod: post
Expand All @@ -86,7 +86,7 @@ AddLostUinApplication:
Prereg_AddLostUinApplication_without_version_attribute:
endPoint: /preregistration/v1/applications/lostuin
uniqueIdentifier: TC_prereg_AddLostUinApplication_05
description: Process the lost UIN application without specifying the version and attribute
description: validate that the Lost UIN application is not processed when the version attribute is not provided
role: batch
checkErrorsOnlyInResponse: true
templateFields: ["langCode"]
Expand All @@ -109,7 +109,7 @@ AddLostUinApplication:
Prereg_AddLostUinApplication_with_INVALID_version:
endPoint: /preregistration/v1/applications/lostuin
uniqueIdentifier: TC_prereg_AddLostUinApplication_06
description: Process the lost UIN application with an invalid version
description: validate that the Lost UIN application is not processed when an invalid version is provided
role: batch
templateFields: ["langCode"]
restMethod: post
Expand All @@ -131,7 +131,7 @@ AddLostUinApplication:
Prereg_AddLostUinApplication_Valid_Smoke_sid:
endPoint: /preregistration/v1/applications/lostuin
uniqueIdentifier: TC_prereg_AddLostUinApplication_07
description: Process the lost UIN application with a valid data
description: validate that the Lost UIN application is successfully processed when all mandatory fields are provided with valid values
role: batch
templateFields: ["langCode"]
restMethod: post
Expand All @@ -151,7 +151,7 @@ AddLostUinApplication:
Prereg_AddLostUinApplication_inValid_langcode:
endPoint: /preregistration/v1/applications/lostuin
uniqueIdentifier: TC_prereg_AddLostUinApplication_08
description: Process the lost UIN application with an invalid language code
description: validate that the Lost UIN application is not processed when an invalid language code is provided
role: batch
restMethod: post
inputTemplate: preReg/AddLostUinApplication/addLostUinApplication
Expand All @@ -172,7 +172,7 @@ AddLostUinApplication:
Prereg_AddLostUinApplication_without_any_data:
endPoint: /preregistration/v1/applications/lostuin
uniqueIdentifier: TC_prereg_AddLostUinApplication_09
description: Process the lost UIN application with no data provided
description: validate that the Lost UIN application is not processed when no data is provided
role: batch
restMethod: post
inputTemplate: preReg/AddLostUinApplication/addLostUinApplication
Expand Down
8 changes: 4 additions & 4 deletions api-test/src/main/resources/preReg/AddUpdateRegistration/AddUpdateRegistration.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ AddUpdateRegistration:
Prereg_AddUpdateRegistration_All_Valid_Smoke_sid:
endPoint: /preregistration/v1/applications/updateregistration
uniqueIdentifier: TC_prereg_AddUpdateRegistration_01
description: add an update to the registration with valid data
description: validate that the Update Registration application is successfully processed when all mandatory fields are provided with valid values
role: batch
templateFields: ["langCode"]
restMethod: post
Expand All @@ -22,7 +22,7 @@ AddUpdateRegistration:
Prereg_AddUpdateRegistration_inValid_langcode:
endPoint: /preregistration/v1/applications/updateregistration
uniqueIdentifier: TC_prereg_AddUpdateRegistration_02
description: add an update to the registration with an invalid data
description: Validate that the Update Registration application is not processed when an invalid language code is provided
role: batch
restMethod: post
inputTemplate: preReg/AddUpdateRegistration/addUpdateRegistration
Expand All @@ -44,7 +44,7 @@ AddUpdateRegistration:
Prereg_AddUpdateRegistration_without_input_Neg:
endPoint: /preregistration/v1/applications/updateregistration
uniqueIdentifier: TC_prereg_AddUpdateRegistration_03
description: Add an update to the registration without input data
description: Validate that the Update Registration application is not processed when no input data is provided
role: batch
restMethod: post
inputTemplate: preReg/AddUpdateRegistration/addUpdateRegistration
Expand All @@ -66,7 +66,7 @@ AddUpdateRegistration:
Prereg_AddUpdateRegistration_missing_langCode_Neg:
endPoint: /preregistration/v1/applications/updateregistration
uniqueIdentifier: TC_prereg_AddUpdateRegistration_04
description: Add an update to the registration with missing lang code
description: Validate that the Update Registration application is not processed when the language code is missing
role: batch
restMethod: post
inputTemplate: preReg/AddUpdateRegistration/addUpdateRegistration
Expand Down
1 change: 1 addition & 0 deletions api-test/src/main/resources/preReg/AuditLog/AuditLog.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ addContact:
Prereg_AddContact_All_Valid_Smoke:
endPoint: SELECT module_name,cr_by FROM audit.app_audit_log where module_name LIKE '%prereg%' order by action_dtimes desc LIMIT 5
role: audit
description: Validate that adding a contact is successfully processed when all mandatory fields are provided with valid values

@coderabbitai coderabbitai Bot Mar 23, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

Replace tab indentation on Line 5 to keep YAML parseable.

Line 5 starts with a tab before description. YAML parsing will fail on this token.

💡 Proposed fix 
-	  description: Validate that adding a contact is successfully processed when all mandatory fields are provided with valid values
+      description: Validate that adding a contact is successfully processed when all mandatory fields are provided with valid values
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
description: Validate that adding a contact is successfully processed when all mandatory fields are provided with valid values
description: Validate that adding a contact is successfully processed when all mandatory fields are provided with valid values
🧰 Tools
🪛 YAMLlint (1.38.0)

[error] 5-5: syntax error: found character '\t' that cannot start any token

(syntax)

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@api-test/src/main/resources/preReg/AuditLog/AuditLog.yml` at line 5, The YAML
file contains a tab character at the start of the description line which breaks
parsing; replace the leading tab on the `description` line with spaces to match
the file's indentation style (e.g., two spaces) so the `description: Validate
that adding a contact...` entry is space-indented and YAML-parseable.

templateFields: ["preregQuery","prereg service"]
restMethod: post
inputTemplate: prereg/AuditLog/auditlog
Expand Down
1 change: 1 addition & 0 deletions api-test/src/main/resources/preReg/BookMultipleAppointment/BookMultipleAppointment.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ BookMultipleAppointment:
Prereg_BookMultipleAppointment_All_Valid_Smoke:
endPoint: /preregistration/v1/appointment
role: batch
description: Validate that booking multiple appointments is successfully processed when all mandatory fields are provided with valid values
checkErrorsOnlyInResponse: true
restMethod: post
inputTemplate: preReg/BookMultipleAppointment/bookMultipleAppointment
Expand Down
4 changes: 2 additions & 2 deletions ...src/main/resources/preReg/CancelApplicationsAppointment/CancelApplicationsAppointment.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ CancelApplicationsAppointment:
Prereg_CancelApplicationsAppointment_All_Valid_Smoke_sid:
endPoint: /preregistration/v1/applications/appointment/{preRegistrationId}
uniqueIdentifier: TC_prereg_CancelApplicationsAppointment_01
description: Canceling appointment for a valid preregistration ID
description: Validate that canceling an appointment is successfully processed when all mandatory fields are provided with valid values
role: batch
checkErrorsOnlyInResponse: true
restMethod: put
Expand All @@ -17,7 +17,7 @@ CancelApplicationsAppointment:
Prereg_CancelApplicationsAppointment_inValid_preRegistrationId_sid:
endPoint: /preregistration/v1/applications/appointment/{preRegistrationId}
uniqueIdentifier: TC_prereg_CancelApplicationsAppointment_02
description: Canceling appointment for an invalid preregistration ID
description: Validate that canceling an appointment fails when an invalid pre registration ID is provided
role: batch
restMethod: put
inputTemplate: preReg/CancelApplicationsAppointment/cancelApplicationsAppointment
Expand Down
10 changes: 5 additions & 5 deletions api-test/src/main/resources/preReg/CancelAppointment/CancelAppointment.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ CancelAppointment:
Prereg_CancelAppointment_All_Valid_Smoke_sid:
endPoint: /preregistration/v1/appointment/{preRegistrationId}
uniqueIdentifier: TC_prereg_CancelAppointment_01
description: Cancel appointment for a valid preregistration ID
description: Validate that an appointment cancellation is successfully processed when all mandatory fields are provided with valid values
role: batch
checkErrorsOnlyInResponse: true
restMethod: put
Expand All @@ -17,7 +17,7 @@ CancelAppointment:
Prereg_CancelAppointment_with_already_cancelled_appoinment:
endPoint: /preregistration/v1/appointment/{preRegistrationId}
uniqueIdentifier: TC_prereg_CancelAppointment_02
description: Canceling an appointment that has already been cancelled
description: Validate that an appointment cancellation fails when the appointment has already been canceled
role: batch
checkErrorsOnlyInResponse: true
restMethod: put
Expand All @@ -37,7 +37,7 @@ CancelAppointment:
Prereg_CancelAppointment_cancell_incomplete_appoinment:
endPoint: /preregistration/v1/appointment/{preRegistrationId}
uniqueIdentifier: TC_prereg_CancelAppointment_03
description: Canceling an appointment with an incomplete appointment data
description: Validate that an appointment cancellation fails when the appointment is incomplete
role: batch
checkErrorsOnlyInResponse: true
restMethod: put
Expand All @@ -57,7 +57,7 @@ CancelAppointment:
Prereg_CancelAppointment_inValid_preRegistrationId_sid:
endPoint: /preregistration/v1/appointment/{preRegistrationId}
uniqueIdentifier: TC_prereg_CancelAppointment_04
description: Canceling an appointment with an invalid preregistration ID
description: Validate that an appointment cancellation fails when an invalid pre registration ID is provided
role: batch
restMethod: put
inputTemplate: preReg/CancelAppointment/cancelAppointment
Expand All @@ -75,7 +75,7 @@ CancelAppointment:
Prereg_CancelAppointment_forPendingAppointment_preRegistrationId_Neg:
endPoint: /preregistration/v1/appointment/{preRegistrationId}
uniqueIdentifier: TC_prereg_CancelAppointment_05
description: Canceling an appointment with an pending appointment preregistration ID
description: Validate that an appointment cancellation fails when the pre registration ID corresponds to a pending appointment
role: batch
restMethod: put
additionalDependencies: TC_prereg_UpdatePreRegStatus_04
Expand Down
8 changes: 4 additions & 4 deletions api-test/src/main/resources/preReg/CopyDocument/CopyDocument.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ CopyDocument:
Prereg_CopyDocument_All_Valid_Smoke_sid:
endPoint: /preregistration/v1/documents/{preRegistrationId}?catCode={doc_cat_code}&sourcePreId={preRegistrationId}
uniqueIdentifier: TC_prereg_CopyDocument_01
description: Copy the valid Proof of Address (POA) document
description: Validate that copying a document is successfully processed when all mandatory fields are provided with valid values
role: batch
restMethod: put
inputTemplate: preReg/CopyDocument/copyDocument
Expand All @@ -18,7 +18,7 @@ CopyDocument:
Prereg_CopyDocument_in_Valid_doc_cat_code_sid:
endPoint: /preregistration/v1/documents/{preRegistrationId}?catCode={doc_cat_code}&sourcePreId={preRegistrationId}
uniqueIdentifier: TC_prereg_CopyDocument_02
description: Copy the Proof of Address (POA) document with the invalid document category code
description: Validate that copying a document fails when an invalid document category code is provided
role: batch
restMethod: put
inputTemplate: preReg/CopyDocument/copyDocument
Expand All @@ -37,7 +37,7 @@ CopyDocument:
Prereg_CopyDocument_blank_doc_cat_code_sid:
endPoint: /preregistration/v1/documents/{preRegistrationId}?catCode={doc_cat_code}&sourcePreId={preRegistrationId}
uniqueIdentifier: TC_prereg_CopyDocument_03
description: Copy the blank document and categorize it with the code
description: Validate that copying a document fails when the document category code is left blank
role: batch
restMethod: put
inputTemplate: preReg/CopyDocument/copyDocument
Expand All @@ -56,7 +56,7 @@ CopyDocument:
Prereg_CopyDocument_Invalid_preRegistrationId_sid:
endPoint: /preregistration/v1/documents/{preRegistrationId}?catCode={doc_cat_code}&sourcePreId={preRegistrationId}
uniqueIdentifier: TC_prereg_CopyDocument_04
description: Copy the Proof of Address (POA) document with the invalid preregistration ID
description: Validate that copying a document fails when an invalid pre-registration ID is provided
role: batch
restMethod: put
inputTemplate: preReg/CopyDocument/copyDocument
Expand Down
4 changes: 2 additions & 2 deletions api-test/src/main/resources/preReg/DeleteAllDocForaPRID/DeleteAllDocForaPRID.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ DeleteAllDocForaPRID:
Prereg_DeletePreRegistration_All_Valid_Smoke_sid:
endPoint: /preregistration/v1/documents/preregistration/{preRegistrationId}
uniqueIdentifier: TC_prereg_DeleteAllDocForaPRID_01
description: Delete preregistration with all valid preregistration ID
description: Validate that deleting a pre registration is successfully processed when all mandatory fields are provided with valid values
role: batch
checkErrorsOnlyInResponse: true
restMethod: delete
Expand All @@ -17,7 +17,7 @@ DeleteAllDocForaPRID:
Prereg_DeletePreRegistration_InValid_preRegistrationId_sid:
endPoint: /preregistration/v1/documents/preregistration/{preRegistrationId}
uniqueIdentifier: TC_prereg_DeleteAllDocForaPRID_02
description: Delete preregistration with an invalid preregisteration ID
description: Validate that deleting a pre registration fails when an invalid pre registration ID is provided
role: batch
restMethod: delete
inputTemplate: preReg/DeleteAllDocForaPRID/deleteAllDocForaPRID
Expand Down
2 changes: 1 addition & 1 deletion api-test/src/main/resources/preReg/DeleteBooking/DeleteBooking.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ DeleteBooking:
Prereg_DeleteBooking_All_Valid_Smoke_sid:
endPoint: /preregistration/v1/applications/appointment?preRegistrationId={preRegistrationId}
uniqueIdentifier: TC_prereg_DeleteBooking_01
description: Delete bookings for all valid preregistration ID
description: Validate that deleting a booking is successfully processed when all mandatory fields are provided with valid values
role: batch
restMethod: delete
additionalDependencies: TC_prereg_BookAppointment_01
Expand Down
10 changes: 5 additions & 5 deletions api-test/src/main/resources/preReg/DeleteLostUinApplication/DeleteLostUinApplication.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ DeleteLostUinApplication:
Prereg_DeleteLostUinApplication_All_Valid_Smoke_sid:
endPoint: /preregistration/v1/applications/prereg/{applicationId}
uniqueIdentifier: TC_prereg_DeleteLostUinApplication_01
description: Delete the lost UIN application with valid data
description: Validate that deleting a lost UIN application is successfully processed when all mandatory fields are provided with valid values
role: batch
checkErrorsOnlyInResponse: true
restMethod: delete
Expand All @@ -17,7 +17,7 @@ DeleteLostUinApplication:
Prereg_DeleteLostUinApplication_after_its_deletion:
endPoint: /preregistration/v1/applications/prereg/{applicationId}
uniqueIdentifier: TC_prereg_DeleteLostUinApplication_02
description: Delete the lost UIN application after it has been removed
description: Validate that deleting a lost UIN application fails if the application has already been deleted
role: batch
checkErrorsOnlyInResponse: true
restMethod: delete
Expand All @@ -36,7 +36,7 @@ DeleteLostUinApplication:
Prereg_DeleteLostUinApplication_multiple_application_sid:
endPoint: /preregistration/v1/applications/prereg/{applicationId}
uniqueIdentifier: TC_prereg_DeleteLostUinApplication_03
description: Delete the lost UIN application with multiple preregistration IDs for the application
description: Validate that deleting multiple lost UIN applications is successfully processed when all mandatory fields are provided with valid values
role: batch
checkErrorsOnlyInResponse: true
restMethod: delete
Expand All @@ -51,7 +51,7 @@ DeleteLostUinApplication:
Prereg_DeleteLostUinApplication_inValid_preRegistrationId_sid:
endPoint: /preregistration/v1/applications/prereg/{applicationId}
uniqueIdentifier: TC_prereg_DeleteLostUinApplication_04
description: Delete the lost UIN application with an invalid preregistration ID
description: Validate that deleting a lost UIN application fails when an invalid pre registration ID is provided
role: batch
restMethod: delete
inputTemplate: preReg/DeleteLostUinApplication/deleteLostUinApplication
Expand All @@ -69,7 +69,7 @@ DeleteLostUinApplication:
Prereg_DeleteLostUinApplication_random_number_sid:
endPoint: /preregistration/v1/applications/prereg/{applicationId}
uniqueIdentifier: TC_prereg_DeleteLostUinApplication_05
description: Delete the lost UIN application with a random number preregistration ID
description: Validate that deleting a lost UIN application fails when a random or non-existent pre registration ID is provided
role: batch
restMethod: delete
inputTemplate: preReg/DeleteLostUinApplication/deleteLostUinApplication
Expand Down
4 changes: 2 additions & 2 deletions api-test/src/main/resources/preReg/DeletePreRegistration/DeletePreRegistration.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ DeletePreRegistration:
Prereg_DeletePreRegistration_All_Valid_Smoke_sid:
endPoint: /preregistration/v1/applications/prereg/{preRegistrationId}
uniqueIdentifier: TC_prereg_DeletePreRegistration_01
description: Delete the preregistration with a valid preregistration ID
description: Validate that deleting a pre registration is successfully processed when all mandatory fields are provided with valid values
role: batch
restMethod: delete
inputTemplate: preReg/DeletePreRegistration/deletePreRegistration
Expand All @@ -16,7 +16,7 @@ DeletePreRegistration:
Prereg_DeletePreRegistration_inValid_preRegistrationId_sid:
endPoint: /preregistration/v1/applications/prereg/{preRegistrationId}
uniqueIdentifier: TC_prereg_DeletePreRegistration_02
description: Delete the preregistration with an invalid preregistration ID
description: Validate that deleting a pre registration fails when an invalid pre registration ID is provided
role: batch
restMethod: delete
inputTemplate: preReg/DeletePreRegistration/deletePreRegistration
Expand Down
Loading
Loading