Skip to content

Commit

Permalink
nathelper: add Max-Forwards headers which is mandatory for OPTIONS ac…
Browse files Browse the repository at this point in the history
…cording to RFC 3261 (GH 3657)

- add Max-Forwards headers which is mandatory for OPTIONS according to RFC 3261
- similar patch is used from several large customer installations since many years
- GH #3657 with small formatting change
  • Loading branch information
khoegh authored and henningw committed Nov 30, 2023
1 parent aef29af commit e1d9bec
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/modules/nathelper/sip_pinger.h
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ static inline char *build_sipping(str *curi, struct socket_info *s, str *path,
+ ruid->len + 1 + 8 + 1 + 8 + s_len(CRLF "To: ") + curi->len
+ s_len(CRLF "Call-ID: ") + sipping_callid.len + 1 + 8 + 1
+ 8 + 1 + s->address_str.len + s_len(CRLF "CSeq: 1 ")
+ sipping_method.len
+ sipping_method.len + s_len(CRLF "Max-Forwards: 70")
+ s_len(CRLF "Content-Length: 0" CRLF CRLF)
> MAX_SIPPING_SIZE) {
LM_ERR("len exceeds %d\n", MAX_SIPPING_SIZE);
Expand Down Expand Up @@ -205,6 +205,7 @@ static inline char *build_sipping(str *curi, struct socket_info *s, str *path,
append_str(p, s->address_str.s, s->address_str.len);
append_fix(p, CRLF "CSeq: 1 ");
append_str(p, sipping_method.s, sipping_method.len);
append_fix(p, CRLF "Max-Forwards: 70");
append_fix(p, CRLF "Content-Length: 0" CRLF CRLF);

*len_p = p - buf;
Expand Down

0 comments on commit e1d9bec

Please sign in to comment.