IAX2 Changes
------------
-* authdebug is now disabled by default. To enable this functionaility again
+ * authdebug is now disabled by default. To enable this functionaility again
set authdebug = yes in iax.conf.
RTP Changes
* The rtp.conf setting "strictrtp" is now enabled by default. In previous
releases it was disabled.
+PBX Core
+--------
+ * The PBX core previously made a call with a non-existing extension test for
+ extension s@default and jump there if the extension existed.
+ This was a bad default behaviour and violated the principle of least surprise.
+ It has therefore been changed in this release. It may affect some
+ applications and configurations that rely on this behaviour. Most channel
+ drivers have avoided this for many releases by testing whether the extension
+ called exists before starting the PBX and generating a local error.
+ This behaviour still exists and works as before.
+
+ Extension "s" is used when no extension is given in a channel driver,
+ like immediate answer in DAHDI or calling to a domain with no user part
+ in a SIP uri.
+
------------------------------------------------------------------------------
--- Functionality changes from Asterisk 1.6.2 to Asterisk 1.8 ----------------
------------------------------------------------------------------------------
; In macros, it is the start extension. In most other cases,
; you have to goto "s" to execute that extension.
;
+; Note: In old versions of Asterisk the PBX in some cases defaulted to
+; extension "s" when a given extension was wrong (like in AMI originate).
+; This is no longer the case.
+;
; For wildcard matches, see above - all pattern matches start with
; an underscore.
exten => s,1,Goto(${ARG1},1)
autoloopflag = ast_test_flag(c, AST_FLAG_IN_AUTOLOOP); /* save value to restore at the end */
ast_set_flag(c, AST_FLAG_IN_AUTOLOOP);
- /* Start by trying whatever the channel is set to */
- if (!ast_exists_extension(c, c->context, c->exten, c->priority,
- S_COR(c->caller.id.number.valid, c->caller.id.number.str, NULL))) {
- /* If not successful fall back to 's' */
+ if (ast_strlen_zero(c->exten)) {
+ /* If not successful fall back to 's' - but only if there is no given exten */
ast_verb(2, "Starting %s at %s,%s,%d failed so falling back to exten 's'\n", c->name, c->context, c->exten, c->priority);
/* XXX the original code used the existing priority in the call to
* ast_exists_extension(), and reset it to 1 afterwards.
* I believe the correct thing is to set it to 1 immediately.
- */
+ */
set_ext_pri(c, "s", 1);
- if (!ast_exists_extension(c, c->context, c->exten, c->priority,
- S_COR(c->caller.id.number.valid, c->caller.id.number.str, NULL))) {
- /* JK02: And finally back to default if everything else failed */
- ast_verb(2, "Starting %s at %s,%s,%d still failed so falling back to context 'default'\n", c->name, c->context, c->exten, c->priority);
- ast_copy_string(c->context, "default", sizeof(c->context));
- }
}
+
if (c->cdr) {
/* allow CDR variables that have been collected after channel was created to be visible during call */
ast_cdr_update(c);