Projet

Général

Profil

Bug #61813

ValueError : invalid literal for int() with base 10: '00+336********'

Ajouté par Corentin Séchet il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
15 février 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Exception quand on entre un numéro avec un indicatif en "+nn" dans la fenêtre de test d'envoi d'un connecteur SMS.

ValueError at /manage/sfr-dmc/Test/test-send/

invalid literal for int() with base 10: '00+336********'

Request Method: POST
Request URL: https://passerelle.dev.publik.love/manage/sfr-dmc/Test/test-send/
Django Version: 2.2.24
Exception Type: ValueError
Exception Value:

invalid literal for int() with base 10: '00+336********'

Exception Location: /home/csechet/src/passerelle/passerelle/sms/models.py in authorize_numbers, line 180
Python Executable: /home/csechet/envs/publik-env-py3/bin/python
Python Version: 3.9.10
Python Path:


Fichiers


Demandes liées

Lié à Passerelle - Support #61862: Les indicatifs téléphoniques internationnaux sont limités à 3 chiffresNouveau16 février 2022

Actions

Révisions associées

Révision 4baad974 (diff)
Ajouté par Nicolas Roche il y a environ 2 ans

sms: ensure country code and prefix are numbers (#61813)

Historique

#1

Mis à jour par Corentin Séchet il y a environ 2 ans

Au temps pour moi : c'est quand on entre un numéro sans l'indicatif national que l'exception est levée (06********).

#2

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

Je n'arrive pas à reproduire (je n'arrive pas à obtenir autre chose que des entiers à traiter dans authorize_numbers ).

#3

Mis à jour par Corentin Séchet il y a environ 2 ans

Effectivement, j'avais entré "+33" dans le préfixe de pays par défaut et pas "33", mais ça peut être un peu déroutant.

#4

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

Compris, merci.

Je corrige en amont, parce qu'à priori on ne devrait pas envoyer autre chose que des numéros.

#5

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

  • Lié à Support #61862: Les indicatifs téléphoniques internationnaux sont limités à 3 chiffres ajouté
#6

Mis à jour par Corentin Séchet il y a environ 2 ans

  • Statut changé de Solution proposée à Solution validée
#7

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

  • Statut changé de Solution validée à Résolu (à déployer)
  • Assigné à mis à Nicolas Roche
commit 4baad9742b4eb720ed0fc6a53755c4f6d911e299
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Wed Feb 16 09:15:40 2022 +0100

    sms: ensure country code and prefix are numbers (#61813)
#8

Mis à jour par Transition automatique il y a environ 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#9

Mis à jour par Transition automatique il y a environ 2 ans

Automatic expiration

Formats disponibles : Atom PDF