search

wolfd / pwm

PWM is an open source password self service application for LDAP directories. - exported from code.google.com/p/pwm
http://pwmdemo.weisberg.net/
0 stars 1 forks source link

SMS does not work due to setting type mismatch #529

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The SMS country code does not seem to be evaluated properly, resulting in the 
SMS to not being sent.

The reason for that is a bug in SmsQueueManager.java in formatSmsNumber(). In 
revision 674 it's in line 305 which needs to be changed from:

final String configuredNumber =  
config.readSettingAsString(PwmSetting.SMS_DEFAULT_COUNTRY_CODE);

to:

final String configuredNumber =  
String.valueOf(config.readSettingAsLong(PwmSetting.SMS_DEFAULT_COUNTRY_CODE));

because SMS_DEFAULT_COUNTRY_CODE is configured as NUMERIC in PwmSetting.java:

SMS_DEFAULT_COUNTRY_CODE("sms.defaultCountryCode", PwmSettingSyntax.NUMERIC, 
Category.SMS),

What version of PWM are you using?
Revision 674, but the error also occurs in revision 679.

What ldap directory and version are you using?
Novell eDirectory 8.8.7

The error in log will be something like:
queue.AbstractQueueManager, EmailQueueManager setting value is not readable as 
string

Original issue reported on code.google.com by nils.rekow on 29 Jan 2014 at 3:36

GoogleCodeExporter commented 9 years ago 
Thanks for finding the bug. Please try again with the next daily build 
(available approx. 06:10 Europe/Amsterdam).

Original comment by menno.pi...@gmail.com on 29 Jan 2014 at 8:40