Remove reference to log_to_syslog() if libdaemon support is included.

Merge pull request #2243 from daeho-ro/fix-log-to-syslog-libdaemon-build [skip ci]

Comment out the last remaining call to log_to_syslog()

Comment out the last remaining call to log_to_syslog()

Its definition in common.c and its declaration in common.h are commented
out, but the call in the libdaemon daemonise path was left behind, so
building with --with-libdaemon fails: clang reports "use of undeclared
identifier 'log_to_syslog'" and gcc fails to link with "undefined
reference to `log_to_syslog'".

Update release notes for Version 5.3-dev [skip ci]

Added details for Version 5.3-dev and metadata feature restoration.

5.3-dev is essentially 5.2.

This is version 5.2

Drop references to log-to and friends -- no longer used. [skip ci].

Merge pull request #2242 from mikebrady/dependabot/github_actions/development/docker/login-action-4.5.1 [skip ci]

Bump docker/login-action from 4.4.0 to 4.5.1

Merge pull request #2241 from mikebrady/dependabot/github_actions/development/actions/checkout-7.0.1 [skip ci]

Bump actions/checkout from 7.0.0 to 7.0.1

Bump docker/login-action from 4.4.0 to 4.5.1

Bumps [docker/login-action](https://github.com/docker/login-action) from 4.4.0 to 4.5.1.
- [Release notes](https://github.com/docker/login-action/releases)
- [Commits](https://github.com/docker/login-action/compare/v4.4.0...v4.5.1)

---
updated-dependencies:
- dependency-name: docker/login-action
  dependency-version: 4.5.1
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Bump actions/checkout from 7.0.0 to 7.0.1

Bumps [actions/checkout](https://github.com/actions/checkout) from 7.0.0 to 7.0.1.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v7.0.0...v7.0.1)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: 7.0.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Quieten a debug message. [skip ci]

Merge branch 'development' of github.com:mikebrady/shairport-sync into development

Online documention updates.

Remove the redundant "--log-to-syslog" setting from the startup script [skip ci]

Default to receiving metadata the older "classic" way for compatibility. Add a temporary setting to change this in the "diagnostics" part of the configuration file.

Update release notes for version 5.2-dev-46 [skip ci]

Restores previous metadata receiving method for compatibility and adds temporary configuration option for plist-based metadata.

Update version to 5.2-dev-44-geb49aaab [skip ci]

Updated version number and enhanced bug fix acknowledgment.

Settle on a convolver block size of 1024.

Update version to 5.2-dev-42 and fix bugs [skip ci]

Updated version number and fixed bugs in convolution and loudness code related to input/output rates and channel counts.

Temporarily stop block length check.

Fix typo in comment out of convolution_ir_files in config [skip ci]

Comment out the convolution_ir_files configuration line.

Update convolution comments in the configuration file. [skip ci]

Update release notes for version 5.2-dev-38 [skip ci]

Fixed bugs in convolution and loudness code related to input/output rates and channel counts. Added acknowledgment for the bug report.

Fix mixups between input and output rates and channel counts during convolution initialisation and in convolution attenuation and loudness application.

Update release notes with deprecations and bug fixes [skip ci]

Added deprecation notice for obsolete logging options.

Update release notes for version 5.2-dev-35 [skip ci]

Added a new exit handler for clean exits and fixed various bugs related to mutex handling and compiler warnings.

Add a new exit handler to enable Shairport Sync to exit cleanly — that is, on exit, to clean up everything, e.g. open files, ports, sockets, devices, threads, it has been using.
If cleanup isn’t finished after one second, exit will occur unconditionally.
Use the new exit handler for normal exit, for SIGINT and SIGTERM handling, for abnormal exit after fatal errors, and for handling quit requests from the D-Bus and MPRIS interfaces.

When an unfixable error occurs without an installed unfixable error handler, unlock the mutex protecting Shairport Sync’s alsa backend to permit a clean exit.

Fix a very low-level mutex-locking error that interfered with debugging.

Simplify much mutex lock handling.
Remove lots of old and unused code, variables and definitions.

Update audio_sndio.c to remove some harmless compiler warnings.

Undo adding two seconds for messages to be output before _Exit() on die() to allow messages to be output...
Undo making die() exit unconditionally.

Add 2 seconds for messages to be output before _Exit() on die() to allow messages to be output...

Make die() exit unconditionally. Clean up some more redundant code. Revert unfixable error to use die() rather than warn() and _Exit().

Update release notes for version 5.2-dev-31 [skip ci]

Added a bug fix for Shairport Sync hanging on unfixable errors and updated version number.

Merge pull request #2237 from mikebrady/dependabot/github_actions/development/actions/stale-10.4.0 [skip ci]

Bump actions/stale from 10.3.0 to 10.4.0

Exit immediately when an unfixable error is detected. Don't try to clean up before exit.

Bump actions/stale from 10.3.0 to 10.4.0

Bumps [actions/stale](https://github.com/actions/stale) from 10.3.0 to 10.4.0.
- [Release notes](https://github.com/actions/stale/releases)
- [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/stale/compare/v10.3.0...v10.4.0)

---
updated-dependencies:
- dependency-name: actions/stale
  dependency-version: 10.4.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Fix broken link in RELEASENOTES-DEVELOPMENT.md [skip ci]

typo

Fix broken link in RELEASENOTES-DEVELOPMENT.md [skip ci]

Fixed a broken link in the documentation referencing CONFIGURATION FLAGS.md from ADVANCED TOPICS.

Merge pull request #2235 from int3rrupt/fix-broken-doc-link [skip ci]

Fix broken link to `CONFIGURATION_FLAGS.md`

Fix broken link to `CONFIGURATION_FLAGS.md`

Update documentation for FreeBSD build requirements [skip ci]

Updated documentation to include 'gcrypt' in the FreeBSD build guide.

Update FreeBSD version in BUILD.md [skip ci]

Updated FreeBSD version from 14.3 to 15.1 in the documentation and adjusted related notes.

Fix package list for AirPlay 2 build [skip ci]

Updated package list for building Shairport Sync for AirPlay 2.

Update release notes for version 5.2-dev-10 [skip ci]

Added details about new MQTT command and bug fix.

Merge pull request #2226 from zutroy97/queue-next-track-id

Add MQTT queue_next command and fix track_id byte order.

Merge pull request #2232 from mikebrady/dependabot/github_actions/development/docker/login-action-4.4.0 [skip ci]

Bump docker/login-action from 4.2.0 to 4.4.0

Merge pull request #2231 from mikebrady/dependabot/github_actions/development/docker/build-push-action-7.3.0 [skip ci]

Bump docker/build-push-action from 7.2.0 to 7.3.0

Merge pull request #2230 from mikebrady/dependabot/github_actions/development/docker/setup-qemu-action-4.2.0 [skip ci]

Bump docker/setup-qemu-action from 4.1.0 to 4.2.0

Merge pull request #2229 from mikebrady/dependabot/github_actions/development/docker/setup-buildx-action-4.2.0 [skip ci]

Bump docker/setup-buildx-action from 4.1.0 to 4.2.0

Reverted back track_id as hex string

Bump docker/login-action from 4.2.0 to 4.4.0

Bumps [docker/login-action](https://github.com/docker/login-action) from 4.2.0 to 4.4.0.
- [Release notes](https://github.com/docker/login-action/releases)
- [Commits](https://github.com/docker/login-action/compare/v4.2.0...v4.4.0)

---
updated-dependencies:
- dependency-name: docker/login-action
  dependency-version: 4.4.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Bump docker/build-push-action from 7.2.0 to 7.3.0

Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 7.2.0 to 7.3.0.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](https://github.com/docker/build-push-action/compare/v7.2.0...v7.3.0)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  dependency-version: 7.3.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Bump docker/setup-qemu-action from 4.1.0 to 4.2.0

Bumps [docker/setup-qemu-action](https://github.com/docker/setup-qemu-action) from 4.1.0 to 4.2.0.
- [Release notes](https://github.com/docker/setup-qemu-action/releases)
- [Commits](https://github.com/docker/setup-qemu-action/compare/v4.1.0...v4.2.0)

---
updated-dependencies:
- dependency-name: docker/setup-qemu-action
  dependency-version: 4.2.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Bump docker/setup-buildx-action from 4.1.0 to 4.2.0

Bumps [docker/setup-buildx-action](https://github.com/docker/setup-buildx-action) from 4.1.0 to 4.2.0.
- [Release notes](https://github.com/docker/setup-buildx-action/releases)
- [Commits](https://github.com/docker/setup-buildx-action/compare/v4.1.0...v4.2.0)

---
updated-dependencies:
- dependency-name: docker/setup-buildx-action
  dependency-version: 4.2.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Publish and accept track_id as decimal instead of hex

Changes the mper-derived track_id published over MQTT from hex to
decimal, and updates queue_next to embed the value in the DACP
persistentid query without a 0x prefix, matching the new format.
Queueing behavior (command=add&mode=3) is unchanged.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Fix wording in release notes for version 5.2-dev [skip ci]

Update release notes for version 5.2-dev [skip ci]

Added notes for version 5.2-dev, including bug fixes and improvements.

This is 5.2-dev

Fix build: restore stop_play() for MQTT disconnect command

An unrelated change in a prior commit swapped stop_play() for
release_play_lock(NULL), which isn't declared in rtsp.h and broke
the build with an implicit-declaration error.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Fix formatting in AirPlay 2 release notes [skip ci]

Corrected formatting in the release notes regarding AirPlay 2.

Remove commented-out code from mqtt.c

Fix byte order of track_id published over MQTT

The 'mper' handler cast the raw big-endian persistent-id bytes directly
to a host-native uint64_t, which reverses the value on little-endian
hosts (e.g. Raspberry Pi). metadata_hub.c already reconstructs the same
field correctly by applying ntohl() to each 32-bit half; mqtt.c now does
the same, so the published track_id matches the real dmap.persistentid
and can be used directly with the queue_next command.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Add test track_id reference notes to mqtt.c

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Fix command format in MQTT queue_next function

Update MQTT command for queue_next functionality

Add queue_next MQTT remote command to enqueue a track by persistent ID

Sends ctrl-int/1/cue?command=add&query='dmap.persistentid:0x<id>'&mode=3
to the DACP source, where <id> is the hex track_id string taken directly
from the MQTT payload after "queue_next ".

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Fix grammatical error in README.md [skip ci]

Add systemctl daemon-reload command after cleanup [skip ci]

Added instructions for reloading systemd daemon after removing service files.

Update BUILD.md with repository setup instructions [skip ci]

Added instructions for cloning and updating the Shairport Sync repository.

Bug fix, docker-compose updates, statistics readoout improvement.

Update RELEASENOTES-DEVELOPMENT.md for version 5.1-dev-52 [skip ci]

Document bug fixes and minor improvements in release notes.

Add sys-nice request to docker compose file and comment out unnecessary stuff.

Add a few tools and do a recompile for more comportable troubleshooting.

If the minimum DAC queue size is not changed from maxint, assume it has never been set and indicate that it is not available.

Stop allocating large buffers on the stack, as it will overflow on, e.g. Alpine and embedded Linuxes, where the stack frame allocations are smaller. Use mallocs instead.

Merge branch 'master' of github.com:mikebrady/shairport-sync

Online update.

Add Jack deprecation note to the configuration file.

Add Jack deprecation note to the configuration file. [skip ci]

Add note about new --with-metadata-pipe and --with-metadata-multicast configuration flags.

 Add note about new --with-metadata- configuration settings.

Remove markdown test from Bug Report template [skip ci]

Removed markdown test section from Bug Report template.

Add markdown test section to Bug Report template [skip ci]

Added a markdown test section to the Bug Report template.

Merge branch 'development'

Update of configuration flag documentation.

Merge branch 'development' of github.com:mikebrady/shairport-sync into development

Online update.

Remove redundant text formatting in flags section [skip ci]

Merge branch 'development'

Update settings document.

Merge branch 'development' of github.com:mikebrady/shairport-sync into development

online update

Revise configuration flags and add metadata options [skip ci]

Updated the configuration flags documentation to indicate deprecation of the --with-jack flag and added new sections for metadata pipe and multicast.

Merge branch 'development'

Update copyright notice in the man source file.

Merge pull request #2220 from mikebrady/dependabot/github_actions/development/actions/checkout-7.0.0 [skip ci]

Bump actions/checkout from 6.0.3 to 7.0.0

Bump actions/checkout from 6.0.3 to 7.0.0

Bumps [actions/checkout](https://github.com/actions/checkout) from 6.0.3 to 7.0.0.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v6.0.3...v7.0.0)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: 7.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Update release notes with security bug fixes [skip ci]

Documented fixes for two security issues in pair_ap/pair-tlv.c and rtsp.c, including bounds checks and validation improvements. Acknowledged contributions from TristanInSec for reporting and fixing the issues.

Merge pull request #2218 from TristanInSec/fix-tlv-dmap-oob-reads-v2

Fix Out-Of-Bounds reads and infinite loop in TLV and DMAP parsers.

Fix OOB reads and infinite loop in TLV and DMAP parsers

pair_ap/pair-tlv.c (pair_tlv_parse):
- Add bounds check before reading type and length bytes (i + 2 > length)
- Fix off-by-one in chunked TLV scan loop (j + 1 < length)
- Add bounds checks inside data copy loop to prevent OOB memcpy
- Add else branch to advance index when size == 0 (prevents infinite loop)

rtsp.c (handle_set_parameter_metadata):
- Require 8 bytes remaining for tag + length fields (off + 8 <= cl)
- Validate value length against remaining buffer before read

Reported-by: Tristan Madani <tristan@talencesecurity.com>
Signed-off-by: Tristan Madani <tristan@talencesecurity.com>

Correct grammar in bug fix note for Classic operation [skip ci]

Update release notes with new bug fix details [skip ci]

Added a new bug fix related to immediate flush and PTP clock anchor reset.

When an immediate flush is requested, reset the PTP clock anchor information immediately to disable clocking until play is resumed.

Merge pull request #2212 from mikebrady/dependabot/github_actions/development/actions/checkout-6.0.3 [skip ci]

Bump actions/checkout from 6.0.2 to 6.0.3

Merge pull request #2209 from mikebrady/dependabot/github_actions/development/docker/setup-qemu-action-4.1.0 [skip ci]

Bump docker/setup-qemu-action from 4.0.0 to 4.1.0

Merge pull request #2207 from mikebrady/dependabot/github_actions/development/docker/setup-buildx-action-4.1.0 [skip ci]

Bump docker/setup-buildx-action from 4.0.0 to 4.1.0

Bump docker/setup-buildx-action from 4.0.0 to 4.1.0

Bumps [docker/setup-buildx-action](https://github.com/docker/setup-buildx-action) from 4.0.0 to 4.1.0.
- [Release notes](https://github.com/docker/setup-buildx-action/releases)
- [Commits](https://github.com/docker/setup-buildx-action/compare/v4.0.0...v4.1.0)

---
updated-dependencies:
- dependency-name: docker/setup-buildx-action
  dependency-version: 4.1.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #2206 from mikebrady/dependabot/github_actions/development/actions/stale-10.3.0 [skip ci]

Bump actions/stale from 10.2.0 to 10.3.0