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

[BO - Interconnexion] Empêcher tout appel vers des api partenaire depuis localhost #3543

Merged
merged 4 commits into from
Jan 10, 2025

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Jan 9, 2025

Ticket

#3542

Description

Empêcher toute appel vers api externe depuis localhost

  • SISH
  • SCHS
  • Oilhi
  • Idoss

Changements apportés

  • Exception levée sur JobEventHttpClient
  • Exception levée sur IdossService

Pré-requis

  • Requête SQL
--- IDOSS & SCHS
UPDATE `partner` 
SET 
    `esabora_url` = 'https://example.com/ws/rest', 
    `idoss_url` = 'https://example.com/idoss/' 
WHERE `partner`.`id` = 6;

--- SI-SH
UPDATE `partner` SET `esabora_url` = 'https://example.com/ws/rest' WHERE `partner`.`id` = 7;
  • Mise à jour adresse
    Editer un signalement du pas de calais avec un code insee autorisé (62091) du 62 (ex: 1 Rue de Sus Saint-Léger, 62810 Beaudricourt)

  • Redémarrage des services
    make mock-stop && make mock-start
    make worker-stop && make worker-consume

Tests



  • [Oilhi] Affecter le partenaire Partenaire 62-02 (BEAUDRICOURT) et vérifier que l'exception est bien levée dans la console du worker

TNR

  • Faire une affection sur le partenaire 13-05 SCHS et 13-06 ARS et accepter le en tant que user-13-05, vérifier que tout est ok dans la table job_event

@sfinx13 sfinx13 changed the title Empêcher toute appel vers api externe depuis localhost Empêcher toute appel vers des api externe depuis localhost Jan 9, 2025
@sfinx13 sfinx13 changed the title Empêcher toute appel vers des api externe depuis localhost Empêcher toute appel vers des api partenaire depuis localhost Jan 10, 2025
.env Show resolved Hide resolved
@sfinx13 sfinx13 marked this pull request as ready for review January 10, 2025 09:46
@sfinx13 sfinx13 changed the title Empêcher toute appel vers des api partenaire depuis localhost [BO - Interconnexion] Empêcher toute appel vers des api partenaire depuis localhost Jan 10, 2025
@emilschn emilschn changed the title [BO - Interconnexion] Empêcher toute appel vers des api partenaire depuis localhost [BO - Interconnexion] Empêcher tout appel vers des api partenaire depuis localhost Jan 10, 2025
Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK d'après ce que j'ai vu

@emilschn emilschn merged commit c7677f2 into develop Jan 10, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants