code smell fixes #11
Merged
code smell fixes #11
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Vidyaaa24
approved these changes
Jun 20, 2023
swetatech24
added a commit
that referenced
this pull request
Oct 13, 2023
* Readme file added (#1) * Create README.md Added new readme.md file * Update README.md * Removed unnecessary dependencies from pom.xml (#2) * Removed unnecessary dependencies from pom.xml, added crypto.utils to resolve build issues * pom.xml cleanup * updated code-of-conduct, copying, contributing, readme (#3) * Create CONTRIBUTING.md * Create CODE_OF_CONDUCT.md * Create COPYING * Update README.md * modified .gitignore file (#4) * modified .gitignore file * Delete target directory * Update .gitignore * Delete logs directory * code smell fixes (#6) * Added copyright text (#7) Co-authored-by: Anjana K J <[email protected]> * code smell fix (#8) * code smell fix * code smell fixes * Sonarqube reported code smell fix * done correction for application.properties * Swagger changes (#9) * Swagger changes * Swagger changes * Code smell fix (#10) * code smell fixes (#11) * Create sast-and-package-prod.yml (#12) * Create sast-and-package-prod.yml * Update sast-and-package-prod.yml * Code smell changes (#13) * Update README.md * code smell fixes * Added environment variable * Added password changes & edit environment variable (#16) * Added environment variable * Added password changes * Passing environment variable * Passing environment variable (#17) * Passing environment variable * Passing environment variable (#19) * Passing environment variable (#20) * Passing environment variable * Passing environment variable * Workflow change (#21) * Update sast-and-package-prod.yml * Update sast-and-package.yml * CreatedDate filed for Identity (#22) * CreatedDate filed for Identity * removed uncommented code * Send SMS null Id issue (#23) * CreatedDate filed for Identity * removed uncommented code * deleting DB config * Send SMS Null ID Issue * mmu cr changes * update changes * changes mmu cr * Update sast-and-package-prod.yml (#26) * edit api changes * Identity API cleanup and password masking * Updated the language option for codeql and added maven test * Added CodeQl tool to do static code analysis on pull request * Implementing Basic reverse sync operation * Fix issue on controller * Updated the language option for codeql and added maven test * Added CodeQl tool to do static code analysis on pull request * Implementing Basic reverse sync operation * Fix issue on controller * Identity API cleanup and password masking * edit api changes * change made in description of identity controller (#30) * Multiple beneficiaries sync issue changes for FLW (#31) Co-authored-by: Devika <[email protected]> * Delete .github/workflows/sast-and-package-prod.yml * Delete .github/workflows/sast-and-package.yml * Create package-prod.yml * Create package.yml * Create sast.yml * Use a logger to log this exception (#32) * Added Pull Request template, added placeholders in properties file, and added admin_local to git ignore (#33) --------- Co-authored-by: Mahima Elizabeth George <[email protected]> Co-authored-by: Sweta Prakash <[email protected]> Co-authored-by: Vidyadevi Salunkhe <[email protected]> Co-authored-by: Anjana2310 <[email protected]> Co-authored-by: Anjana K J <[email protected]> Co-authored-by: Harimohan Rajamohanan <[email protected]> Co-authored-by: swetatech24 <[email protected]> Co-authored-by: Anjana <[email protected]> Co-authored-by: mukul5947 <[email protected]> Co-authored-by: Indrani Bandyopadhyay <[email protected]> Co-authored-by: devikasuresh20 <[email protected]> Co-authored-by: indraniBan <[email protected]> Co-authored-by: Sunil GV <[email protected]> Co-authored-by: roopesh-beehyv <[email protected]> Co-authored-by: helenKaryamsetty <[email protected]> Co-authored-by: Babacar Ndiaye <[email protected]> Co-authored-by: Devika <[email protected]>
harimohanr
added a commit
that referenced
this pull request
Dec 14, 2023
* Develop branch merge to master (#5) * Readme file added (#1) * Create README.md Added new readme.md file * Update README.md * Removed unnecessary dependencies from pom.xml (#2) * Removed unnecessary dependencies from pom.xml, added crypto.utils to resolve build issues * pom.xml cleanup * updated code-of-conduct, copying, contributing, readme (#3) * Create CONTRIBUTING.md * Create CODE_OF_CONDUCT.md * Create COPYING * Update README.md * modified .gitignore file (#4) * modified .gitignore file * Delete target directory * Update .gitignore * Delete logs directory --------- Co-authored-by: Mahima Elizabeth George <[email protected]> Co-authored-by: Sweta Prakash <[email protected]> Co-authored-by: Vidyadevi Salunkhe <[email protected]> * Feature/dev tomaster (#36) * Readme file added (#1) * Create README.md Added new readme.md file * Update README.md * Removed unnecessary dependencies from pom.xml (#2) * Removed unnecessary dependencies from pom.xml, added crypto.utils to resolve build issues * pom.xml cleanup * updated code-of-conduct, copying, contributing, readme (#3) * Create CONTRIBUTING.md * Create CODE_OF_CONDUCT.md * Create COPYING * Update README.md * modified .gitignore file (#4) * modified .gitignore file * Delete target directory * Update .gitignore * Delete logs directory * code smell fixes (#6) * Added copyright text (#7) Co-authored-by: Anjana K J <[email protected]> * code smell fix (#8) * code smell fix * code smell fixes * Sonarqube reported code smell fix * done correction for application.properties * Swagger changes (#9) * Swagger changes * Swagger changes * Code smell fix (#10) * code smell fixes (#11) * Create sast-and-package-prod.yml (#12) * Create sast-and-package-prod.yml * Update sast-and-package-prod.yml * Code smell changes (#13) * Update README.md * code smell fixes * Added environment variable * Added password changes & edit environment variable (#16) * Added environment variable * Added password changes * Passing environment variable * Passing environment variable (#17) * Passing environment variable * Passing environment variable (#19) * Passing environment variable (#20) * Passing environment variable * Passing environment variable * Workflow change (#21) * Update sast-and-package-prod.yml * Update sast-and-package.yml * CreatedDate filed for Identity (#22) * CreatedDate filed for Identity * removed uncommented code * Send SMS null Id issue (#23) * CreatedDate filed for Identity * removed uncommented code * deleting DB config * Send SMS Null ID Issue * mmu cr changes * update changes * changes mmu cr * Update sast-and-package-prod.yml (#26) * edit api changes * Identity API cleanup and password masking * Updated the language option for codeql and added maven test * Added CodeQl tool to do static code analysis on pull request * Implementing Basic reverse sync operation * Fix issue on controller * Updated the language option for codeql and added maven test * Added CodeQl tool to do static code analysis on pull request * Implementing Basic reverse sync operation * Fix issue on controller * Identity API cleanup and password masking * edit api changes * change made in description of identity controller (#30) * Multiple beneficiaries sync issue changes for FLW (#31) Co-authored-by: Devika <[email protected]> * Delete .github/workflows/sast-and-package-prod.yml * Delete .github/workflows/sast-and-package.yml * Create package-prod.yml * Create package.yml * Create sast.yml * Use a logger to log this exception (#32) * Added Pull Request template, added placeholders in properties file, and added admin_local to git ignore (#33) --------- Co-authored-by: Mahima Elizabeth George <[email protected]> Co-authored-by: Sweta Prakash <[email protected]> Co-authored-by: Vidyadevi Salunkhe <[email protected]> Co-authored-by: Anjana2310 <[email protected]> Co-authored-by: Anjana K J <[email protected]> Co-authored-by: Harimohan Rajamohanan <[email protected]> Co-authored-by: swetatech24 <[email protected]> Co-authored-by: Anjana <[email protected]> Co-authored-by: mukul5947 <[email protected]> Co-authored-by: Indrani Bandyopadhyay <[email protected]> Co-authored-by: devikasuresh20 <[email protected]> Co-authored-by: indraniBan <[email protected]> Co-authored-by: Sunil GV <[email protected]> Co-authored-by: roopesh-beehyv <[email protected]> Co-authored-by: helenKaryamsetty <[email protected]> Co-authored-by: Babacar Ndiaye <[email protected]> Co-authored-by: Devika <[email protected]> * Resolved a merge conflict --------- Co-authored-by: Mahima Elizabeth George <[email protected]> Co-authored-by: Sweta Prakash <[email protected]> Co-authored-by: Vidyadevi Salunkhe <[email protected]> Co-authored-by: ravishanigarapu <[email protected]> Co-authored-by: Anjana2310 <[email protected]> Co-authored-by: Anjana K J <[email protected]> Co-authored-by: swetatech24 <[email protected]> Co-authored-by: Anjana <[email protected]> Co-authored-by: mukul5947 <[email protected]> Co-authored-by: Indrani Bandyopadhyay <[email protected]> Co-authored-by: devikasuresh20 <[email protected]> Co-authored-by: indraniBan <[email protected]> Co-authored-by: Sunil GV <[email protected]> Co-authored-by: roopesh-beehyv <[email protected]> Co-authored-by: helenKaryamsetty <[email protected]> Co-authored-by: Babacar Ndiaye <[email protected]> Co-authored-by: Devika <[email protected]>
ravishanigarapu
added a commit
that referenced
this pull request
Dec 14, 2023
* Readme file added (#1) * Create README.md Added new readme.md file * Update README.md * Removed unnecessary dependencies from pom.xml (#2) * Removed unnecessary dependencies from pom.xml, added crypto.utils to resolve build issues * pom.xml cleanup * updated code-of-conduct, copying, contributing, readme (#3) * Create CONTRIBUTING.md * Create CODE_OF_CONDUCT.md * Create COPYING * Update README.md * modified .gitignore file (#4) * modified .gitignore file * Delete target directory * Update .gitignore * Delete logs directory * code smell fixes (#6) * Added copyright text (#7) Co-authored-by: Anjana K J <[email protected]> * code smell fix (#8) * code smell fix * code smell fixes * Sonarqube reported code smell fix * done correction for application.properties * Swagger changes (#9) * Swagger changes * Swagger changes * Code smell fix (#10) * code smell fixes (#11) * Create sast-and-package-prod.yml (#12) * Create sast-and-package-prod.yml * Update sast-and-package-prod.yml * Code smell changes (#13) * Update README.md * code smell fixes * Added environment variable * Added password changes & edit environment variable (#16) * Added environment variable * Added password changes * Passing environment variable * Passing environment variable (#17) * Passing environment variable * Passing environment variable (#19) * Passing environment variable (#20) * Passing environment variable * Passing environment variable * Workflow change (#21) * Update sast-and-package-prod.yml * Update sast-and-package.yml * CreatedDate filed for Identity (#22) * CreatedDate filed for Identity * removed uncommented code * Send SMS null Id issue (#23) * CreatedDate filed for Identity * removed uncommented code * deleting DB config * Send SMS Null ID Issue * mmu cr changes * update changes * changes mmu cr * Update sast-and-package-prod.yml (#26) * edit api changes * Identity API cleanup and password masking * Updated the language option for codeql and added maven test * Added CodeQl tool to do static code analysis on pull request * Implementing Basic reverse sync operation * Fix issue on controller * Updated the language option for codeql and added maven test * Added CodeQl tool to do static code analysis on pull request * Implementing Basic reverse sync operation * Fix issue on controller * Identity API cleanup and password masking * edit api changes * change made in description of identity controller (#30) * Multiple beneficiaries sync issue changes for FLW (#31) Co-authored-by: Devika <[email protected]> * Delete .github/workflows/sast-and-package-prod.yml * Delete .github/workflows/sast-and-package.yml * Create package-prod.yml * Create package.yml * Create sast.yml * Use a logger to log this exception (#32) * Added Pull Request template, added placeholders in properties file, and added admin_local to git ignore (#33) * Add CI build properties, add pom.xml plugins to tweak build process (#34) * Add CI build properties, add pom.xml plugins to tweak build process * Correct .gitignore file names * http insecure changes (#39) Co-authored-by: Burdhi. Shivani <[email protected]> * Merge branch 'master' (#40) * Merge branch 'master' of https://github.com/helenKaryamsetty/Identity-API * Delete src/main/java/com/iemr/common/identity/utils/CryptoUtil.java --------- Co-authored-by: KA40094929 <[email protected]> Co-authored-by: ravishanigarapu <[email protected]> * Merge conflict resolution prior to develop -> master merge (#42) * Develop branch merge to master (#5) * Readme file added (#1) * Create README.md Added new readme.md file * Update README.md * Removed unnecessary dependencies from pom.xml (#2) * Removed unnecessary dependencies from pom.xml, added crypto.utils to resolve build issues * pom.xml cleanup * updated code-of-conduct, copying, contributing, readme (#3) * Create CONTRIBUTING.md * Create CODE_OF_CONDUCT.md * Create COPYING * Update README.md * modified .gitignore file (#4) * modified .gitignore file * Delete target directory * Update .gitignore * Delete logs directory --------- Co-authored-by: Mahima Elizabeth George <[email protected]> Co-authored-by: Sweta Prakash <[email protected]> Co-authored-by: Vidyadevi Salunkhe <[email protected]> * Feature/dev tomaster (#36) * Readme file added (#1) * Create README.md Added new readme.md file * Update README.md * Removed unnecessary dependencies from pom.xml (#2) * Removed unnecessary dependencies from pom.xml, added crypto.utils to resolve build issues * pom.xml cleanup * updated code-of-conduct, copying, contributing, readme (#3) * Create CONTRIBUTING.md * Create CODE_OF_CONDUCT.md * Create COPYING * Update README.md * modified .gitignore file (#4) * modified .gitignore file * Delete target directory * Update .gitignore * Delete logs directory * code smell fixes (#6) * Added copyright text (#7) Co-authored-by: Anjana K J <[email protected]> * code smell fix (#8) * code smell fix * code smell fixes * Sonarqube reported code smell fix * done correction for application.properties * Swagger changes (#9) * Swagger changes * Swagger changes * Code smell fix (#10) * code smell fixes (#11) * Create sast-and-package-prod.yml (#12) * Create sast-and-package-prod.yml * Update sast-and-package-prod.yml * Code smell changes (#13) * Update README.md * code smell fixes * Added environment variable * Added password changes & edit environment variable (#16) * Added environment variable * Added password changes * Passing environment variable * Passing environment variable (#17) * Passing environment variable * Passing environment variable (#19) * Passing environment variable (#20) * Passing environment variable * Passing environment variable * Workflow change (#21) * Update sast-and-package-prod.yml * Update sast-and-package.yml * CreatedDate filed for Identity (#22) * CreatedDate filed for Identity * removed uncommented code * Send SMS null Id issue (#23) * CreatedDate filed for Identity * removed uncommented code * deleting DB config * Send SMS Null ID Issue * mmu cr changes * update changes * changes mmu cr * Update sast-and-package-prod.yml (#26) * edit api changes * Identity API cleanup and password masking * Updated the language option for codeql and added maven test * Added CodeQl tool to do static code analysis on pull request * Implementing Basic reverse sync operation * Fix issue on controller * Updated the language option for codeql and added maven test * Added CodeQl tool to do static code analysis on pull request * Implementing Basic reverse sync operation * Fix issue on controller * Identity API cleanup and password masking * edit api changes * change made in description of identity controller (#30) * Multiple beneficiaries sync issue changes for FLW (#31) Co-authored-by: Devika <[email protected]> * Delete .github/workflows/sast-and-package-prod.yml * Delete .github/workflows/sast-and-package.yml * Create package-prod.yml * Create package.yml * Create sast.yml * Use a logger to log this exception (#32) * Added Pull Request template, added placeholders in properties file, and added admin_local to git ignore (#33) --------- Co-authored-by: Mahima Elizabeth George <[email protected]> Co-authored-by: Sweta Prakash <[email protected]> Co-authored-by: Vidyadevi Salunkhe <[email protected]> Co-authored-by: Anjana2310 <[email protected]> Co-authored-by: Anjana K J <[email protected]> Co-authored-by: Harimohan Rajamohanan <[email protected]> Co-authored-by: swetatech24 <[email protected]> Co-authored-by: Anjana <[email protected]> Co-authored-by: mukul5947 <[email protected]> Co-authored-by: Indrani Bandyopadhyay <[email protected]> Co-authored-by: devikasuresh20 <[email protected]> Co-authored-by: indraniBan <[email protected]> Co-authored-by: Sunil GV <[email protected]> Co-authored-by: roopesh-beehyv <[email protected]> Co-authored-by: helenKaryamsetty <[email protected]> Co-authored-by: Babacar Ndiaye <[email protected]> Co-authored-by: Devika <[email protected]> * Resolved a merge conflict --------- Co-authored-by: Mahima Elizabeth George <[email protected]> Co-authored-by: Sweta Prakash <[email protected]> Co-authored-by: Vidyadevi Salunkhe <[email protected]> Co-authored-by: ravishanigarapu <[email protected]> Co-authored-by: Anjana2310 <[email protected]> Co-authored-by: Anjana K J <[email protected]> Co-authored-by: swetatech24 <[email protected]> Co-authored-by: Anjana <[email protected]> Co-authored-by: mukul5947 <[email protected]> Co-authored-by: Indrani Bandyopadhyay <[email protected]> Co-authored-by: devikasuresh20 <[email protected]> Co-authored-by: indraniBan <[email protected]> Co-authored-by: Sunil GV <[email protected]> Co-authored-by: roopesh-beehyv <[email protected]> Co-authored-by: helenKaryamsetty <[email protected]> Co-authored-by: Babacar Ndiaye <[email protected]> Co-authored-by: Devika <[email protected]> --------- Co-authored-by: Mahima Elizabeth George <[email protected]> Co-authored-by: Sweta Prakash <[email protected]> Co-authored-by: Vidyadevi Salunkhe <[email protected]> Co-authored-by: Anjana2310 <[email protected]> Co-authored-by: Anjana K J <[email protected]> Co-authored-by: Harimohan Rajamohanan <[email protected]> Co-authored-by: swetatech24 <[email protected]> Co-authored-by: Anjana <[email protected]> Co-authored-by: ravishanigarapu <[email protected]> Co-authored-by: mukul5947 <[email protected]> Co-authored-by: Indrani Bandyopadhyay <[email protected]> Co-authored-by: Ravi Shanigarapu <[email protected]> Co-authored-by: devikasuresh20 <[email protected]> Co-authored-by: indraniBan <[email protected]> Co-authored-by: Sunil GV <[email protected]> Co-authored-by: roopesh-beehyv <[email protected]> Co-authored-by: Babacar Ndiaye <[email protected]> Co-authored-by: Devika <[email protected]> Co-authored-by: Mithun James <[email protected]> Co-authored-by: bshivani7 <[email protected]> Co-authored-by: Burdhi. Shivani <[email protected]> Co-authored-by: KA40094929 <[email protected]> Co-authored-by: Harimohan Rajamohanan <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.