On 12/17/09 9:58 PM, Paul Mossman wrote:
> 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.
>
Sorry. Completely missed this issue. If it causes too much problems
change XmlModelBuilder to use HostnameSetting for 'ipaddr' tag until
it's fixed.
>
>> 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/
