[asterisk-dev] [Code Review] Parsing of reg->username containing @domain

[Nick Lewis]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/628/
-----------------------------------------------------------

(Updated 2010-05-12 09:48:16.229570)


Review request for Asterisk Developers and David Vossel.


Changes
-------

Addressed issues raised by dvossel. Also fixed bug in parser when secret and authuser are present but empty 


Summary
-------

reg->username is parsed for each registration refresh rather than once on sip reload. A previously committed change that addressed this was reverted because it did not cater for registration strings that contained a peer name in the host field. This change does. 


This addresses bug 14331.
    https://issues.asterisk.org/view.php?id=14331


Diffs (updated)
-----

  trunk/main/app.c 258227 
  trunk/channels/sip/include/sip.h 258228 
  trunk/channels/sip/config_parser.c 258227 
  trunk/channels/chan_sip.c 258227 

Diff: https://reviewboard.asterisk.org/r/628/diff


Testing
-------

Passes a new unit test for parsing usernames containing domains. Passes existing unit tests. Passes peername functional test with sip.conf snippet

[general]
register=nick at quirk
[quirk]
fromdomain=10.10.235.48


Thanks,

Nick