[Git][pkg-voip-team/asterisk][debian/latest] 2 commits: update patch 2016 to avoid excessive logging; closes: bug#1060275, thanks to Daniel

[Jonas Smedegaard (@js)]

Jonas Smedegaard pushed to branch debian/latest at Debian VoIP Packaging Team / asterisk


Commits:
7e84eb4d by Jonas Smedegaard at 2024-01-30T11:41:49+01:00
update patch 2016 to avoid excessive logging; closes: bug#1060275, thanks to Daniel

- - - - -
d2f048e8 by Jonas Smedegaard at 2024-01-30T11:42:37+01:00
update changelog

- - - - -


2 changed files:

- debian/changelog
- debian/patches/2016_opus_plc.patch


Changes:

=====================================
debian/changelog
=====================================
@@ -1,3 +1,10 @@
+asterisk (1:20.6.0~dfsg+~cs6.13.40431414-2) unstable; urgency=medium
+
+  * update patch 2016 to avoid excessive logging;
+    closes: bug#1060275, thanks to Daniel
+
+ -- Jonas Smedegaard <dr at jones.dk>  Tue, 30 Jan 2024 11:42:08 +0100
+
 asterisk (1:20.6.0~dfsg+~cs6.13.40431414-1) unstable; urgency=medium
 
   [ upstream ]


=====================================
debian/patches/2016_opus_plc.patch
=====================================
@@ -1,11 +1,14 @@
 Description: detect lost (and late) RTP packets for native PLC
- Original patch adapted by Jonas Smedegaard to fit Debian packaging,
+ Original patch adapted by Jonas Smedegaard
+ to fit Debian packaging,
+ disable debugging NOTICEs,
  and to remove original patch file to help detect later upstream changes.
 Author: Alexander Traud <pabstraud at compuserve.com>
 Author: Jonas Smedegaard <dr at jones.dk>
 Source: https://github.com/traud/asterisk-opus/blob/asterisk-13.7/enable_native_plc.patch
+Bug-Debian: https://bugs.debian.org/1060275
 Forwarded: not-needed
-Last-Update: 2023-08-27
+Last-Update: 2024-01-30
 ---
 This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
 --- a/main/translate.c
@@ -18,7 +21,7 @@ This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
  
  	/*
  	 * If the translator has not provided a format
-@@ -564,13 +565,46 @@
+@@ -564,13 +565,47 @@
  /*! \brief do the actual translation */
  struct ast_frame *ast_translate(struct ast_trans_pvt *path, struct ast_frame *f, int consume)
  {
@@ -58,8 +61,9 @@ This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
 +		if (frames_missing > 96) {
 +			struct ast_str *str = ast_str_alloca(256);
 +
-+			/* not DEBUG but NOTICE because of WARNING in main/cannel.c:__ast_queue_frame */
-+			ast_log(LOG_NOTICE, "%d lost frame(s) %d/%d %s\n", frames_missing, f->seqno, path->f.seqno, ast_translate_path_to_str(path, &str));
++			/* disabled by default to not flood log in production even in non-Opus use
++			/* not DEBUG but NOTICE because of WARNING in main/channel.c:__ast_queue_frame */
++			//ast_log(LOG_NOTICE, "%d lost frame(s) %d/%d %s\n", frames_missing, f->seqno, path->f.seqno, ast_translate_path_to_str(path, &str));
 +		}
 +	} else {
 +		frames_missing = 0;
@@ -67,7 +71,7 @@ This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
  
  	if (f->frametype == AST_FRAME_RTCP) {
  		/* Just pass the feedback to the right callback, if it exists.
-@@ -611,19 +645,94 @@
+@@ -611,19 +646,94 @@
  			 f->samples, ast_format_get_sample_rate(f->subclass.format)));
  	}
  	delivery = f->delivery;



View it on GitLab: https://salsa.debian.org/pkg-voip-team/asterisk/-/compare/81d62582115c31330ade8ae71e1cab98beecac33...d2f048e88cadcbc552342961f59382722b8dd266

-- 
View it on GitLab: https://salsa.debian.org/pkg-voip-team/asterisk/-/compare/81d62582115c31330ade8ae71e1cab98beecac33...d2f048e88cadcbc552342961f59382722b8dd266
You're receiving this email because of your account on salsa.debian.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/pkg-voip-maintainers/attachments/20240130/b6e30df1/attachment-0001.htm>