pucardotorg / dristi

MIT License
2 stars 12 forks source link

DRISTI: Fix all code scanning alerts #1603

Open subhashini-egov opened 1 month ago

subhashini-egov commented 1 month ago

Fix all critical and high code smells

TusharKatiyar2002 commented 1 month ago

Most of the issues were related to globalConfigs file, deleted the file. The remaining issues are solved.

Beehyv-Vinod commented 1 month ago

Checked with @TusharKatiyar2002 , the file changes were done. Did a quick testing of flow to make sure any dependency change didn't impact on any of the module. It's working fine

Ramu-kandimalla commented 1 month ago

Hi @subhashini-egov This alert resolved and taken care

subhashini-egov commented 1 month ago

@Ramu-kandimalla @Beehyv-Vinod - Please check this tab https://github.com/pucardotorg/dristi-solutions/security/code-scanning?query=is%3Aopen+branch%3Adevelop+severity%3Acritical. We still have 56 critical issues open. Atul has forwarded an e-mail with fixes for the DIGIT core platform user service. All those should be incorporated.

Ramu-kandimalla commented 1 month ago

Hi @TusharKatiyar2002 @Beehyv-Vinod Please check and confirm

Hemadri-Chinna commented 4 days ago

Image initally suggested to change webmvc version to 5.3.18 after updating and changing the version and raise the pr, in pr it is sggesting to change to higher version Image

rajeshcherukumalli commented 2 days ago

Moving this ticket to sprint 12 for QA testing