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.

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

Story #11868: api gateway ansible rework #1594

Merged
merged 43 commits into from
Dec 22, 2023

Conversation

Ro3034
Copy link
Contributor

@Ro3034 Ro3034 commented Dec 14, 2023

Description

Description des modifications

Ajout du déploiement de la gateway et des ui en mode gateway avec un mode activable/désactivable.

Ajout du build et packaging des ressources statiques utilisées par les UI.

Type de changement:

Indiquer le ou les types de changements
Ansiblerie
PKI
Scripts mongodb
Pom java et scripts de build.
Jenkinsfile de build

  • Build

  • PKI
    Rajout de l'extension serveur et de la génération des pem dans un cas supplémentaire.

  • Ansiblerie
    Rajout des roles asset_copy, nginx_webapp et ajout de variables et code pour activer/désactiver
    l'installation des ui en mode legacy ou gateway. Modification des templates pour avoir les 2 modes.

  • Nouveau Code

  • Correction

  • Refactorisation de code

  • Autre

Documentation:

Indiquer la documentation mise à jour

[ ] Quels sont les nouvelles documentations ?

[ ] Quels sont les modifications existantes ?

[ ] Quels sont les documentations ou sections de documentations supprimés ?

Tests:

Indiquer comment le code à été testé (manuel, environnement, TU, etc)

manuel

environnement

TU

Migration:

Indiquer si les modifications apportées impliquent une migration sur l'existant et comment la faire

Checklist:

Sélectionner les éléments de la checklist

[ ] Mon code suit le style de code de ce projet.

[ ] J'ai commenté mon code, en particulier dans les classes et les méthodes difficile à comprendre.

[ ] J'ai fait les changements correspondant dans la documentation RAML.

[ ] J'ai fait les changements correspondant dans la documentation Métier.

[ ] J'ai fait les changements correspondant dans la documentation Technique.

[ ] J'ai rajouté les tests unitaires vérifiant mes fonctionnalités.

[ ] J'ai rajouté les tests de non régression vérifiant mes fonctionnalités.

[ ] Les tests unitaires nouveaux et existants passent avec succès localement.

[ ] Toutes les dépendances ont été mergées en priorité

Contributeur

Indiquer qui a développé cette fonctionnalité

VAS (Vitam Accessible en Service)

CEA (Commissariat à l'énergie atomique et aux énergies alternatives)

@Ro3034 Ro3034 requested a review from GiooDev December 14, 2023 09:15
@Ro3034 Ro3034 self-assigned this Dec 14, 2023
@GiooDev GiooDev added this to the IT 129 milestone Dec 14, 2023
@vitam-devops
Copy link
Collaborator

vitam-devops commented Dec 14, 2023

Logo
Checkmarx One – Scan Summary & Details575ca884-cd4c-408b-a947-4b24a5f38877

New Issues

Severity Issue Source File / Package Checkmarx Insight
MEDIUM Unchecked_Input_for_Loop_Condition /api/api-iam/iam-external/src/main/java/fr/gouv/vitamui/iam/external/server/rest/ExternalParamProfileExternalController.java: 186 Attack Vector
LOW Log_Forging /api/api-iam/iam-internal/src/main/java/fr/gouv/vitamui/iam/internal/server/rest/ProfileInternalController.java: 168 Attack Vector
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 193 Attack Vector
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 193 Attack Vector
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 193 Attack Vector
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 193 Attack Vector
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 193 Attack Vector

Fixed Issues

Severity Issue Source File / Package
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 228
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 228
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 228
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 228
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 228
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 228
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 228
LOW Log_Forging /api/api-referential/referential-external/src/main/java/fr/gouv/vitamui/referential/external/server/rest/RuleExternalController.java: 228

@GiooDev GiooDev changed the title Story 11868 api gateway ansible rework Story #11868: api gateway ansible rework Dec 14, 2023
@Ro3034 Ro3034 force-pushed the story_11868_api_gateway_ansible_rework branch 2 times, most recently from a981838 to 7b2f2a0 Compare December 14, 2023 17:30
@Ro3034 Ro3034 force-pushed the story_11868_api_gateway_ansible_rework branch 21 times, most recently from 8468342 to a6ca02a Compare December 15, 2023 00:05
@Ro3034 Ro3034 force-pushed the story_11868_api_gateway_ansible_rework branch from 026453a to 09449f6 Compare December 19, 2023 15:57
@Ro3034 Ro3034 force-pushed the story_11868_api_gateway_ansible_rework branch from 09449f6 to 3fad40c Compare December 19, 2023 16:35
@Ro3034 Ro3034 force-pushed the story_11868_api_gateway_ansible_rework branch from 3fad40c to 544b90d Compare December 19, 2023 16:41
@Ro3034 Ro3034 force-pushed the story_11868_api_gateway_ansible_rework branch from a895245 to 9a5bb90 Compare December 19, 2023 20:36
@Ro3034 Ro3034 force-pushed the story_11868_api_gateway_ansible_rework branch from 9a5bb90 to 28f928b Compare December 19, 2023 21:27
@Ro3034 Ro3034 force-pushed the story_11868_api_gateway_ansible_rework branch from 72eac86 to 4d921cb Compare December 21, 2023 12:48
@GiooDev
Copy link
Contributor

GiooDev commented Dec 21, 2023

Ne pas oublier de Squash and merge plz !

Merci pour le super boulot :)

@GiooDev GiooDev merged commit 0cd8a72 into develop Dec 22, 2023
2 checks passed
@GiooDev GiooDev deleted the story_11868_api_gateway_ansible_rework branch December 22, 2023 13:20
@GiooDev GiooDev added the OPS REVIEW Mandatory if deployment/ directory is modified. label Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OPS REVIEW Mandatory if deployment/ directory is modified.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants