[OpenSIPS-Devel] [OpenSIPS/opensips] 3b6685: digest+m_auth: Allow a domain in the digest userna...

Bogdan Andrei IANCU bogdan at opensips.org
Wed Feb 5 18:20:37 CET 2014


  Branch: refs/heads/master
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 3b6685542b780ec47042a0e376de108e6d9c5fb7
      https://github.com/OpenSIPS/opensips/commit/3b6685542b780ec47042a0e376de108e6d9c5fb7
  Author: Walter Doekes <walter+github at wjd.nu>
  Date:   2014-02-05 (Wed, 05 Feb 2014)

  Changed paths:
    M modules/auth/auth_mod.c
    M parser/digest/digest.c

  Log Message:
  -----------
  digest+m_auth: Allow a domain in the digest username.

Before this patch, a domain in the digest username was allowed:

    Digest username="abc at domain", realm="domain"

but only if the domain is equal to the realm. This was introduced in
a92bf789 (Juha Heinanen, Mar. 29 2006).

The SIP spec. doesn't mention any such restriction.

This patch undoes that commit and makes sure the whole username is used
when authenticating through `pv_www_authorize` and `pv_proxy_authorize`.
That makes the following valid:

    Digest username="abc at domain", realm="something different"


  Commit: 79491755798f588db9b65ecdb63c19aa4eb3b834
      https://github.com/OpenSIPS/opensips/commit/79491755798f588db9b65ecdb63c19aa4eb3b834
  Author: Bogdan Andrei IANCU <bogdan at opensips.org>
  Date:   2014-02-05 (Wed, 05 Feb 2014)

  Changed paths:
    M modules/auth/auth_mod.c
    M parser/digest/digest.c

  Log Message:
  -----------
  Merge pull request #164 from wdoekes/wjd-allow_domain_in_digest_user

digest+m_auth: Allow a domain in the digest username.


Compare: https://github.com/OpenSIPS/opensips/compare/11db9b80f5a1...79491755798f


More information about the Devel mailing list