The rtjson module is modifying the "from" header twice and the "to" header keeps unchanged.
`[root@dwrfsd01 kamailio]# git diff
diff --git a/modules/rtjson/rtjson_routing.c b/modules/rtjson/rtjson_routing.c
index 5542e63..60bdcad 100644
--- a/modules/rtjson/rtjson_routing.c
+++ b/modules/rtjson/rtjson_routing.c
@@ -481,7 +481,7 @@ int rtjson_prepare_branch(sip_msg_t *msg, srjson_doc_t *jdoc, srjson_t *nj)
xuri.len = strlen(xuri.s);
}
if(xdsp.len>0 || xuri.len>0) {
`
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/867
Commit Summary
- pkg/rpm: update spec files for v4.4.2
- pkg/kamailio: version updates in some spec files
- Makefile.defs: version set to 4.4.2
- ChangeLog: content updated for v4.4.2
- dmq Correct README
- modules: readme files regenerated - dmq ...
- topos: note in docs that masking callid is not implemented yet
- db_unixodbc: detect DB disconnection with generic HY000 status
- core: fix Sun Studio build
- counters: fix return statement in void function
- dialog: remove return statement from void functions
- ctl: remove return statement from void function
- uid_avp_db: remove GCC specific -Wall
- core: ensure cfg_block structure is 8 byte aligned
- modules: readme files regenerated - topos ...
- userblacklist: fix typo on evaluating the table parameter
- userblacklist: more debug info when failing to do the query
- userblacklist: fixed copy-paste typo when evaluating table parameter
- rr: fix spelling errors catched by lintian #688
- rr: refactored the check route params to detect properly if same message is used
- dialog: open db connection from rpc processes
- tm: check branch uri against current uri when deciding if append a new branch
- rtpengine: init rtp engine sockets array after allocation
- rtpproxy: init rtpproxy sockets array after allocation
- tm: execut cfg_update() inside t_continue()
- janssonrpc-c: init cfg framework for additional process
- kazoo: init cfg framework in child processes
- http_async_client: don't stop routing script execution for a POST query if transaction is not suspended
- http_async_client: init PV API sooner
- Makefile.rules: exclude test folder from tags generation paths
- core: structure to pass cfg env attributes between components
- dialog: pass info on cseq update tracking via cfg environment
- uac: add cookie header for cseq tracking
- core: helper functions for header management with strz values
- dialog: use cookie header to detect cseq auth increment needs
- db_mongodb: fix warning for bool comparison
- pv: return 5061 for port variables (e.g., $dp) when transport is tls
- rtpproxy: comment about child init rank filtering
- core: fix regex error for modules parameters
- modules/ipops: fix subnet matching for mask /0
- tls: test if ssl crypto locking callback is already set
- core/mem: qm - test and print status for chunks start/end markers for mem status
- core: mem - qm: coherent indentation and whitespacing
- core: mem_status_all - added to cfg core structure
- core: mem/qm - test cfg core mem_status_all to print memory status
- core: fix offset to in hdr add helper added recently
- dialog: buffer-based remove of custome header tracking cseq updates
- core: added mem_status_mode parameter
- dialog: parse all headers to check if cseq increment is needed
- avpops: note that avp_copy() is reversing the order of the values
- core: helper functions to add headers with str parameters
- core: srapi - cseq update field update to be a function
- core: add header helper uses after lump anchor insert
- uac: use cseq update callback exported by dialog via core srapi
- dialog: cseq update callback implemented inside the module
- tm: wrap uri values in brackets inside error messages
- tls: more detailed log message when tls module cannot set libssl memory manager
- tls: coherent whitespacing and indentation
- tls: added note in docs about loading tls module first
- tls: print ssl memory functions if they cannot be set
- topos: skip options from contact header check
- presence: fix potential double free in shutdown routine
- presence: Always check if a record exists for this dialog before inserting
- core: parse rr - removed function name from logs
- core: print content when failing to parse rr body
- presence: fix memory leak introduced by last commit
- registrar: Add warnings if contact is invalid and REGISTER will be rejected
- registrar: Check max URI size of contact also for first contact header
- presence: log when presentity is deleted due to already existing
- presentity: don't update terminated presentity entries in the database
- uac: uac_reg: fix uac_reg_get_byuser()
- http_async_client: fixed small typo in docs
- core: msg_parser log output adaptations
- dialog: jump to error if fails to parse requests in cseq update callback
- tm: Set tcx.branch_index to picked branch before running failure route.
- rtimer: Validate mode parameter
- dispatcher: free memory on shutdown
- pv: xavp: log error message when memory allocation fails
- core: Enable SIGUSR1 for child processes
- mqueu: Convert items to single linked list
- tmx: Try to get branch_index for all replies
- jsonrpc_s: Return an error when JSON parsing fails
- core: Don't bind to IPv6 addresses if auto_bind_ipv6 is disabled
- topos: skip comma at the end when reinserting rr headers
- sl: Reset timeout before sending the message
- presence: fix crash when field value is null
- dialog: set DLG_FLAG_CHANGED for changed dialog variables
- topos: force checking of Contact header for INVITE only
- core: fix some logging in tcp_read.c
- outbound: use pkg-config for libcrypto in makefile
- presence: fix multiple double free on shutdown caught with memlog=1
- core: fix double free on clearing hdr
- core: fix double free on clearing lump
- core: parser - safety check and clean log messages for disposition header
- dialog: refresh cseq update if it was incremented in the past
- topos: record routing storage for handling sip responses
- dialog: proper safety test if cseq value in via is larger than hdr
- dialog: proper refresh of cseq for requests withing dialog after an increment
- core: handle return code for tcp read headers/ws before printing extra debug logs
- core: tcp read - proper safety when priting log messages for read error case
- topos: error message about missing uuid when loading dlg made dbg
- path: added section ids in docs
- topos: store record route set only for initial 200ok response
- jansson: test if pkg-config knows about jansson library
- dialog: ensure all headers are parsed to find the custom ones for cseq refresh
- topos: more appropriate error log when attepting to delete header
- tcpops: add documentation for tcp:closed
- core: macro helper to trim zeros from str value
- topos: trim values before reinserting the route header
- topos: proper check if the last char in headr value is eol
- http_client: added semicolons in the examples
- core: try to catch dash immediately after ID in define/ifdef
- uac: updated the docs for uac registration refresh rpc command
- mohqueue: prevent calls sticking in queue
- utils Add missing semi-colons to documentation
- modules: readme files regenerated - avpops ...
- evapi: fixed the communication with dispatcher after initializing it as rpc worker
- jsonrpc-c: set the cpm pipe for all child processes
- janssonrpc-c: set the cmd pipe for all child processes
- db_mongodb: elaborated the docs for usage with tls module
- ndb_mongodb: elaborated the docs for usage with tls module
- modules: readme files regenerated - db_mongodb ...
- pkg/rpm: spec files updated for v4.4.3
- textops: more details for set_body_multipart() and need of msg_apply_changes()
- modules: readme files regenerated - textops ...
- pkg/kamailio/deb: set version to 4.4.3
- Makefile.defs: version set to 4.4.3
- ChangeLog: content updated for v4.4.3
- presence: test if event is dialog for checking terminated state in presentity update
- presence: free db result when get_dialog_state() retrieves no rows
- presence: Fix memory leak in DB code
- pkg/kamailio/deb: add Ubuntu Xenial support
- presence: fix crash when no dialog.id on dialog-info xml
- dispatcher: proper selection of the gateway when the maxload is 0
- http_client Terminate exported commands list
- dialog: wrapper defines for ref/unref operations to get file name and line
- dialog: no need to release the dialog when refreshing cseq
- core: allow '_' in event route name
- presence: expire value kept and printed in logs as unsigned int
- core: fixed misleading indentation for return statement
- core: proper code block when saving chosen naptr record
- core: ut - fix misleading indentation for IF blocks
- auth_identity: fix misleading indentation level
- counters: fixed rpc command to get counter by group and name
- utils: coherent indentation for xcap functions
- core: rely on msg->last_header to get the anchor inside sr_hdr_add()
- core: print sip message on error related to missing body
- websocket: handle CRLF received inside a frame (as SIP layer ping)
- presence: fix crush if dialog id is NULL
- core: defined macro for CRLFCRLF
- websocket: catch CRLFCRLF pings over websocket connections
- tmx: fix condition for private copy of the message
- modules/tmx: fix mi command t_uac_cancel
- auth_identity: switched to use pointer of X509_STORE_CTX
- auth_identity: still use ASN1_STRING_data() for libssl < v1.1.0
- crypto: updated to be compatible with api from libssl v1.1.0
- tls: updates for compatibilty with libssl v1.1.0 api
- tls: updated bio functions to support libssl v1.1.0 api
- core: parser - removed level and function prefix from log messages for identity info
- tm: fixed misleading indentation of block after an IF (reported by gcc 6.2)
- pv: adjusted indentation after if blocks to avoid gcc 6.2 warnings
- ldap: removed unused variable
- db_perlvdb: fixed warning of using static function inside inline one
- presence: remove unused variable
- pua_xmpp: fixed misleading missing guard for if block
- purple: Fix multiple memory leaks in DB code
- core: print the len for added header in debug message
- core: msg parser - relocated static inline functions from header file
- core: msg parser - more log message for get body function
- ctl: split the definition of default ctl socket
- ctl: take in consideration runtime_dir when creation the ctl socket
- Makefile.defs: option to disable raw sockets at compile time
- Presence: Fix startup inconsistency in presentity hash table
- dialog: propre reparse of msg structure when doing cseq refresh ops
- janssonrpc-c: fix memory leak when using janssonrpc_notification
- mem: TLSF bit ops cleanup / 64 bits improvement
- pv: Use memcpy to copy len bytes instead of strcpy
- rr: more log details when 2nd rr with r2=on is not matching a local socket
- rr: fixed typo in log message
- permissions: update doc with missing MI and RPC commands:
- rr: use default port for protocol if it is omitted in route header
- lib/srdb1: fix the memset to 0 in util function
- db_postgres: safety check of str buffer pointer before writing
- htable: earlier init of jdoc used for dmq for proper clean on error cases
- pua: copy over subs flag in subscribe_cbparam()
- pua: one more proper propagation of ua_flags in subs_cbparam_indlg()
- pua: fix propagation of ua_flag - it is func param, not in subs
- core: parser - use square brakets for log messages of addr-spec parsing
- core: parse addr spec - removed old style log message prefixes
- permissions: doc typo fixed
- uac: coherent indentation of code for replace from/to values
- uac: safety check for new uri to include only printable characters
- ndb_redis: get the srv structure after checking parameter sanity
- lib/srutils: removed valueint field from json struct
- dialog: converted to use json helper macros for dlg profile
- dmq_usrloc: use usigned int for contact info values
- htable: switched to json macros for numbers in dmq operations
- rtjson: use json macros to retrieve int values
- jsonrpc-s: switched to use json helper macros for numbers
- lib/srutils: srjson - init number value for null, true and false
- modules/nathelper: fix small typo in documentation
- modules/sanity: fix small typo in documentation
- modules/dispatcher: fix small typo in documentation
- modules/crypto: fix small typo in documentation
- domain: fixed typo in the log message
- mi_datagram: safety check so the length of mi socket doesn't result in overflow
- core: parser/sdp - init pf variable to avoid compile time warnings
- rr: removed unused assignment
- janssonrpc-c: declare variables at beginning of netstring_read_fd()
- core: parser/parse_diversion - allow comma separated bodies
- modules: readme files regenerated - crypto ...
- xmpp: shift head of connection list when first is closed
- jansson: init structure to avoid compile warnings
- presence: free cloned dialog_id in case of errors
- auth_identity: safer condition not to overflow building date header
- statsd: increase buffer for miliseconds printing as function uses 21 size
- statsd: free srv variable in error handling
- statsd: one more increase for milliseconds buffer
- rtpengine: free addrinfo in case of unknown address family
- dialog: secure space in case \r\n needs to be added after extra headers
- janssonrpc-c: fix shared memory leak on notification requests
- app_python: dname is always allocated in system memory
- core: init structure to 0 for pass fd operations
- mi_fifo: close reply stream in case of continuing due to no response
- ratelimit: close file pointer in case of scan error
- janssonrpc-c: fix double free introduced by
- debugger: re-order loop condition to avoid out of bounds access
- ctl: proper trim for long error messages to avoid out of bounds access
- mi_fifo: close fifo stream in case of errors
- mi_rpc: init structures to avoid access to invalid content in case of errors
- rtpengine: init all struct to get rid of compile warnings
- lib/binrpc: memset addr variable to avoind uninitialized fields
- auth_ephemeral: re-ordered code so access to sip msg is done after NULL check
- app_lua: safety check of returned anchor before using it
- tm: print warn messages when failing to update r-uri and path
- usrloc: handle return code when registering the timer routine
- corex: handle return code for ftell
- pv: pv_var_to_xavp() - proper test for type int of script var
- pv: return false if pv_unset() fails
- textops: handle return code for parse_headers()
- pkg/kamailio/deb: version set to 4.4.4
- pkg: version set to 4.4.4 in rpm specs
- Makefile.defs: version set to 4.4.4
- ChangeLog: content updated for v4.4.4
- dispatcher: updates to sameple config file
- dispatcher: docs - removed extra spaces and rephrased sample config text
- modules: readme files regenerated - dispatcher ...
- pua_dialoginfo: safety check of paramters for __dialog_sendpublish()
- lib/srdb1: schema - don't set default value for text/blob columns of topos
- kamctl: regenerated database schema
File Changes
Patch Links:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.