[bluez/bluez] d8ed4a: shared/bap: Skip local metadata Config callbacks

[bluez/bluez] d8ed4a: shared/bap: Skip local metadata Config callbacks

[github-actions[bot]]

  Branch: refs/heads/1167792
  Home:   https://github.com/bluez/bluez
  Commit: d8ed4a642abad923c56e90c038547d9cc7f7ba44
      https://github.com/bluez/bluez/commit/d8ed4a642abad923c56e90c038547d9cc7f7ba44
  Author: Derek Tern <1263187-dt@users.noreply.gitlab.freedesktop.org>
  Date:   2026-09-17 (Thu, 17 Sep 2026)

  Changed paths:
    M src/shared/bap.c

  Log Message:
  -----------
  shared/bap: Skip local metadata Config callbacks

When reconfiguring a client stream that is already Codec Configured,
setup_config() queues Codec Configuration and then caches metadata.
stream_metadata() emits a same-state CONFIG notification for that local
update. The profile callback treats it as the peer's configuration
completion and clears config_pending before the peer has responded.

If the control point response precedes the ASE state notification, QoS
setup can run while need_reconfig is still set and fail locally.

Cache client metadata in CONFIG without emitting a state notification.
Keep genuine peer CONFIG notifications and server metadata handling
unchanged.


  Commit: c27e1581c612b134d33ec19e56b92fcfbf20d6c6
      https://github.com/bluez/bluez/commit/c27e1581c612b134d33ec19e56b92fcfbf20d6c6
  Author: Derek Tern <1263187-dt@users.noreply.gitlab.freedesktop.org>
  Date:   2026-09-17 (Thu, 17 Sep 2026)

  Changed paths:
    M doc/functional-bap.rst
    M test/functional/test_bap.py

  Log Message:
  -----------
  test-functional: Cover BAP metadata reconfiguration

Exercise the real daemon by clearing a configured unicast endpoint and
reconfiguring its retained ASE with empty or Media-context metadata.
Require successful configuration and check the resulting transport.

Use ATT MTU 48 to prevent the control-point response and fresh ASE
notification from being batched together. Without the shared/bap fix,
the Media case fails QoS setup; the empty-metadata control still passes.

Call SetConfiguration directly because endpoint.config does not pass
metadata for unicast presets. Remove the fixture's redundant config
command: SelectProperties already configures both ASEs after pairing.


Compare: https://github.com/bluez/bluez/compare/d8ed4a642aba%5E...c27e1581c612

To unsubscribe from these emails, change your notification settings at https://github.com/bluez/bluez/settings/notifications