I split the pull request in three commits to seperate the fix from the new feature, but they go together so I propose them in the same pull request:

cc0b07d: I added some log message that would have helped me to understand a 400 Bad Request response.
c0697d7: I fixed a bug that were the contact size is not checked on the first header field and 200 OK is sent instead of 400 Bad Request for too long contact URIs but the uri ends up beein truncated by the database.
eb6b9c6: I made the contact size configurable to allow longer contacts

You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/725

Commit Summary

• registrar: Add warnings if contact is invalid and REGISTER will be rejected
• registrar: Check max URI size of contact also for first contact header
• registrar: Add module parameter "contact_max_size" to make max contact size configurable

File Changes

• M modules/registrar/doc/registrar_admin.xml (30)
• M modules/registrar/registrar.c (3)
• M modules/registrar/registrar.h (2)
• M modules/registrar/sip_msg.c (19)

Patch Links:

• https://github.com/kamailio/kamailio/pull/725.patch
• https://github.com/kamailio/kamailio/pull/725.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.