Project

General

Profile

Bug #30409

montpellier-sig crash sur un name.strip() => 'NoneType' object has no attribute 'strip'

Added by Thomas Noël 3 months ago. Updated about 1 month ago.

Status:
Résolu (à déployer)
Priority:
Normal
Assignee:
Target version:
-
Start date:
05 Feb 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Internal Server Error: /montpellier-sig/hebe/reverse
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/django/core/handlers/base.py", line 132, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/usr/lib/python2.7/dist-packages/passerelle/urls_utils.py", line 54, in f
return func(request, *args, **kwargs)
File "/usr/lib/python2.7/dist-packages/django/views/generic/base.py", line 71, in view
return self.dispatch(request, *args, **kwargs)
File "/usr/lib/python2.7/dist-packages/django/views/generic/base.py", line 89, in dispatch
return handler(request, *args, **kwargs)
File "/usr/lib/python2.7/dist-packages/passerelle_montpellier_sig/views.py", line 221, in get
house_number, road = split_street(result.get('voie'))
File "/usr/lib/python2.7/dist-packages/passerelle_montpellier_sig/views.py", line 69, in split_street
name = name.strip()
AttributeError: 'NoneType' object has no attribute 'strip'

0001-do-not-crash-on-empty-name-30409.patch View (1.17 KB) Thomas Noël, 05 Feb 2019 04:46 PM

History

#1 Updated by Thomas Noël 3 months ago

patch pour passerelle-montpellier-sig

#2 Updated by Frédéric Péters 2 months ago

  • Status changed from Solution proposée to Solution validée

Go, sans chercher la situation qui a fait renvoyer n'importe quoi au service.

#3 Updated by Thomas Noël 2 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 5198c0a9bf287166f045b15433620fdea3ed5be0
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Tue Feb 5 16:44:37 2019 +0100

    do not crash on empty name (#30409)

#4 Updated by Benjamin Dauvergne about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée
  • Assignee set to Thomas Noël

#5 Updated by Frédéric Péters about 1 month ago

  • Status changed from Solution déployée to Résolu (à déployer)

Ça n'a pas été déployé.

Also available in: Atom PDF