Damian wrote:
> Based on Ernest patch for XX-7180 I added 2 new settings 
> types: ipaddr and hostname. They are just simple, specialized 
> versions of a StringSetting constrained with a regular 
> expression. Strickly speaking we accepted 'ipaddr' as a 
> setting type for quite some time now, but it was not mapped 
> to anything useful. 'hostname' type accepts IPv4 addresses as 
> well ('ipaddr' does not accept hostnames).
> 
> http://code.sipfoundry.org/changelog/sipXecs/?cs=17550
> 
> 
> When you have a chance please review your phone plug-in or 
> service and start using those where it makes sense.

The 'ipaddr' setting type was used throughout plugins for setting where
it's actually a hostname that can be accepted.  So, I think we've got a
lot of plugins that now have problems.

I've scheduled http://track.sipfoundry.org/browse/XX-7233 for 4.1.6, and
increased the priority.  It was a superset of this recent change, but
also covers the plugin updates.


> The error message associated with those is for now a generic Tapestry
> message. That can be improved upon if StringSetting is allowed to 
> provide customized error message keys.

We will want to improve it.  The current message is: "IMAP server host
does not fulfill the required pattern
((([A-Za-z0-9]([-A-Za-z0-9]*[A-Za-z0-9])?)\.){1,}[A-Za-z]{2,}|localhost)
|(25[0-5]|2[0-4][0-9]|[01]?[0-9]{1,2})(\.(25[0-5]|2[0-4][0-9]|[01]?[0-9]
{1,2})){3}."

I'll raise a JIRA for that.


-Paul
[email protected]

_______________________________________________
sipx-dev mailing list [email protected]
List Archive: http://list.sipfoundry.org/archive/sipx-dev
Unsubscribe: http://list.sipfoundry.org/mailman/listinfo/sipx-dev
sipXecs IP PBX -- http://www.sipfoundry.org/

Reply via email to