Merge remote-tracking branch 'upstream/dev' into mass

.github/ISSUE_TEMPLATE/bug_report.md

@@ -7,22 +7,33 @@ assignees: ''
 
 ---
 
-**Describe the bug**
-A clear and concise description of what the bug is.
+### Look for similar bugs
+Please check if there's [already an issue](https://github.com/librespot-org/librespot/issues) for your problem.
+If you've only a "me too" comment to make, consider if a :+1: [reaction](https://github.blog/news-insights/product-news/add-reactions-to-pull-requests-issues-and-comments/)
+will suffice. 
 
-**To reproduce**
-Steps to reproduce the behavior:
+### Description
+A clear and concise description of what the problem is.
+
+### Version
+What version(s) of *librespot* does this problem exist in?
+
+### How to reproduce
+Steps to reproduce the behavior in *librespot* e.g.
 1. Launch `librespot` with '...'
 2. Connect with '...'
 3. In the client click on '...'
-4. See error
+4. See some error/problem
 
-**Log**
-A full log so we may trace your problem (launch `librespot` with `--verbose`). Format the log as code.
+### Log
+* A *full* **debug** log so we may trace your problem (launch `librespot` with `--verbose`).
+* Ideally contains your above steps to reproduce.
+* Format the log as code ([help](https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/creating-and-highlighting-code-blocks)) or use a *non-expiring* [pastebin](https://pastebin.com/).
+* Redact data you consider personal but do not remove/trim anything else.
 
-**Host (what you are running `librespot` on):**
+### Host (what you are running `librespot` on):
 - OS: [e.g. Linux]
 - Platform: [e.g. RPi 3B+]
 
-**Additional context**
+### Additional context
 Add any other context about the problem here. If your issue is related to sound playback, at a minimum specify the type and make of your output device.

.github/workflows/test.yml

@@ -51,7 +51,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/[email protected].0
+        uses: actions/[email protected].1
       - name: Install toolchain
         run: curl https://sh.rustup.rs -sSf | sh -s -- --profile default --default-toolchain stable  -y
       - run: cargo fmt --all -- --check
@@ -68,7 +68,7 @@ jobs:
         toolchain: [stable]
     steps:
       - name: Checkout code
-        uses: actions/[email protected].0
+        uses: actions/[email protected].1
 
       - name: Install toolchain
         run: curl https://sh.rustup.rs -sSf | sh -s -- --profile default --default-toolchain ${{ matrix.toolchain }}  -y
@@ -79,7 +79,7 @@ jobs:
         shell: bash
 
       - name: Cache Rust dependencies
-        uses: actions/cache@v4.0.2
+        uses: actions/cache@v4.1.1
         with:
           path: |
             ~/.cargo/registry/index
@@ -119,7 +119,7 @@ jobs:
             experimental: true
     steps:
       - name: Checkout code
-        uses: actions/[email protected].0
+        uses: actions/[email protected].1
 
       - name: Install toolchain
         run: curl https://sh.rustup.rs -sSf | sh -s -- --profile minimal --default-toolchain ${{ matrix.toolchain }}  -y
@@ -130,7 +130,7 @@ jobs:
         shell: bash
 
       - name: Cache Rust dependencies
-        uses: actions/cache@v4.0.2
+        uses: actions/cache@v4.1.1
         with:
           path: |
             ~/.cargo/registry/index
@@ -168,7 +168,7 @@ jobs:
           - stable
     steps:
       - name: Checkout code
-        uses: actions/[email protected].0
+        uses: actions/[email protected].1
 
       # hyper-rustls >=0.27 uses aws-lc as default backend which requires NASM to build
       - name: Install NASM
@@ -183,7 +183,7 @@ jobs:
         shell: bash
 
       - name: Cache Rust dependencies
-        uses: actions/cache@v4.0.2
+        uses: actions/cache@v4.1.1
         with:
           path: |
             ~/.cargo/registry/index
@@ -219,7 +219,7 @@ jobs:
           - stable
     steps:
       - name: Checkout code
-        uses: actions/[email protected].0
+        uses: actions/[email protected].1
 
       - name: Install toolchain
         run: curl https://sh.rustup.rs -sSf | sh -s -- --profile minimal --default-toolchain ${{ matrix.toolchain }}  -y
@@ -230,7 +230,7 @@ jobs:
         shell: bash
 
       - name: Cache Rust dependencies
-        uses: actions/cache@v4.0.2
+        uses: actions/cache@v4.1.1
         with:
           path: |
             ~/.cargo/registry/index

CHANGELOG.md

@@ -5,9 +5,26 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html) since v0.2.0.
 
-## [0.5.0-dev] - YYYY-MM-DD
+## [Unreleased]
 
-This version will be a major departure from the architecture up until now. It
+### Changed
+
+- [core] The `access_token` for http requests is now acquired by `login5`
+
+### Added
+
+- [core] Add `login` (mobile) and `auth_token` retrieval via login5
+- [core] Add `OS` and `os_version` to `config.rs`
+
+### Removed
+
+### Fixed 
+
+- [connect] Fixes initial volume showing zero despite playing in full volume instead
+
+## [0.5.0] - 2024-10-15
+
+This version is be a major departure from the architecture up until now. It
 focuses on implementing the "new Spotify API". This  means moving large parts
 of the Spotify protocol from Mercury to HTTP. A lot of this was reverse
 engineered before by @devgianlu of librespot-java. It was long overdue that we
@@ -17,7 +34,7 @@ hopefully upcoming Spotify HiFi depend on it.
 Splitting up the work on the new Spotify API, v0.5.0 brings HTTP-based file
 downloads and metadata access. Implementing the "dealer" (replacing the current
 Mercury-based SPIRC message bus with WebSockets, also required for social plays)
-is separate large effort, to be targeted for v0.6.0.
+is a large and separate effort, slated for some later release.
 
 While at it, we are taking the liberty to do some major refactoring to make
 librespot more robust. Consequently not only the Spotify API changed but large
@@ -39,13 +56,15 @@ https://github.com/librespot-org/librespot
 - [all] Use a single `player` instance. Eliminates occasional `player` and
   `audio backend` restarts, which can cause issues with some playback
   configurations.
+- [all] Updated and removed unused dependencies
 - [audio] Files are now downloaded over the HTTPS CDN (breaking)
 - [audio] Improve file opening and seeking performance (breaking)
 - [core] MSRV is now 1.74 (breaking)
 - [connect] `DeviceType` moved out of `connect` into `core` (breaking)
 - [connect] Update and expose all `spirc` context fields (breaking)
 - [connect] Add `Clone, Defaut` traits to `spirc` contexts
 - [connect] Autoplay contexts are now retrieved with the `spclient` (breaking)
+- [contrib] Updated Docker image
 - [core] Message listeners are registered before authenticating. As a result
   there now is a separate `Session::new` and subsequent `session.connect`.
   (breaking)
@@ -59,6 +78,7 @@ https://github.com/librespot-org/librespot
 - [core] `Credentials.username` is now an `Option` (breaking)
 - [core] `Session::connect` tries multiple access points, retrying each one.
 - [core] Each access point connection now timesout after 3 seconds.
+- [core] Listen on both IPV4 and IPV6 on non-windows hosts
 - [main] `autoplay {on|off}` now acts as an override. If unspecified, `librespot`
   now follows the setting in the Connect client that controls it. (breaking)
 - [metadata] Most metadata is now retrieved with the `spclient` (breaking)
@@ -76,6 +96,7 @@ https://github.com/librespot-org/librespot
 
 - [all] Check that array indexes are within bounds (panic safety)
 - [all] Wrap errors in librespot `Error` type (breaking)
+- [audio] Make audio fetch parameters tunable
 - [connect] Add option on which zeroconf will bind. Defaults to all interfaces. Ignored by DNS-SD.
 - [connect] Add session events
 - [connect] Add `repeat`, `set_position_ms` and `set_volume` to `spirc.rs`
@@ -95,6 +116,8 @@ https://github.com/librespot-org/librespot
 - [core] Support parsing `SpotifyId` for local files
 - [core] Support parsing `SpotifyId` for named playlists
 - [core] Add checks and handling for stale server connections.
+- [core] Fix potential deadlock waiting for audio decryption keys.
+- [discovery] Add option to show playback device as a group
 - [main] Add all player events to `player_event_handler.rs`
 - [main] Add an event worker thread that runs async to the main thread(s) but
   sync to itself to prevent potential data races for event consumers
@@ -118,11 +141,15 @@ https://github.com/librespot-org/librespot
 - [connect] Loading previous or next tracks, or looping back on repeat, will
   only start playback when we were already playing
 - [connect, playback] Clean up and de-noise events and event firing
+- [core] Fixed frequent disconnections for some users
+- [core] More strict Spotify ID parsing
+- [discovery] Update active user field upon connection
 - [playback] Handle invalid track start positions by just starting the track
   from the beginning
 - [playback] Handle disappearing and invalid devices better
 - [playback] Handle seek, pause, and play commands while loading
 - [playback] Handle disabled normalisation correctly when using fixed volume
+- [playback] Do not stop sink in gapless mode
 - [metadata] Fix missing colon when converting named spotify IDs to URIs
 
 ## [0.4.2] - 2022-07-29
@@ -284,16 +311,17 @@ v0.4.x as a stable branch until then.
 
 ## [0.1.0] - 2019-11-06
 
-[0.5.0-dev]: https://github.com/librespot-org/librespot/compare/v0.4.1..HEAD
-[0.4.2]: https://github.com/librespot-org/librespot/compare/v0.4.1..v0.4.2
-[0.4.1]: https://github.com/librespot-org/librespot/compare/v0.4.0..v0.4.1
-[0.4.0]: https://github.com/librespot-org/librespot/compare/v0.3.1..v0.4.0
-[0.3.1]: https://github.com/librespot-org/librespot/compare/v0.3.0..v0.3.1
-[0.3.0]: https://github.com/librespot-org/librespot/compare/v0.2.0..v0.3.0
-[0.2.0]: https://github.com/librespot-org/librespot/compare/v0.1.6..v0.2.0
-[0.1.6]: https://github.com/librespot-org/librespot/compare/v0.1.5..v0.1.6
-[0.1.5]: https://github.com/librespot-org/librespot/compare/v0.1.3..v0.1.5
-[0.1.3]: https://github.com/librespot-org/librespot/compare/v0.1.2..v0.1.3
-[0.1.2]: https://github.com/librespot-org/librespot/compare/v0.1.1..v0.1.2
-[0.1.1]: https://github.com/librespot-org/librespot/compare/v0.1.0..v0.1.1
+[unreleased]: https://github.com/librespot-org/librespot/compare/v0.5.0...HEAD
+[0.5.0]: https://github.com/librespot-org/librespot/compare/v0.4.2...v0.5.0
+[0.4.2]: https://github.com/librespot-org/librespot/compare/v0.4.1...v0.4.2
+[0.4.1]: https://github.com/librespot-org/librespot/compare/v0.4.0...v0.4.1
+[0.4.0]: https://github.com/librespot-org/librespot/compare/v0.3.1...v0.4.0
+[0.3.1]: https://github.com/librespot-org/librespot/compare/v0.3.0...v0.3.1
+[0.3.0]: https://github.com/librespot-org/librespot/compare/v0.2.0...v0.3.0
+[0.2.0]: https://github.com/librespot-org/librespot/compare/v0.1.6...v0.2.0
+[0.1.6]: https://github.com/librespot-org/librespot/compare/v0.1.5...v0.1.6
+[0.1.5]: https://github.com/librespot-org/librespot/compare/v0.1.3...v0.1.5
+[0.1.3]: https://github.com/librespot-org/librespot/compare/v0.1.2...v0.1.3
+[0.1.2]: https://github.com/librespot-org/librespot/compare/v0.1.1...v0.1.2
+[0.1.1]: https://github.com/librespot-org/librespot/compare/v0.1.0...v0.1.1
 [0.1.0]: https://github.com/librespot-org/librespot/releases/tag/v0.1.0