Skip to content
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

Jump to bottom

story #11496 fix(archive-search): missing search for trackTotalHits + bug #11641 fix(ui-frontend-common): archive-unit-count component state #1409

Merged
merged 2 commits into from
Aug 28, 2023
Merged

story #11496 fix(archive-search): missing search for trackTotalHits + bug #11641 fix(ui-frontend-common): archive-unit-count component state #1409

merged 2 commits into from
Aug 28, 2023

Conversation

Regzox
Copy link
Contributor

@Regzox Regzox commented Jul 10, 2023 •
edited
Loading

Description

  • Corrige l'oublie du passage de la fonction de recherche du trackTotalHits au composant gérant les 10k+ éléments suite à une refactorisation.
  • Corrige les problème de gestion d'état du bug #11641

@Regzox Regzox self-assigned this Jul 10, 2023
@Regzox Regzox added bug Something isn't working javascript Pull requests that update Javascript code small pr embarquant peu de changements et à review rapide, ne nécessitant qu'un reviewer VAS VAS contribution labels Jul 10, 2023
@Regzox Regzox added this to the IT 121 milestone Jul 10, 2023
@TDevillechabrolle
Copy link
Contributor

TDevillechabrolle commented Jul 10, 2023 •
edited
Loading

Logo
Checkmarx One – Scan Summary & Details – 1a4f1d71-34b8-496f-b64d-bfb7824c5886

New Issues

Severity Issue Source File / Package Checkmarx Insight
LOW Log_Forging /ui/ui-archive-search/src/main/java/fr/gouv/vitamui/archives/search/rest/ArchivesSearchController.java: 322 Attack Vector
LOW Log_Forging /ui/ui-archive-search/src/main/java/fr/gouv/vitamui/archives/search/rest/ArchivesSearchController.java: 277 Attack Vector
LOW Log_Forging /ui/ui-archive-search/src/main/java/fr/gouv/vitamui/archives/search/rest/ArchivesSearchController.java: 228 Attack Vector
LOW Log_Forging /ui/ui-archive-search/src/main/java/fr/gouv/vitamui/archives/search/rest/ArchivesSearchController.java: 239 Attack Vector
LOW Log_Forging /ui/ui-archive-search/src/main/java/fr/gouv/vitamui/archives/search/rest/ArchivesSearchController.java: 311 Attack Vector
LOW Log_Forging /ui/ui-archive-search/src/main/java/fr/gouv/vitamui/archives/search/rest/ArchivesSearchController.java: 277 Attack Vector
LOW Log_Forging /ui/ui-archive-search/src/main/java/fr/gouv/vitamui/archives/search/rest/ArchivesSearchController.java: 228 Attack Vector
LOW Log_Forging /ui/ui-archive-search/src/main/java/fr/gouv/vitamui/archives/search/rest/ArchivesSearchController.java: 311 Attack Vector
LOW Log_Forging /api/api-pastis/pastis-standalone/src/main/java/fr/gouv/vitamui/pastis/standalone/controller/PastisController.java: 178 Attack Vector

Fixed Issues

Severity Issue Source File / Package Checkmarx Insight
MEDIUM Privacy_Violation /api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/InternalSecurityService.java: 110 Attack Vector
MEDIUM Privacy_Violation /api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/InternalSecurityService.java: 118 Attack Vector
MEDIUM Privacy_Violation /api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/InternalSecurityService.java: 110 Attack Vector
MEDIUM Privacy_Violation /api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/InternalSecurityService.java: 118 Attack Vector
LOW Log_Forging /api/api-pastis/pastis-standalone/src/main/java/fr/gouv/vitamui/pastis/standalone/controller/PastisController.java: 178 Attack Vector

@Regzox Regzox force-pushed the story_11496_fix branch 2 times, most recently from 9fa87a2 to 05b0a60 Compare July 19, 2023 14:43
@Regzox Regzox changed the title story #11496 fix(archive-search): missing search for trackTotalHits story #11496 fix(archive-search): missing search for trackTotalHits + bug #11641 fix(ui-frontend-common): archive-unit-count component state Jul 19, 2023
@Regzox Regzox force-pushed the story_11496_fix branch 2 times, most recently from 02241e2 to 3fd7aab Compare August 24, 2023 09:39
@GiooDev GiooDev merged commit 24502b4 into develop Aug 28, 2023
@GiooDev GiooDev deleted the story_11496_fix branch August 28, 2023 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@laedanrex laedanrex laedanrex approved these changes

@soukami soukami soukami approved these changes

Assignees

@Regzox Regzox

Labels
bug Something isn't working javascript Pull requests that update Javascript code small pr embarquant peu de changements et à review rapide, ne nécessitant qu'un reviewer VAS VAS contribution
Projects
None yet
Milestone
IT 121
Development

Successfully merging this pull request may close these issues.
5 participants
@Regzox @TDevillechabrolle @laedanrex @soukami @GiooDev