View Issue Details

IDProjectCategoryView StatusLast Update
0011471Talerdeployment and operationspublic2026-06-04 16:42
Reportervecirex Assigned Tovecirex  
PriorityurgentSeveritymajorReproducibilityhave not tried
Status closedResolutionfixed 
Summary0011471: mytops: test phone numbers prod config accepts
DescriptionApparently phone no. for fixed lines still get accepted by account creation... shouldn't be allowed by merchant's regex for phone no.
TagsNo tags attached.

Activities

vecirex

2026-06-04 16:04

manager   ~0028780

Last edited: 2026-06-04 16:09

Ehm, yeah, PHONE_REGEX is too liberal and just accepts any kind of number which looks like a Swiss no., purely formwise -- my fault closing 0011105 like this:

cat /etc/taler-merchant/merchant-overrides.conf | grep -A 1 11105
# cf. #​11105, enable regex checks in phone no.
PHONE_REGEX = ^\+41 ?[1-9][0-9]( ?[0-9]{3})( ?[0-9]{2})( ?[0-9]{2})$


Fix underway...

vecirex

2026-06-04 16:31

manager   ~0028781

Last edited: 2026-06-04 16:32

Regex tested to work in stage, only allowed mobile phone numbers starting w/ +417X, X from [5,9]:

# cf. #​11105, enable regex checks in phone no.
# cf. #​11471, revised and fixed
PHONE_REGEX = ^\+41 ?7[5-9]( ?[0-9]{3})( ?[0-9]{2})( ?[0-9]{2})$

vecirex

2026-06-04 16:42

manager   ~0028782

Fixed also in prod.

Issue History

Date Modified Username Field Change
2026-06-04 15:57 vecirex New Issue
2026-06-04 15:58 vecirex Assigned To => vecirex
2026-06-04 15:58 vecirex Status new => assigned
2026-06-04 16:04 vecirex Note Added: 0028780
2026-06-04 16:09 vecirex Note Edited: 0028780
2026-06-04 16:31 vecirex Note Added: 0028781
2026-06-04 16:32 vecirex Note Edited: 0028781
2026-06-04 16:42 vecirex Note Added: 0028782
2026-06-04 16:42 vecirex Status assigned => closed
2026-06-04 16:42 vecirex Resolution open => fixed