| Summary: | ASTERISK-23938: pri set debug 0 span <span> disables file debugging for all spans | ||
| Reporter: | Jonathan Rose (jrose) | Labels: | |
| Date Opened: | 2014-06-25 16:47:44 | Date Closed: | |
| Priority: | Major | Regression? | No |
| Status: | Open/New | Components: | Channels/chan_dahdi |
| Versions: | SVN 1.8.28.2 11.10.2 12.3.2 13.18.4 | Frequency of Occurrence | Constant |
| Related Issues: | |||
| Environment: | Attachments: | ||
| Description: | If debugging is going on for a number of spans and
'pri set debug 0 span <any span>' is used, Asterisk will disable the file output for pri debug entirely without bothering to check if other spans still have log levels higher than 0. Note that there is no handle for pri set debug without setting span. My personal feeling on this is that 'pri set debug 0 span <span>' shouldn't actually touch the file debug at all and instead 'pri unset debug file' should be added for this purpose. The alternative is to check all spans to see if their levels are 0 and disable it accordingly, but this still would unset the file when that might not actually be expected since the user didn't unset it explicitly. | ||
| Comments: | By: Rusty Newton (rnewton) 2014-06-25 19:19:49.900-0500 {quote} My personal feeling on this is that 'pri set debug 0 span <span>' shouldn't actually touch the file debug at all and instead 'pri unset debug file' should be added for this purpose. {quote} I generally agree, but would prefer something like "pri set debug file off". Though, that is just my preference, I think either would do. By: Rusty Newton (rnewton) 2014-06-25 19:20:31.276-0500 There was no version of Asterisk set, I'm going to assume this affects all versions. You can change it later if that isn't the case. By: Jonathan Rose (jrose) 2014-06-26 11:42:10.262-0500 The assumption was correct. My apologies for forgetting to fill in the field. | ||