Bug #16747: Logbook : reports download issue by hazco75 · Pull Request #3922 · ProgrammeVitam/vitam-ui

hazco75 · 2026-08-17T12:41:43Z

Description

Problèmes de téléchargement des rapports pour l'import de griffons et l'import de scénarios de préservation

Summary by CodeRabbit

New Features
- Added support for downloading reports associated with STP Griffin imports.
- Added support for downloading reports associated with STP preservation scenario imports.
- These event types are now recognized in master data downloads.

coderabbitai · 2026-08-17T12:42:08Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: c5b2ae22-86bd-462e-b73e-05064d95e296

📥 Commits

Reviewing files that changed from the base of the PR and between d5c0c96 and a430aa1.

📒 Files selected for processing (1)

ui/ui-frontend/projects/referential/src/app/logbook-operation/logbook-download.service.ts

Included review availability: Your plan includes up to 3 reviews per rolling hour; 2 remain after this review.

📝 Walkthrough

Walkthrough

The logbook download service now permits STP_IMPORT_PRESERVATION_SCENARIO and maps STP_IMPORT_GRIFFIN and STP_IMPORT_PRESERVATION_SCENARIO masterdata events to report downloads.

Changes

Logbook report download support

Layer / File(s)	Summary
Import event type download mapping `ui/ui-frontend/projects/referential/src/app/logbook-operation/logbook-download.service.ts`	The service allows `STP_IMPORT_PRESERVATION_SCENARIO` and returns the report download type for `STP_IMPORT_GRIFFIN` and `STP_IMPORT_PRESERVATION_SCENARIO` masterdata events.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to a430a

This localized change addresses report downloads in the logbook area, and no actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description states the affected report downloads but omits the required change type, documentation, tests, migration, checklist, and contributor sections.	Complete the template with the change type, documentation impact, tests performed, migration status, checklist, and contributor.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the logbook report download bug and matches the main change.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch bug_16747_logbook

_{Comment @coderabbitai help to get the list of available commands.}

vitam-prg · 2026-08-17T12:47:32Z

Checkmarx One – Scan Summary & Details – e2e0b131-1084-44dd-b3ed-ae398d767fca

New Issues (13)

Checkmarx found the following issues in this Pull Request

#	Issue	Source File / Package	Checkmarx Insight
1	CVE-2026-54428	Maven-org.apache.httpcomponents.core5:httpcore5-h2-5.2.4	details Recommended version: 5.4.3 Description: Allocation of resources without limits or throttling in the HTTP/2 HPACK decoder in Apache HttpComponents Core through 5.4.2 and 5.5.x through 5.5-... Attack Vector: NETWORK Attack Complexity: LOW Vulnerable Package
2	CVE-2026-54428	Maven-org.apache.httpcomponents.core5:httpcore5-h2-5.3.6	details Recommended version: 5.4.3 Description: Allocation of resources without limits or throttling in the HTTP/2 HPACK decoder in Apache HttpComponents Core through 5.4.2 and 5.5.x through 5.5-... Attack Vector: NETWORK Attack Complexity: LOW Vulnerable Package
3	CVE-2026-19880	Maven-ch.qos.logback:logback-classic-1.4.14	details Recommended version: 1.5.16.redhat-00001 Description: Path-traversal vulnerability in QOS.CH Sarl Logback-classic on Java (logback-classic module) allows path-traversal vulnerability. More specifically... Attack Vector: NETWORK Attack Complexity: LOW Vulnerable Package
4	CVE-2026-19880	Maven-ch.qos.logback:logback-classic-1.5.32	details Recommended version: 1.6.3 Description: Path-traversal vulnerability in QOS.CH Sarl Logback-classic on Java (logback-classic module) allows path-traversal vulnerability. More specifically... Attack Vector: NETWORK Attack Complexity: LOW Vulnerable Package
5	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 239	details Method getUser at line 239 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java gets user input from element embe... Attack Vector
6	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java: 130	details Method create at line 130 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java gets user input from element dt... Attack Vector
7	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 301	details Method logout at line 301 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java gets user input from element authT... Attack Vector
8	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 197	details Method changePassword at line 197 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java gets user input from eleme... Attack Vector
9	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java: 130	details Method create at line 130 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java gets user input from element dt... Attack Vector
10	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 301	details Method logout at line 301 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java gets user input from element authT... Attack Vector
11	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java: 130	details Method create at line 130 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java gets user input from element dt... Attack Vector
12	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 221	details Method getUsersByEmail at line 221 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java gets user input from elem... Attack Vector
13	Parameter_Tampering	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java: 130	details Method create at line 130 of /api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java gets user input from element dt... Attack Vector

Fixed Issues (395)

Great job! The following issues were fixed in this Pull Request

Severity	Issue	Source File / Package
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 85
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 85
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 85
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 175
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 175
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 117
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 85
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 85
	~~Privacy_Violation~~	api/api-iam/iam-security/src/main/java/fr/gouv/vitamui/iam/security/service/SecurityService.java: 85
	~~Log_Forging~~	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/CasController.java: 266
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/ProjectController.java: 216
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/ProjectController.java: 217
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/ProjectController.java: 216
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/ProjectController.java: 216
	~~Log_Forging~~	api/api-referential/referential/src/main/java/fr/gouv/vitamui/referential/server/rest/AccessContractController.java: 271
	~~Log_Forging~~	api/api-referential/referential/src/main/java/fr/gouv/vitamui/referential/server/rest/IngestContractController.java: 231
	~~Log_Forging~~	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/TenantController.java: 130
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 459
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 467
	~~Log_Forging~~	api/api-referential/referential/src/main/java/fr/gouv/vitamui/referential/server/rest/SchemaUnitController.java: 70
	~~Log_Forging~~	api/api-referential/referential/src/main/java/fr/gouv/vitamui/referential/server/rest/SchemaUnitController.java: 58
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/TransactionArchiveUnitController.java: 163
	~~Log_Forging~~	api/api-referential/referential/src/main/java/fr/gouv/vitamui/referential/server/rest/IngestContractController.java: 265
	~~Log_Forging~~	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/UserController.java: 148
	~~Log_Forging~~	api/api-iam/iam/src/main/java/fr/gouv/vitamui/iam/server/rest/UserController.java: 148
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 217
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/TransactionController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 217
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 179
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/TransactionController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 217
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 180
	~~Log_Forging~~	api/api-collect/collect/src/main/java/fr/gouv/vitamui/collect/server/rest/ProjectObjectGroupController.java: 151
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 268
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 179
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 248
	~~Log_Forging~~	api/api-archive-search/archive-search/src/main/java/fr/gouv/vitamui/archives/search/server/rest/ArchivesSearchController.java: 287

More results are available on the CxOne platform

Use @Checkmarx to interact with Checkmarx PR Assistant.
Examples:
@Checkmarx how are you able to help me?
@Checkmarx rescan this PR

github-actions · 2026-09-25T14:25:18Z

Successfully created backport PR for master_8.1.x:

CP V8.1 - Bug #16747: Logbook : reports download issue #4032

github-actions · 2026-09-25T14:25:18Z

Successfully created backport PR for master_9.1.x:

CP V9.1 - Bug #16747: Logbook : reports download issue #4031

github-actions · 2026-09-25T14:25:51Z

Successfully created backport PR for master_9.0.x:

CP V9.0 - Bug #16747: Logbook : reports download issue #4033

Bug #16747: Logbook : reports download issue

a430aa1

hazco75 added this to the IT 174 milestone Aug 17, 2026

hazco75 added bug Something isn't working small pr embarquant peu de changements et à review rapide, ne nécessitant qu'un reviewer labels Aug 17, 2026

bbenaissa approved these changes Aug 17, 2026

View reviewed changes

Salimdev approved these changes Aug 18, 2026

View reviewed changes

hazco75 merged commit fff7355 into develop Aug 18, 2026
21 checks passed

hazco75 deleted the bug_16747_logbook branch August 18, 2026 07:37

This was referenced Sep 25, 2026

CP V8.1 - Bug #16747: Logbook : reports download issue #4032

Open

CP V9.1 - Bug #16747: Logbook : reports download issue #4031

Open

CP V9.0 - Bug #16747: Logbook : reports download issue #4033

Open

Navigation Menu

Bug #16747: Logbook : reports download issue - #3922

Bug #16747: Logbook : reports download issue#3922
hazco75 merged 1 commit into
developfrom
bug_16747_logbook

hazco75 commented Aug 17, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Aug 17, 2026 •

edited

Loading