Projet

Général

Profil

Development #62336

retirer l'option « Utiliser l’expéditeur configuré : » dans le connecteur Orange SMS

Ajouté par Thomas Noël il y a environ 2 ans. Mis à jour il y a presque 2 ans.

Statut:
Information nécessaire
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
02 mars 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Dans le connecteur SMS Orange on a une case à cocher : « Utiliser l’expéditeur configuré »

Son help_text est « L’expéditeur doit d’abord être configuré dans le backoffice Orange » mais on ne sait pas si c'est vrai quand la case est cochée ou pas.

L'intitulé devrait également être éclairci, soit « Utiliser l’expéditeur configuré côté Orange » soit « Utiliser l’expéditeur reçu dans l'appel webservice »...?


J'ai regardé le code sans être bien sûr d'avoir compris dans le bon sens :

        if self.provide_sender:
            payload['smsParam']['senderName'] = sender

Pour moi ça veut dire que quand la case est cochée, c'est l'expéditeur choisi par Publik qui compte, sinon c'est l'expéditeur configuré côté Orange. C'est ça ?


Fichiers


Demandes liées

Lié à Passerelle - Development #56345: orange: SMS avec expéditeur personaliséFermé24 août 2021

Actions

Historique

#2

Mis à jour par Nicolas Roche il y a environ 2 ans

#3

Mis à jour par Nicolas Roche il y a environ 2 ans

On passe toujours l'expéditeur défini dans Publik.
edit:
On ne passe jamais d'autre expéditeur que celui défini dans Publik.

La dé-coche permet de ne pas passer d'expéditeur, ce qui était le comportement original, parce que
https://dev.entrouvert.org/issues/54792#note-6

Orange vérifiera par ailleurs si il est déclaré et autorisé de leur côté, ce qui est bien le cas chez nous, dans le cas contraire il renvoie une erreur 400 BadRequest SenderNameNotAllowed L'émetteur n'est pas autorisé par la configuration

#4

Mis à jour par Frédéric Péters il y a environ 2 ans

On passe toujours (...)
(...) permet de ne pas passer

Pas nécessairement clair.

#5

Mis à jour par Nicolas Roche il y a environ 2 ans

  • Statut changé de Nouveau à En cours

label : Utiliser l’expéditeur configuré
aide : L’expéditeur doit d’abord être configuré dans le backoffice Orange

Je propose ça :
label : Utiliser l’expéditeur configuré dans Publik
aide : Décocher la case pour utiliser l'expéditeur défini par défaut dans Orange

(mais on perd l'information que si l'expéditeur n'est pas reconnu par orange, alors les SMS tomberont en erreur)

#6

Mis à jour par Frédéric Péters il y a environ 2 ans

C'est pour moi inutilement confus; si je comprends bien l'utilité, c'est "on aurait configuré dans publik un expéditeur pas valable et dans ce cas on coche pour ne pas l'envoyer et orange prendra son expéditeur par défaut".

Volontiers je dégagerais. Et si l'expéditeur configuré dans publik n'est pas bon, ça sera visible dans les logs d'erreur.

#7

Mis à jour par Thomas Noël il y a environ 2 ans

  • Tracker changé de Support à Development
  • Sujet changé de précision sur l'option « Utiliser l’expéditeur configuré : » dans le connecteur Orange SMS à retirer l'option « Utiliser l’expéditeur configuré : » dans le connecteur Orange SMS

Frédéric Péters a écrit :

C'est pour moi inutilement confus (...)

Tout à fait d'accord : toujours prendre l'expéditeur demandé par Publik.

#8

Mis à jour par Nicolas Roche il y a environ 2 ans

La dé-coche permet de ne pas passer d'expéditeur, ce qui était le comportement original

Désolé, j'ai pas été clair.

L'idée de #56345 (case cochée) c'est de pouvoir passer l'expéditeur Publik, si et seulement si, on sait qu'il sera déclaré et autorisé côté Orange (Nanterre, https://dev.entrouvert.org/issues/54792#note-6)
Sinon (case dé-cochée), à priori il ne faut rien passer du tout pour que cela fonctionne (à Fondette et au CD44).

Si j'essaie d'intuiter vos demandes (même si je sais qu'il vaut mieux que j'évite), vous me suggéreriez de ne pas envoyer l'expéditeur s'il est défini à "vide" dans Hobo (ou l'expéditeur "vide" si je m'assure que cela fonctionne) ?

#9

Mis à jour par Thomas Noël il y a environ 2 ans

Dans https://hobo.../sms/ l'expéditeur ne peut pas être vide. Donc il faut envoyer "sender" dans payload['smsParam']['senderName'], toujours, et voilà.

#10

Mis à jour par Nicolas Roche il y a environ 2 ans

Je suis désolé, mais je ne suis vraiment pas du tout clair dans mes explications (j'essaye la version courte).

il ne faut rien passer du tout pour que cela fonctionne (à Fondette et au CD44).

#11

Mis à jour par Thomas Noël il y a environ 2 ans

Nicolas Roche a écrit :

Je suis désolé, mais je ne suis vraiment pas du tout clair dans mes explications (j'essaye la version courte).

il ne faut rien passer du tout pour que cela fonctionne (à Fondette et au CD44).

Est-ce que tu veux dire que si on envoie "FONDETTES" comme sender, alors ça bogue ? Est-ce que ça veut dire qu'on ne doit jamais envoyer de sender ?

#12

Mis à jour par Nicolas Roche il y a environ 2 ans

J'ai mal lu #62323, désolé.

Avec le compte configuré (sur les plateformes Recette et Production), un envoyeur personnalisé est paramétré chez Orange : FONDETTES

A présent Fondette se range dans la même catégorie que Nanterre où la case doit être cochée.

Est-ce que tu veux dire que si on envoie "FONDETTES" comme sender, alors ça bogue ?

Non, puisqu'il est défini chez orange.

Est-ce que ça veut dire qu'on ne doit jamais envoyer de sender ?

Non, il faut envoyer le sender si on veut que le SMS mentionne un expéditeur autre que "38975".
https://dev.entrouvert.org/issues/54792#note-9

Si je reformule ta question :

Est-ce que tu veux dire que si on envoie "le sender par défaut défini par Orange ("38975") comme sender, alors ça bogue ?

Je ne sais pas.
Comme nous n'hébergeons pas le CD44, nous ne pouvons pas tester l'envoie d'un SMS au CD44 avec "38975" comme sender.

Nouvelle proposition :
label : Utiliser l’expéditeur configuré dans Publik
aide : Si vous cochez cette case, vous devez vous assurer que l'expéditeur configuré dans Publik sera accepté par Orange

#13

Mis à jour par Thomas Noël il y a environ 2 ans

Nicolas Roche a écrit :

Si je reformule ta question :

Est-ce que tu veux dire que si on envoie "le sender par défaut défini par Orange ("38975") comme sender, alors ça bogue ?

Je ne sais pas.

Il faut tester l'API.

1. Prendre le connecteur vers une platforme Orange configurée avec expéditeur par défaut TRUC.
2. S'envoyer un SMS, via l'API Orange sans smsParam/senderName : vérifier qu'il arrive bien avec TRUC en expéditeur
3. S'envoyer un SMS, via l'API Orange, en posant smsParam/senderName = TRUC. Si le SMS arrive bien, c'est qu'on peut définitivement supprimer cette option.

Comme nous n'hébergeons pas le CD44, nous ne pouvons pas tester l'envoie d'un SMS au CD44 avec "38975" comme sender.

Mais on a FONDETTES.

Nouvelle proposition :
label : Utiliser l’expéditeur configuré dans Publik
aide : Si vous cochez cette case, vous devez vous assurer que l'expéditeur configuré dans Publik sera accepté par Orange

Si cette possibilité existe (configurer Orange pour accepter le sender), c'est bien qu'on peut supprimer l'option.

#14

Mis à jour par Nicolas Roche il y a environ 2 ans

Il faut tester l'API.

A toutes fin utiles, je rappelle que ce connecteur n'a pas été développé par nous, et que nous ne disposions pas de compte de test pour le tester (seuls 2 comptes utilisés en production sont configurés chez nos clients).

1. Prendre le connecteur vers une plateforme Orange configurée avec un expéditeur par défaut TRUC.

On n'a à ce jour jamais eu accès au backoffice Orange.
C'est pourquoi je dit que l'on n'a pas de plateforme Orange de configurée avec expéditeur par défaut sous la main.
("expéditeur par défaut TRUC" on ne sait même pas comment ça se présente, parce que d'après Benjamin "38975" serait engendré par une sorte de NAT)

Mais on a FONDETTES

Pas super chaud pour faire des tests à leur frais sans leur demander.
Nous n'avons pas non plus accès au backoffice d'Orange à Fondette, mais idem, pas chaud pour faire des tests sur leur compte de production (ils n'en n'ont qu'un).

Si cette possibilité existe (configurer Orange pour accepter le sender), c'est bien qu'on peut supprimer l'option.

Si on supprime cette option, ça entraînera à coup sûr une régession au CD44 d'une part,
et d'autre part la personne en charge de configurer une nouvelle instance du connecteur devra composer avec cette erreur :

400 BadRequest SenderNameNotAllowed
"smsParam":{
"senderName":"VILNANTERRE",
},

Puis, trouver dans la doc Contact-Everyone la phrase : "Seuls les numéros émetteurs personnalisés qui vous sont attribués sont utilisables dans l'API".
Aller retrouver l'expéditeur configuré en backoffice (s'il existe, on en sait toujours rien),
ou d'en ajouter un, puis de demander à orange d'aller vérifier s'il est déclaré et autorisé de leur côté.
Le mieux pour moi serait de demander au CD44 pourquoi ils n'ont pas prévu d'envoyer l'expéditeur dans le connecteur qu'ils nous ont livré.
(j'imagine que c'est parce qu'ils n'ont pas été au bout de la démarche ci-dessus effectuée à Nanterre)

Tout ça pour dire que vouloir simplifier oui, mais pour quel résultat ?

#15

Mis à jour par Frédéric Péters il y a environ 2 ans

Tout ça pour dire que vouloir simplifier oui, mais pour quel résultat ?

Pour quelque chose d'intelligible, où on n'est actuellement pas du tout.

De ce que j'ai compris,

 Utiliser l’expéditeur configuré dans le backoffice Orange
  [x]
 Note : si cette option est décochée, il faut que l’expéditeur configuré
 au niveau de Publik soit également configuré dans le backoffice d’Orange.
#16

Mis à jour par Nicolas Roche il y a environ 2 ans

C'est l'inverse mais sinon oui : si cette option est s/décochée/cochée/
Voir actuellement à Fondette et Nanterre où les SMS sont bien personnalisés : l’expéditeur configuré au niveau de Publik est également configuré dans le backoffice d’Orange.
Je propose d'ajouter (ou pas) une première phrase, si on veut être complet :

Note : si cette option est décochée, aucun expéditeur ne sera pas envoyé à
 Orange qui enverra un SMS avec l’expéditeur "38975".
 Si par contre cette option est cochée, il faut que l’expéditeur configuré
 au niveau de Publik soit également configuré dans le backoffice d’Orange.

#17

Mis à jour par Frédéric Péters il y a environ 2 ans

aucun expéditeur ne sera pas envoyé

On va éviter.

~~

C'est l'inverse mais sinon oui : si cette option est s/décochée/cochée/

Non non non.

#18

Mis à jour par Frédéric Péters il y a environ 2 ans

Le truc qui n'est pas clair visiblement dans ma proposition, c'est qu'il s'agit d'une proposition, pour rendre l'affaire intelligible. Et c'est un nouveau flag, qui inverse et remplace l'autre, pas juste renommer.

#19

Mis à jour par Nicolas Roche il y a environ 2 ans

Compris, merci.
(nouveau flag, qui inverse et remplace l'autre : je note pour moi qu'il faut prévoir une migration qui inverse la valeur en base, et coche la case par défaut)

#20

Mis à jour par Nicolas Roche il y a environ 2 ans

(j'ai laissé la migration inverse utilisée pour tester, mais je ne suis pas sûr que ce soit une bonne idée)

#21

Mis à jour par Thomas Noël il y a presque 2 ans

Pour moi, la question reste : est-ce qu'on peut utiliser un autre "sender" que celui configuré dans le backoffice Orange ? Si ce n'est pas le cas, alors inutile de se fatiguer à avoir un choix.

#22

Mis à jour par Benjamin Dauvergne il y a presque 2 ans

  • Statut changé de Solution proposée à Information nécessaire

Question de Thomas Noël de Massy.

#23

Mis à jour par Nicolas Roche il y a presque 2 ans

  • Assigné à Nicolas Roche supprimé

Je ne sais pas : possible qu'aucun "sender" ne soit défini dans la configuration de base, personne chez n'a eu accès au backoffice.

Formats disponibles : Atom PDF