Compare commits
126
Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
8e85af2b3b | ||
|
|
2d23eb87c6 | ||
|
|
e5ce63a192 | ||
|
|
b5c9658d65 | ||
|
|
ba0a2373b5 | ||
|
|
b0163f6a4d | ||
|
|
0a192645d6 | ||
|
|
351b333529 | ||
|
|
155f0e2b6a | ||
|
|
a54d1ef077 | ||
|
|
2361b88e0a | ||
|
|
e7c2fd5571 | ||
|
|
3126cb6dd6 | ||
|
|
03afcc8a90 | ||
|
|
f210fa4702 | ||
|
|
4b708aa987 | ||
|
|
bd07b1134e | ||
|
|
c041daf649 | ||
|
|
975fa7e4fd | ||
|
|
1a2ae37f49 | ||
|
|
1cad36b929 | ||
|
|
8cb9386447 | ||
|
|
5cda7cb4c1 | ||
|
|
e89463fb83 | ||
|
|
b011ccdd5b | ||
|
|
1df7f6ccf6 | ||
|
|
2915737f03 | ||
|
|
5b2a0e4d46 | ||
|
|
37fc229bf6 | ||
|
|
bc43b4b6eb | ||
|
|
dda8c30d45 | ||
|
|
906f38140a | ||
|
|
3e546d4a1e | ||
|
|
b2f450dcf7 | ||
|
|
87c3f97529 | ||
|
|
302dd1e0f8 | ||
|
|
421fc929e8 | ||
|
|
c4ab5dc388 | ||
|
|
920dcc6779 | ||
|
|
430c711ff2 | ||
|
|
cfdc61a4b8 | ||
|
|
0f28e19c1d | ||
|
|
7316e355ee | ||
|
|
b39e55fd87 | ||
|
|
cb046bba9e | ||
|
|
b9a40186b1 | ||
|
|
fd0a39b879 | ||
|
|
cc7afe4326 | ||
|
|
72cbc18a48 | ||
|
|
84408b842b | ||
|
|
339fcb56ad | ||
|
|
74cf534be5 | ||
|
|
89820c1377 | ||
|
|
2fe26b15dd | ||
|
|
2cb560fda4 | ||
|
|
d8530b9209 | ||
|
|
5049b0ba91 | ||
|
|
1b98a75c0e | ||
|
|
cbbde95773 | ||
|
|
26bb8a50a3 | ||
|
|
bf19bb31f9 | ||
|
|
55cc803708 | ||
|
|
805e521b1d | ||
|
|
f8d431a0d7 | ||
|
|
7b72db8a24 | ||
|
|
b61ff1ebec | ||
|
|
8870c65c6f | ||
|
|
5df5e635f3 | ||
|
|
d5e3cadc43 | ||
|
|
505ae05c53 | ||
|
|
a7929c0ac9 | ||
|
|
4d39008eab | ||
|
|
5efd59826b | ||
|
|
8970f29410 | ||
|
|
523318fc06 | ||
|
|
ba531b4f61 | ||
|
|
3d51f08e0b | ||
|
|
2660c124d5 | ||
|
|
d7199b4d1d | ||
|
|
c60704080b | ||
|
|
1e3e46cfdc | ||
|
|
93d4220f3a | ||
|
|
8a748f2b3a | ||
|
|
2138a4a890 | ||
|
|
94a1981a4c | ||
|
|
2272a401bb | ||
|
|
3ab43a2ef6 | ||
|
|
79698f2265 | ||
|
|
94f80c03c5 | ||
|
|
d98d01ab20 | ||
|
|
6be5c43e6d | ||
|
|
8deb762a52 | ||
|
|
fe21c19810 | ||
|
|
6e86c3d2c5 | ||
|
|
aca4b73fac | ||
|
|
078f7ac6f6 | ||
|
|
b168df6ca6 | ||
|
|
8d178884d4 | ||
|
|
cd2131ef6f | ||
|
|
d6bff15f11 | ||
|
|
b82fca0bc2 | ||
|
|
4fbd4fa81d | ||
|
|
68b5709d88 | ||
|
|
434b2c1305 | ||
|
|
9064ef687c | ||
|
|
c93f3d352c | ||
|
|
a9bfcbbe2f | ||
|
|
717e939038 | ||
|
|
dd1cb8a5d1 | ||
|
|
7c9399a342 | ||
|
|
a3ec9d19d3 | ||
|
|
5f2b5bd5a2 | ||
|
|
3bb4e66ad0 | ||
|
|
fbb102ffbd | ||
|
|
76f7535ffa | ||
|
|
7fcf0645da | ||
|
|
50618865e7 | ||
|
|
fe2f6a785a | ||
|
|
53fc24d4bc | ||
|
|
ff5fa501af | ||
|
|
6e9883018a | ||
|
|
a985ff4fb3 | ||
|
|
63018a7c72 | ||
|
|
c5748e9765 | ||
|
|
23e30d96fc | ||
|
|
3f8d517ee3 |
@@ -0,0 +1,127 @@
|
|||||||
|
name: CI/CD
|
||||||
|
|
||||||
|
on:
|
||||||
|
push:
|
||||||
|
branches: ['**']
|
||||||
|
pull_request:
|
||||||
|
branches: ['**']
|
||||||
|
|
||||||
|
concurrency:
|
||||||
|
group: production-deploy
|
||||||
|
cancel-in-progress: false
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
backend-tests:
|
||||||
|
name: BackEnd
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
timeout-minutes: 20
|
||||||
|
defaults:
|
||||||
|
run:
|
||||||
|
working-directory: backend
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- uses: actions/setup-dotnet@v4
|
||||||
|
with:
|
||||||
|
dotnet-version: '10.0.x'
|
||||||
|
|
||||||
|
- name: Restore
|
||||||
|
run: dotnet restore Musify.slnx
|
||||||
|
|
||||||
|
- name: Build
|
||||||
|
run: dotnet build Musify.slnx --no-restore -c Release
|
||||||
|
|
||||||
|
- name: Test
|
||||||
|
run: dotnet test Musify.slnx --no-build -c Release
|
||||||
|
|
||||||
|
web-player-checks:
|
||||||
|
name: WebPlayer
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
timeout-minutes: 15
|
||||||
|
defaults:
|
||||||
|
run:
|
||||||
|
working-directory: web-player
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version: '22'
|
||||||
|
cache: npm
|
||||||
|
cache-dependency-path: web-player/package-lock.json
|
||||||
|
|
||||||
|
- run: npm ci
|
||||||
|
|
||||||
|
- name: Type-check
|
||||||
|
run: npm run check
|
||||||
|
|
||||||
|
- name: Lint
|
||||||
|
run: npm run lint
|
||||||
|
|
||||||
|
# Runs on the production host itself, through the existing self-hosted
|
||||||
|
# Gitea Actions runner (act_runner), which also carries a "production"
|
||||||
|
# label registered in native (non-docker) mode so its steps see the host's
|
||||||
|
# real Docker daemon directly (Docker socket + a bind-mounted deploy
|
||||||
|
# directory — see deploy/README.md for how that runner is set up).
|
||||||
|
#
|
||||||
|
# DEPLOY_PATH is the persistent clone already on that host (the one whose
|
||||||
|
# deploy/.env.prod is filled in and matches SharedServices/.env.prod) —
|
||||||
|
# deploy reuses it instead of a fresh checkout, since .env.prod is
|
||||||
|
# untracked and only lives there.
|
||||||
|
deploy:
|
||||||
|
name: Deploy
|
||||||
|
needs: [backend-tests, web-player-checks]
|
||||||
|
if: github.ref == 'refs/heads/master' && github.event_name == 'push'
|
||||||
|
runs-on: [production]
|
||||||
|
timeout-minutes: 20
|
||||||
|
env:
|
||||||
|
DEPLOY_PATH: /home/ubuntu/deploys/musify
|
||||||
|
COMPOSE: docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml
|
||||||
|
steps:
|
||||||
|
- name: Pull latest master
|
||||||
|
run: |
|
||||||
|
set -euo pipefail
|
||||||
|
git config --global --add safe.directory "$DEPLOY_PATH"
|
||||||
|
cd "$DEPLOY_PATH"
|
||||||
|
git fetch origin master
|
||||||
|
git reset --hard origin/master
|
||||||
|
|
||||||
|
- name: Generate stream-ticket keys
|
||||||
|
run: |
|
||||||
|
set -euo pipefail
|
||||||
|
cd "$DEPLOY_PATH"
|
||||||
|
$COMPOSE run --rm --no-deps keys-init
|
||||||
|
|
||||||
|
# The migrations are baked into the migrator image, so it is rebuilt on
|
||||||
|
# every deploy (plain `up` would reuse the stale image and re-apply the
|
||||||
|
# old migrations). `run --rm` starts wait-for-postgres, blocks until the
|
||||||
|
# migration finishes and propagates its exit code, so a failed migration
|
||||||
|
# aborts the deploy before the new API is started against an old schema.
|
||||||
|
- name: Migrate database
|
||||||
|
run: |
|
||||||
|
set -euo pipefail
|
||||||
|
cd "$DEPLOY_PATH"
|
||||||
|
$COMPOSE build migrate
|
||||||
|
$COMPOSE run --rm migrate
|
||||||
|
|
||||||
|
# Both jobs verify themselves through the public auth./s3. URLs, served
|
||||||
|
# by the shared edge nginx (Infrastructure's edge/), and are idempotent.
|
||||||
|
- name: Create OIDC app and S3 bucket
|
||||||
|
run: |
|
||||||
|
set -euo pipefail
|
||||||
|
cd "$DEPLOY_PATH"
|
||||||
|
$COMPOSE up -d seaweedfs-init zitadel-init
|
||||||
|
for job in musify-seaweedfs-init musify-zitadel-init; do
|
||||||
|
code=$(docker wait "$job")
|
||||||
|
if [ "$code" != "0" ]; then
|
||||||
|
docker logs "$job"
|
||||||
|
echo "$job exited with $code"
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
done
|
||||||
|
|
||||||
|
- name: Build and start apps
|
||||||
|
run: |
|
||||||
|
set -euo pipefail
|
||||||
|
cd "$DEPLOY_PATH"
|
||||||
|
$COMPOSE --profile apps up -d --build api worker gateway web-player
|
||||||
+3
-1
@@ -7,6 +7,9 @@ obj/
|
|||||||
*.userosscache
|
*.userosscache
|
||||||
.vs/
|
.vs/
|
||||||
artifacts/
|
artifacts/
|
||||||
|
TestResults/
|
||||||
|
*.trx
|
||||||
|
*.coverage
|
||||||
|
|
||||||
# Rider / VS
|
# Rider / VS
|
||||||
.idea/
|
.idea/
|
||||||
@@ -25,5 +28,4 @@ Thumbs.db
|
|||||||
# Deploy secrets and runtime state
|
# Deploy secrets and runtime state
|
||||||
deploy/keys/
|
deploy/keys/
|
||||||
deploy/secrets/
|
deploy/secrets/
|
||||||
deploy/nginx/certs/
|
|
||||||
deploy/zitadel/.output/
|
deploy/zitadel/.output/
|
||||||
|
|||||||
@@ -1,57 +1,55 @@
|
|||||||
# Musify
|
# Musify
|
||||||
|
|
||||||
Music streaming platform: upload your own tracks, process them asynchronously
|
A music streaming app. Upload your own tracks, organize them into playlists
|
||||||
(audio transcoded to `.m4a`, thumbnails generated) and stream them without the
|
and albums, and stream them back with a web player built around your
|
||||||
bytes ever going through the API.
|
listening habits: auto-generated mixes, a home feed, search, likes, and an
|
||||||
|
Explore page for finding something new.
|
||||||
|
|
||||||
|
## What it does
|
||||||
|
|
||||||
|
- **Upload and organize your own music**: tracks, albums, cover art, artist credits.
|
||||||
|
- **Playlists**, likes, and a "recently played" history.
|
||||||
|
- **Mixes**, playlists generated automatically from what you actually listen to.
|
||||||
|
- **Explore**, genre browsing and search across tracks, albums, playlists and people.
|
||||||
|
- **Real streaming**, not just file downloads: seek, queue, shuffle, repeat.
|
||||||
|
- **Accounts via OIDC login**, with an optional anonymous listening mode.
|
||||||
|
- Light and dark themes, and a layout that works on both desktop and mobile.
|
||||||
|
|
||||||
|
## How it's built
|
||||||
|
|
||||||
|
**Backend**: .NET 10, split into a REST API, a background worker, and a
|
||||||
|
dedicated streaming gateway.
|
||||||
|
- Clean architecture (Domain / Application / Infrastructure) with CQRS (Mediator) and `ErrorOr` results.
|
||||||
|
- PostgreSQL via EF Core for metadata. RabbitMQ and MassTransit handle async processing (audio transcoding, thumbnails) with saga-based rollback.
|
||||||
|
- SeaweedFS (S3-compatible) for file storage. Uploads and downloads go straight between the client and storage, never through the API.
|
||||||
|
- ffmpeg for audio transcoding, ImageSharp for thumbnails.
|
||||||
|
- A YARP reverse proxy that streams the audio and checks a signed, short-lived ticket on every request.
|
||||||
|
- Zitadel for login (OIDC/OAuth2).
|
||||||
|
|
||||||
|
**Web client**: SvelteKit 2 on Svelte 5 (runes), TypeScript, Tailwind 4.
|
||||||
|
The session lives on the server, so the backend access token never reaches the browser.
|
||||||
|
|
||||||
|
**Infrastructure**: Musify's own apps run from Docker Compose (dev and
|
||||||
|
production overlays). Postgres, RabbitMQ, SeaweedFS, Zitadel and Jaeger are
|
||||||
|
shared infrastructure that lives in a separate `Infrastructure` git
|
||||||
|
repository, used by Musify and by other projects that need the same OIDC
|
||||||
|
provider and object storage.
|
||||||
|
|
||||||
|
## Project layout
|
||||||
|
|
||||||
```
|
```
|
||||||
backend/ .NET 10 — Domain / Application / Infrastructure / Api / Worker / StreamingGateway / Tests
|
backend/ .NET solution: Api, Worker, StreamingGateway, and the shared Domain/Application/Infrastructure
|
||||||
web-player/ SvelteKit 2 + Svelte 5 + Tailwind 4 web client
|
web-player/ the SvelteKit web client
|
||||||
deploy/ Docker Compose stacks — self-contained, no wrapper scripts
|
deploy/ Docker Compose stacks for dev and production
|
||||||
docs/ What each piece does and why (in Spanish)
|
docs/ architecture notes, one file per area
|
||||||
```
|
```
|
||||||
|
|
||||||
```sh
|
Postgres, Zitadel, SeaweedFS, RabbitMQ and Jaeger are not part of this
|
||||||
dotnet test backend/Musify.slnx # backend/Tests/ — Docker running (Testcontainers: Postgres, SeaweedFS)
|
layout. They live in the separate `Infrastructure` repository instead.
|
||||||
```
|
|
||||||
|
|
||||||
## Run it
|
## Digging deeper
|
||||||
|
|
||||||
Development (Docker Desktop):
|
[docs/](docs/README.md) covers how each piece works and why: architecture,
|
||||||
|
authentication, storage, streaming, media processing, and the local dev
|
||||||
```sh
|
setup. [deploy/README.md](deploy/README.md) has the full instructions for
|
||||||
docker compose -f deploy/compose.yml -f deploy/compose.dev.yml up -d
|
running the stack, in development or on a production server.
|
||||||
```
|
|
||||||
|
|
||||||
Infrastructure plus stream-ticket keys, database migrations and Zitadel OIDC
|
|
||||||
provisioning — all as one-shot jobs baked into the compose file. Apps run from
|
|
||||||
the IDE by default; add `--profile apps up -d --build` as a second command to
|
|
||||||
run them in Docker instead.
|
|
||||||
|
|
||||||
Production, on a server with a domain:
|
|
||||||
|
|
||||||
```sh
|
|
||||||
cp deploy/.env.prod.example deploy/.env.prod # fill in the domain and secrets
|
|
||||||
docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml up -d
|
|
||||||
docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml --profile apps up -d --build
|
|
||||||
```
|
|
||||||
|
|
||||||
Idempotent — run the same commands again to deploy an update. See
|
|
||||||
[deploy/README.md](deploy/README.md) for the details, the port map and the
|
|
||||||
production prerequisites.
|
|
||||||
|
|
||||||
## Components
|
|
||||||
|
|
||||||
| Service | Port (dev) | Role |
|
|
||||||
|---|---|---|
|
|
||||||
| `Musify.Api` | 5111 | REST API, issues presigned upload URLs and stream tickets |
|
|
||||||
| `Musify.StreamingGateway` | 8081 | YARP proxy that validates the stream ticket and serves the audio |
|
|
||||||
| `Musify.Worker` | — | MassTransit consumers: transcoding, thumbnails |
|
|
||||||
| `web-player` | 5173 / 3000 | Web client (SvelteKit, server-side OIDC session) |
|
|
||||||
| Zitadel | 8080 | Identity (OIDC/OAuth2) |
|
|
||||||
| PostgreSQL / RabbitMQ / SeaweedFS / Jaeger | 59000 / 5672 / 8333 / 16686 | Metadata, messaging, object storage, traces |
|
|
||||||
|
|
||||||
More in [docs/](docs/README.md): [architecture](docs/architecture.md),
|
|
||||||
[authentication](docs/authentication.md), [storage](docs/storage.md),
|
|
||||||
[streaming](docs/streaming.md), [media processing](docs/media-processing.md),
|
|
||||||
[development](docs/development.md).
|
|
||||||
|
|||||||
@@ -36,7 +36,7 @@ namespace Musify.Application.Albums
|
|||||||
|
|
||||||
var pagedEntities = await tracksQuery
|
var pagedEntities = await tracksQuery
|
||||||
.OrderBy(albumTrack => albumTrack.TrackNumber)
|
.OrderBy(albumTrack => albumTrack.TrackNumber)
|
||||||
.Select(albumTrack => new { albumTrack.Track, ListensCount = albumTrack.Track.ListeningHistories.Count })
|
.Select(albumTrack => new { albumTrack.Track, ListensCount = albumTrack.Track.ListeningHistories.Count(l => l.IsCounted) })
|
||||||
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
||||||
|
|
||||||
var pagedTracks = new StaticPagedList<TrackApplicationResponse>(
|
var pagedTracks = new StaticPagedList<TrackApplicationResponse>(
|
||||||
|
|||||||
@@ -16,7 +16,7 @@ namespace Musify.Application.Albums
|
|||||||
{
|
{
|
||||||
var listenedTrackIds = database.ListeningHistories
|
var listenedTrackIds = database.ListeningHistories
|
||||||
.AsNoTracking()
|
.AsNoTracking()
|
||||||
.Where(history => history.UserId == request.UserId);
|
.Where(history => history.UserId == request.UserId && history.IsCounted);
|
||||||
|
|
||||||
var ranked = await database.AlbumHasTracks
|
var ranked = await database.AlbumHasTracks
|
||||||
.AsNoTracking()
|
.AsNoTracking()
|
||||||
|
|||||||
@@ -0,0 +1,26 @@
|
|||||||
|
using Mediator;
|
||||||
|
using Musify.Domain.ValueObjects;
|
||||||
|
|
||||||
|
namespace Musify.Application.Genres
|
||||||
|
{
|
||||||
|
public record GetAvailableGenresQuery : IQuery<IReadOnlyList<AvailableGenreResponse>>;
|
||||||
|
|
||||||
|
public record AvailableGenreResponse(Genre Genre, IReadOnlyList<Genre> IncompatibleWith);
|
||||||
|
|
||||||
|
public class GetAvailableGenresQueryHandler
|
||||||
|
: IQueryHandler<GetAvailableGenresQuery, IReadOnlyList<AvailableGenreResponse>>
|
||||||
|
{
|
||||||
|
public ValueTask<IReadOnlyList<AvailableGenreResponse>> Handle(GetAvailableGenresQuery request, CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
var all = Enum.GetValues<Genre>();
|
||||||
|
|
||||||
|
IReadOnlyList<AvailableGenreResponse> result = all
|
||||||
|
.Select(genre => new AvailableGenreResponse(
|
||||||
|
genre,
|
||||||
|
all.Where(other => !GenreCompatibility.AreCompatible(genre, other)).ToList()))
|
||||||
|
.ToList();
|
||||||
|
|
||||||
|
return ValueTask.FromResult(result);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,31 @@
|
|||||||
|
using Mediator;
|
||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
|
using Musify.Application.Contracts;
|
||||||
|
using Musify.Domain.ValueObjects;
|
||||||
|
|
||||||
|
namespace Musify.Application.Genres
|
||||||
|
{
|
||||||
|
public record GetGenresQuery : IQuery<IReadOnlyList<GenreResponse>>;
|
||||||
|
|
||||||
|
public record GenreResponse(Genre Genre, int TrackCount);
|
||||||
|
|
||||||
|
public class GetGenresQueryHandler(IDatabase database)
|
||||||
|
: IQueryHandler<GetGenresQuery, IReadOnlyList<GenreResponse>>
|
||||||
|
{
|
||||||
|
public async ValueTask<IReadOnlyList<GenreResponse>> Handle(GetGenresQuery request, CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
var rows = await database.TrackTags
|
||||||
|
.AsNoTracking()
|
||||||
|
.Where(tag => database.UserHasTracks.Any(userTrack => userTrack.TrackId == tag.TrackId))
|
||||||
|
.GroupBy(tag => tag.Tag)
|
||||||
|
.Select(group => new { Genre = group.Key, TrackCount = group.Select(tag => tag.TrackId).Distinct().Count() })
|
||||||
|
.ToListAsync(cancellationToken);
|
||||||
|
|
||||||
|
return rows
|
||||||
|
.OrderByDescending(row => row.TrackCount)
|
||||||
|
.ThenBy(row => row.Genre)
|
||||||
|
.Select(row => new GenreResponse(row.Genre, row.TrackCount))
|
||||||
|
.ToList();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -30,7 +30,7 @@ namespace Musify.Application.Likes
|
|||||||
|
|
||||||
var pagedEntities = await likesQuery
|
var pagedEntities = await likesQuery
|
||||||
.OrderByDescending(like => like.CreatedAt)
|
.OrderByDescending(like => like.CreatedAt)
|
||||||
.Select(like => new { like.Track, ListensCount = like.Track.ListeningHistories.Count })
|
.Select(like => new { like.Track, ListensCount = like.Track.ListeningHistories.Count(l => l.IsCounted) })
|
||||||
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
||||||
|
|
||||||
var pagedTracks = new StaticPagedList<TrackApplicationResponse>(
|
var pagedTracks = new StaticPagedList<TrackApplicationResponse>(
|
||||||
|
|||||||
@@ -38,7 +38,7 @@ namespace Musify.Application.Mixes
|
|||||||
|
|
||||||
var historyTrackIds = await database.ListeningHistories
|
var historyTrackIds = await database.ListeningHistories
|
||||||
.AsNoTracking()
|
.AsNoTracking()
|
||||||
.Where(history => history.UserId == userId)
|
.Where(history => history.UserId == userId && history.IsCounted)
|
||||||
.Select(history => history.TrackId)
|
.Select(history => history.TrackId)
|
||||||
.Distinct()
|
.Distinct()
|
||||||
.ToListAsync(cancellationToken);
|
.ToListAsync(cancellationToken);
|
||||||
|
|||||||
@@ -19,7 +19,7 @@ namespace Musify.Application.Mixes
|
|||||||
.AsNoTracking()
|
.AsNoTracking()
|
||||||
.Include(track => track.Owner)
|
.Include(track => track.Owner)
|
||||||
.Where(track => trackIds.Contains(track.Id))
|
.Where(track => trackIds.Contains(track.Id))
|
||||||
.Select(track => new { Track = track, ListensCount = track.ListeningHistories.Count })
|
.Select(track => new { Track = track, ListensCount = track.ListeningHistories.Count(l => l.IsCounted) })
|
||||||
.ToDictionaryAsync(x => x.Track.Id, cancellationToken);
|
.ToDictionaryAsync(x => x.Track.Id, cancellationToken);
|
||||||
|
|
||||||
return orderedItems
|
return orderedItems
|
||||||
|
|||||||
@@ -0,0 +1,86 @@
|
|||||||
|
using ErrorOr;
|
||||||
|
using Mediator;
|
||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
|
using Microsoft.Extensions.Logging;
|
||||||
|
using Musify.Application.Contracts;
|
||||||
|
using Musify.Application.PlayLists.Responses;
|
||||||
|
using Musify.Domain.Entities;
|
||||||
|
|
||||||
|
namespace Musify.Application.PlayLists
|
||||||
|
{
|
||||||
|
public record AddAlbumToPlayListCommand(long UserId, Guid PlayListId, Guid AlbumId)
|
||||||
|
: ICommand<ErrorOr<AddAlbumToPlayListResponse>>;
|
||||||
|
|
||||||
|
public class AddAlbumToPlayListCommandHandler(
|
||||||
|
IDatabase database,
|
||||||
|
ILogger<AddAlbumToPlayListCommandHandler> logger)
|
||||||
|
: ICommandHandler<AddAlbumToPlayListCommand, ErrorOr<AddAlbumToPlayListResponse>>
|
||||||
|
{
|
||||||
|
public async ValueTask<ErrorOr<AddAlbumToPlayListResponse>> Handle(AddAlbumToPlayListCommand request, CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
var playList = await database.PlayLists
|
||||||
|
.SingleOrDefaultAsync(pl => pl.Id == request.PlayListId, cancellationToken);
|
||||||
|
if (playList == null)
|
||||||
|
{
|
||||||
|
logger.LogInformation("Playlist {PlayListId} not found", request.PlayListId);
|
||||||
|
return Error.NotFound();
|
||||||
|
}
|
||||||
|
|
||||||
|
if (playList.UserId != request.UserId)
|
||||||
|
{
|
||||||
|
logger.LogWarning("User {UserId} is not the owner of playlist {PlayListId}", request.UserId, request.PlayListId);
|
||||||
|
return Error.Unauthorized();
|
||||||
|
}
|
||||||
|
|
||||||
|
var albumExists = await database.Albums
|
||||||
|
.AsNoTracking()
|
||||||
|
.AnyAsync(album => album.Id == request.AlbumId, cancellationToken);
|
||||||
|
if (!albumExists)
|
||||||
|
{
|
||||||
|
logger.LogInformation("Album {AlbumId} not found", request.AlbumId);
|
||||||
|
return Error.NotFound(description: $"Album {request.AlbumId} not found");
|
||||||
|
}
|
||||||
|
|
||||||
|
var albumTrackIds = await database.AlbumHasTracks
|
||||||
|
.AsNoTracking()
|
||||||
|
.Where(albumTrack => albumTrack.AlbumId == request.AlbumId)
|
||||||
|
.OrderBy(albumTrack => albumTrack.TrackNumber)
|
||||||
|
.Select(albumTrack => albumTrack.TrackId)
|
||||||
|
.ToListAsync(cancellationToken);
|
||||||
|
|
||||||
|
var playListTracksQuery = database.PlayListHasTracks
|
||||||
|
.Where(plt => plt.PlayListId == request.PlayListId);
|
||||||
|
|
||||||
|
var existingTrackIds = (await playListTracksQuery
|
||||||
|
.Select(plt => plt.TrackId)
|
||||||
|
.ToListAsync(cancellationToken)).ToHashSet();
|
||||||
|
|
||||||
|
var newTrackIds = albumTrackIds
|
||||||
|
.Where(trackId => !existingTrackIds.Contains(trackId))
|
||||||
|
.ToList();
|
||||||
|
|
||||||
|
if (newTrackIds.Count == 0)
|
||||||
|
return new AddAlbumToPlayListResponse(0);
|
||||||
|
|
||||||
|
var nextPosition = existingTrackIds.Count > 0
|
||||||
|
? await playListTracksQuery.MaxAsync(plt => plt.Position, cancellationToken) + 1
|
||||||
|
: 0;
|
||||||
|
|
||||||
|
await database.PlayListHasTracks.AddRangeAsync(
|
||||||
|
newTrackIds.Select((trackId, index) => new PlayListHasTrack
|
||||||
|
{
|
||||||
|
PlayListId = request.PlayListId,
|
||||||
|
TrackId = trackId,
|
||||||
|
Position = nextPosition + index
|
||||||
|
}),
|
||||||
|
cancellationToken);
|
||||||
|
|
||||||
|
await database.SaveChangesAsync(cancellationToken);
|
||||||
|
|
||||||
|
logger.LogInformation("Added {Count} tracks from album {AlbumId} to playlist {PlayListId}",
|
||||||
|
newTrackIds.Count, request.AlbumId, request.PlayListId);
|
||||||
|
|
||||||
|
return new AddAlbumToPlayListResponse(newTrackIds.Count);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -21,6 +21,8 @@ namespace Musify.Application.PlayLists
|
|||||||
.Select(p => new
|
.Select(p => new
|
||||||
{
|
{
|
||||||
PlayList = p,
|
PlayList = p,
|
||||||
|
TrackCount = p.PlayListTracks.Count,
|
||||||
|
DurationSeconds = p.PlayListTracks.Sum(plt => (double?)plt.Track.DurationSeconds) ?? 0,
|
||||||
CoverTrackIds = p.PlayListTracks
|
CoverTrackIds = p.PlayListTracks
|
||||||
.OrderBy(plt => plt.Position)
|
.OrderBy(plt => plt.Position)
|
||||||
.Take(PlayListApplicationResponse.CoverTrackCount)
|
.Take(PlayListApplicationResponse.CoverTrackCount)
|
||||||
@@ -36,7 +38,7 @@ namespace Musify.Application.PlayLists
|
|||||||
&& entry.PlayList.UserId != request.RequestingUserId)
|
&& entry.PlayList.UserId != request.RequestingUserId)
|
||||||
return Error.NotFound();
|
return Error.NotFound();
|
||||||
|
|
||||||
return PlayListApplicationResponse.FromEntity(entry.PlayList, entry.CoverTrackIds);
|
return PlayListApplicationResponse.FromEntity(entry.PlayList, entry.TrackCount, entry.DurationSeconds, entry.CoverTrackIds);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -36,7 +36,7 @@ namespace Musify.Application.PlayLists
|
|||||||
|
|
||||||
var pagedEntities = await tracksQuery
|
var pagedEntities = await tracksQuery
|
||||||
.OrderBy(plt => plt.Position)
|
.OrderBy(plt => plt.Position)
|
||||||
.Select(plt => new { plt.Track, ListensCount = plt.Track.ListeningHistories.Count })
|
.Select(plt => new { plt.Track, ListensCount = plt.Track.ListeningHistories.Count(l => l.IsCounted) })
|
||||||
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
||||||
|
|
||||||
var pagedTracks = new StaticPagedList<TrackApplicationResponse>(
|
var pagedTracks = new StaticPagedList<TrackApplicationResponse>(
|
||||||
|
|||||||
@@ -25,6 +25,8 @@ namespace Musify.Application.PlayLists
|
|||||||
.Select(p => new
|
.Select(p => new
|
||||||
{
|
{
|
||||||
PlayList = p,
|
PlayList = p,
|
||||||
|
TrackCount = p.PlayListTracks.Count,
|
||||||
|
DurationSeconds = p.PlayListTracks.Sum(plt => (double?)plt.Track.DurationSeconds) ?? 0,
|
||||||
CoverTrackIds = p.PlayListTracks
|
CoverTrackIds = p.PlayListTracks
|
||||||
.OrderBy(plt => plt.Position)
|
.OrderBy(plt => plt.Position)
|
||||||
.Take(PlayListApplicationResponse.CoverTrackCount)
|
.Take(PlayListApplicationResponse.CoverTrackCount)
|
||||||
@@ -34,7 +36,7 @@ namespace Musify.Application.PlayLists
|
|||||||
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
||||||
|
|
||||||
var pagedPlayLists = new StaticPagedList<PlayListApplicationResponse>(
|
var pagedPlayLists = new StaticPagedList<PlayListApplicationResponse>(
|
||||||
pagedEntities.Select(entry => PlayListApplicationResponse.FromEntity(entry.PlayList, entry.CoverTrackIds)),
|
pagedEntities.Select(entry => PlayListApplicationResponse.FromEntity(entry.PlayList, entry.TrackCount, entry.DurationSeconds, entry.CoverTrackIds)),
|
||||||
pagedEntities.PageNumber, pagedEntities.PageSize, pagedEntities.TotalItemCount);
|
pagedEntities.PageNumber, pagedEntities.PageSize, pagedEntities.TotalItemCount);
|
||||||
|
|
||||||
return PaginatedResponse<PlayListApplicationResponse>.FromPagedList(pagedPlayLists);
|
return PaginatedResponse<PlayListApplicationResponse>.FromPagedList(pagedPlayLists);
|
||||||
|
|||||||
@@ -46,6 +46,8 @@ namespace Musify.Application.PlayLists
|
|||||||
.Select(p => new
|
.Select(p => new
|
||||||
{
|
{
|
||||||
PlayList = p,
|
PlayList = p,
|
||||||
|
TrackCount = p.PlayListTracks.Count,
|
||||||
|
DurationSeconds = p.PlayListTracks.Sum(plt => (double?)plt.Track.DurationSeconds) ?? 0,
|
||||||
CoverTrackIds = p.PlayListTracks
|
CoverTrackIds = p.PlayListTracks
|
||||||
.OrderBy(plt => plt.Position)
|
.OrderBy(plt => plt.Position)
|
||||||
.Take(PlayListApplicationResponse.CoverTrackCount)
|
.Take(PlayListApplicationResponse.CoverTrackCount)
|
||||||
@@ -55,7 +57,7 @@ namespace Musify.Application.PlayLists
|
|||||||
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
||||||
|
|
||||||
var pagedPlayLists = new StaticPagedList<PlayListApplicationResponse>(
|
var pagedPlayLists = new StaticPagedList<PlayListApplicationResponse>(
|
||||||
pagedEntities.Select(entry => PlayListApplicationResponse.FromEntity(entry.PlayList, entry.CoverTrackIds)),
|
pagedEntities.Select(entry => PlayListApplicationResponse.FromEntity(entry.PlayList, entry.TrackCount, entry.DurationSeconds, entry.CoverTrackIds)),
|
||||||
pagedEntities.PageNumber, pagedEntities.PageSize, pagedEntities.TotalItemCount);
|
pagedEntities.PageNumber, pagedEntities.PageSize, pagedEntities.TotalItemCount);
|
||||||
|
|
||||||
return PaginatedResponse<PlayListApplicationResponse>.FromPagedList(pagedPlayLists);
|
return PaginatedResponse<PlayListApplicationResponse>.FromPagedList(pagedPlayLists);
|
||||||
|
|||||||
@@ -0,0 +1,4 @@
|
|||||||
|
namespace Musify.Application.PlayLists.Responses
|
||||||
|
{
|
||||||
|
public record AddAlbumToPlayListResponse(int AddedCount);
|
||||||
|
}
|
||||||
@@ -1,3 +1,5 @@
|
|||||||
|
using System.Text.Json.Serialization;
|
||||||
|
using Musify.Application.Serialization;
|
||||||
using Musify.Domain.Entities;
|
using Musify.Domain.Entities;
|
||||||
using Musify.Domain.ValueObjects;
|
using Musify.Domain.ValueObjects;
|
||||||
|
|
||||||
@@ -10,7 +12,10 @@ namespace Musify.Application.PlayLists.Responses
|
|||||||
string? SmallImageKeyName,
|
string? SmallImageKeyName,
|
||||||
string? MediumImageKeyName,
|
string? MediumImageKeyName,
|
||||||
string? LargeImageKeyName,
|
string? LargeImageKeyName,
|
||||||
|
[property: JsonConverter(typeof(LongAsStringConverter))] long OwnerUserId,
|
||||||
PlaylistVisibility Visibility,
|
PlaylistVisibility Visibility,
|
||||||
|
int TrackCount,
|
||||||
|
double DurationSeconds,
|
||||||
DateTime CreatedAt,
|
DateTime CreatedAt,
|
||||||
DateTime UpdatedAt,
|
DateTime UpdatedAt,
|
||||||
IReadOnlyList<Guid> CoverTrackIds)
|
IReadOnlyList<Guid> CoverTrackIds)
|
||||||
@@ -19,6 +24,8 @@ namespace Musify.Application.PlayLists.Responses
|
|||||||
|
|
||||||
public static PlayListApplicationResponse FromEntity(
|
public static PlayListApplicationResponse FromEntity(
|
||||||
PlayList playList,
|
PlayList playList,
|
||||||
|
int trackCount = 0,
|
||||||
|
double durationSeconds = 0,
|
||||||
IReadOnlyList<Guid>? coverTrackIds = null)
|
IReadOnlyList<Guid>? coverTrackIds = null)
|
||||||
{
|
{
|
||||||
return new PlayListApplicationResponse(
|
return new PlayListApplicationResponse(
|
||||||
@@ -28,7 +35,10 @@ namespace Musify.Application.PlayLists.Responses
|
|||||||
playList.Pictures?.SmallName,
|
playList.Pictures?.SmallName,
|
||||||
playList.Pictures?.MediumName,
|
playList.Pictures?.MediumName,
|
||||||
playList.Pictures?.LargeName,
|
playList.Pictures?.LargeName,
|
||||||
|
playList.UserId,
|
||||||
playList.Visibility,
|
playList.Visibility,
|
||||||
|
trackCount,
|
||||||
|
durationSeconds,
|
||||||
playList.CreatedAt,
|
playList.CreatedAt,
|
||||||
playList.UpdatedAt,
|
playList.UpdatedAt,
|
||||||
coverTrackIds ?? []);
|
coverTrackIds ?? []);
|
||||||
|
|||||||
@@ -112,7 +112,22 @@ namespace Musify.Application.PlayLists
|
|||||||
.ToListAsync(cancellationToken);
|
.ToListAsync(cancellationToken);
|
||||||
|
|
||||||
logger.LogInformation("Updated playlist {PlayListId}", playListEntity.Id);
|
logger.LogInformation("Updated playlist {PlayListId}", playListEntity.Id);
|
||||||
return PlayListApplicationResponse.FromEntity(playListEntity, coverTrackIds);
|
var totals = await database.PlayListHasTracks
|
||||||
|
.AsNoTracking()
|
||||||
|
.Where(plt => plt.PlayListId == playListEntity.Id)
|
||||||
|
.GroupBy(plt => plt.PlayListId)
|
||||||
|
.Select(group => new
|
||||||
|
{
|
||||||
|
TrackCount = group.Count(),
|
||||||
|
DurationSeconds = group.Sum(plt => plt.Track.DurationSeconds)
|
||||||
|
})
|
||||||
|
.SingleOrDefaultAsync(cancellationToken);
|
||||||
|
|
||||||
|
return PlayListApplicationResponse.FromEntity(
|
||||||
|
playListEntity,
|
||||||
|
totals?.TrackCount ?? 0,
|
||||||
|
totals?.DurationSeconds ?? 0,
|
||||||
|
coverTrackIds);
|
||||||
}
|
}
|
||||||
|
|
||||||
private async Task<ErrorOr<Success>> PublishUpdatePlayListPictureSourceEventAsync(
|
private async Task<ErrorOr<Success>> PublishUpdatePlayListPictureSourceEventAsync(
|
||||||
|
|||||||
@@ -30,6 +30,7 @@ public sealed class TrackStreamIssuer(
|
|||||||
folderPath,
|
folderPath,
|
||||||
streamGatewayConfiguration.AudioFileName);
|
streamGatewayConfiguration.AudioFileName);
|
||||||
|
|
||||||
|
Guid? listenId = null;
|
||||||
if (userId != null)
|
if (userId != null)
|
||||||
{
|
{
|
||||||
var newListeningHistory = new ListeningHistory
|
var newListeningHistory = new ListeningHistory
|
||||||
@@ -39,9 +40,10 @@ public sealed class TrackStreamIssuer(
|
|||||||
};
|
};
|
||||||
await database.ListeningHistories.AddAsync(newListeningHistory, cancellationToken);
|
await database.ListeningHistories.AddAsync(newListeningHistory, cancellationToken);
|
||||||
await database.SaveChangesAsync(cancellationToken);
|
await database.SaveChangesAsync(cancellationToken);
|
||||||
|
listenId = newListeningHistory.Id;
|
||||||
}
|
}
|
||||||
|
|
||||||
return new TrackStreamResponse(manifestUrl, ticket.Token, ticket.ExpiresInSeconds);
|
return new TrackStreamResponse(manifestUrl, ticket.Token, ticket.ExpiresInSeconds, listenId);
|
||||||
}
|
}
|
||||||
|
|
||||||
private long? AnonymousFragmentBytes(long? userId)
|
private long? AnonymousFragmentBytes(long? userId)
|
||||||
|
|||||||
@@ -18,7 +18,8 @@ namespace Musify.Application.Tracks
|
|||||||
string Title,
|
string Title,
|
||||||
Guid PictureIntentId,
|
Guid PictureIntentId,
|
||||||
Guid AudioIntentId,
|
Guid AudioIntentId,
|
||||||
IReadOnlyCollection<Genre> Tags)
|
IReadOnlyCollection<Genre> Tags,
|
||||||
|
bool IsExplicit = false)
|
||||||
: ICommand<ErrorOr<TrackApplicationResponse>>;
|
: ICommand<ErrorOr<TrackApplicationResponse>>;
|
||||||
|
|
||||||
public class CreateTrackCommandHandler(
|
public class CreateTrackCommandHandler(
|
||||||
@@ -83,6 +84,7 @@ namespace Musify.Application.Tracks
|
|||||||
NormalizedTitle = request.Title.ToUpperInvariant(),
|
NormalizedTitle = request.Title.ToUpperInvariant(),
|
||||||
OwnerUserId = user.Id,
|
OwnerUserId = user.Id,
|
||||||
Owner = user,
|
Owner = user,
|
||||||
|
IsExplicit = request.IsExplicit,
|
||||||
Pictures = new TrackPictures
|
Pictures = new TrackPictures
|
||||||
{
|
{
|
||||||
OriginalName = pictureIntent.ObjectName,
|
OriginalName = pictureIntent.ObjectName,
|
||||||
|
|||||||
@@ -0,0 +1,33 @@
|
|||||||
|
using Mediator;
|
||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
|
using Microsoft.Extensions.Logging;
|
||||||
|
using Musify.Application.Contracts;
|
||||||
|
|
||||||
|
namespace Musify.Application.Tracks
|
||||||
|
{
|
||||||
|
public record DeleteStaleUncountedListensCommand : ICommand<int>;
|
||||||
|
|
||||||
|
public class DeleteStaleUncountedListensCommandHandler(
|
||||||
|
IDatabase database,
|
||||||
|
ILogger<DeleteStaleUncountedListensCommandHandler> logger)
|
||||||
|
: ICommandHandler<DeleteStaleUncountedListensCommand, int>
|
||||||
|
{
|
||||||
|
public async ValueTask<int> Handle(DeleteStaleUncountedListensCommand request, CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
var cutoff = DateTime.UtcNow.AddDays(-ListeningRules.UncountedRetentionDays);
|
||||||
|
|
||||||
|
var stale = await database.ListeningHistories
|
||||||
|
.Where(l => !l.IsCounted && l.ListenedAt < cutoff)
|
||||||
|
.ToListAsync(cancellationToken);
|
||||||
|
|
||||||
|
if (stale.Count == 0)
|
||||||
|
return 0;
|
||||||
|
|
||||||
|
database.ListeningHistories.RemoveRange(stale);
|
||||||
|
await database.SaveChangesAsync(cancellationToken);
|
||||||
|
|
||||||
|
logger.LogInformation("Deleted {Count} uncounted listens older than {Days} days", stale.Count, ListeningRules.UncountedRetentionDays);
|
||||||
|
return stale.Count;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -19,7 +19,7 @@ namespace Musify.Application.Tracks
|
|||||||
.Include(t => t.Track.Tags)
|
.Include(t => t.Track.Tags)
|
||||||
.Where(t => t.UserId == query.UserId)
|
.Where(t => t.UserId == query.UserId)
|
||||||
.OrderByDescending(t => t.ListenedAt)
|
.OrderByDescending(t => t.ListenedAt)
|
||||||
.Select(t => TrackApplicationResponse.FromEntity(t.Track, t.Track.ListeningHistories.Count))
|
.Select(t => TrackApplicationResponse.FromEntity(t.Track, t.Track.ListeningHistories.Count(l => l.IsCounted)))
|
||||||
.FirstOrDefaultAsync(cancellationToken);
|
.FirstOrDefaultAsync(cancellationToken);
|
||||||
|
|
||||||
return track == null
|
return track == null
|
||||||
|
|||||||
@@ -17,7 +17,7 @@ namespace Musify.Application.Tracks
|
|||||||
{
|
{
|
||||||
var recentTrackIds = await database.ListeningHistories
|
var recentTrackIds = await database.ListeningHistories
|
||||||
.AsNoTracking()
|
.AsNoTracking()
|
||||||
.Where(l => l.UserId == query.UserId)
|
.Where(l => l.UserId == query.UserId && l.IsCounted)
|
||||||
.GroupBy(l => l.TrackId)
|
.GroupBy(l => l.TrackId)
|
||||||
.OrderByDescending(g => g.Max(l => l.ListenedAt))
|
.OrderByDescending(g => g.Max(l => l.ListenedAt))
|
||||||
.Select(g => g.Key)
|
.Select(g => g.Key)
|
||||||
@@ -29,7 +29,7 @@ namespace Musify.Application.Tracks
|
|||||||
.Include(t => t.Owner)
|
.Include(t => t.Owner)
|
||||||
.Include(t => t.Tags)
|
.Include(t => t.Tags)
|
||||||
.Where(t => recentTrackIds.Contains(t.Id))
|
.Where(t => recentTrackIds.Contains(t.Id))
|
||||||
.Select(t => new { t.Id, Track = t, ListensCount = t.ListeningHistories.Count })
|
.Select(t => new { t.Id, Track = t, ListensCount = t.ListeningHistories.Count(l => l.IsCounted) })
|
||||||
.ToDictionaryAsync(t => t.Id, cancellationToken);
|
.ToDictionaryAsync(t => t.Id, cancellationToken);
|
||||||
|
|
||||||
return recentTrackIds
|
return recentTrackIds
|
||||||
|
|||||||
@@ -18,7 +18,7 @@ namespace Musify.Application.Tracks
|
|||||||
.Include(t => t.Owner)
|
.Include(t => t.Owner)
|
||||||
.Include(t => t.Tags)
|
.Include(t => t.Tags)
|
||||||
.Where(t => t.Id == request.TrackId)
|
.Where(t => t.Id == request.TrackId)
|
||||||
.Select(t => new { Track = t, ListensCount = t.ListeningHistories.Count })
|
.Select(t => new { Track = t, ListensCount = t.ListeningHistories.Count(l => l.IsCounted) })
|
||||||
.SingleOrDefaultAsync(cancellationToken);
|
.SingleOrDefaultAsync(cancellationToken);
|
||||||
|
|
||||||
if (entity == null)
|
if (entity == null)
|
||||||
|
|||||||
@@ -5,10 +5,11 @@ using Musify.Application.Tracks.Responses;
|
|||||||
using X.PagedList;
|
using X.PagedList;
|
||||||
using X.PagedList.EF;
|
using X.PagedList.EF;
|
||||||
using Musify.Application.Contracts;
|
using Musify.Application.Contracts;
|
||||||
|
using Musify.Domain.ValueObjects;
|
||||||
|
|
||||||
namespace Musify.Application.Tracks
|
namespace Musify.Application.Tracks
|
||||||
{
|
{
|
||||||
public record GetTracksQuery(string? Name, int PageNumber, int PageSize)
|
public record GetTracksQuery(string? Name, int PageNumber, int PageSize, Genre? Genre = null)
|
||||||
: IQuery<ErrorOr<TracksSearchResponse>>;
|
: IQuery<ErrorOr<TracksSearchResponse>>;
|
||||||
|
|
||||||
public class GetTracksQueryHandler(IDatabase database)
|
public class GetTracksQueryHandler(IDatabase database)
|
||||||
@@ -28,12 +29,15 @@ namespace Musify.Application.Tracks
|
|||||||
tracksQuery = tracksQuery.Where(t => t.Track.NormalizedTitle.Contains(normalizedName));
|
tracksQuery = tracksQuery.Where(t => t.Track.NormalizedTitle.Contains(normalizedName));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (request.Genre is { } genre)
|
||||||
|
tracksQuery = tracksQuery.Where(t => t.Track.Tags.Any(tag => tag.Tag == genre));
|
||||||
|
|
||||||
var totalCount = await tracksQuery.CountAsync(cancellationToken);
|
var totalCount = await tracksQuery.CountAsync(cancellationToken);
|
||||||
|
|
||||||
var pagedEntities = await tracksQuery
|
var pagedEntities = await tracksQuery
|
||||||
.OrderByDescending(t => t.Track.CreatedAt)
|
.OrderByDescending(t => t.Track.CreatedAt)
|
||||||
.ThenBy(t => t.Id)
|
.ThenBy(t => t.Id)
|
||||||
.Select(t => new { t.Track, ListensCount = t.Track.ListeningHistories.Count })
|
.Select(t => new { t.Track, ListensCount = t.Track.ListeningHistories.Count(l => l.IsCounted) })
|
||||||
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
||||||
|
|
||||||
var items = pagedEntities
|
var items = pagedEntities
|
||||||
|
|||||||
@@ -40,7 +40,7 @@ namespace Musify.Application.Tracks
|
|||||||
|
|
||||||
var pagedEntities = await tracksQuery
|
var pagedEntities = await tracksQuery
|
||||||
.OrderBy(t => t.Track.CreatedAt)
|
.OrderBy(t => t.Track.CreatedAt)
|
||||||
.Select(t => new { t.Track, ListensCount = t.Track.ListeningHistories.Count })
|
.Select(t => new { t.Track, ListensCount = t.Track.ListeningHistories.Count(l => l.IsCounted) })
|
||||||
.ToPagedListAsync(pageNumber, pageSize, totalCount, cancellationToken);
|
.ToPagedListAsync(pageNumber, pageSize, totalCount, cancellationToken);
|
||||||
|
|
||||||
var pagedTracks = new StaticPagedList<TrackApplicationResponse>(
|
var pagedTracks = new StaticPagedList<TrackApplicationResponse>(
|
||||||
|
|||||||
@@ -0,0 +1,14 @@
|
|||||||
|
namespace Musify.Application.Tracks
|
||||||
|
{
|
||||||
|
public static class ListeningRules
|
||||||
|
{
|
||||||
|
public const double CountedThresholdSeconds = 30;
|
||||||
|
public const double CountedThresholdFraction = 0.5;
|
||||||
|
public const double DurationToleranceSeconds = 2;
|
||||||
|
public const double ClockToleranceSeconds = 5;
|
||||||
|
public const int UncountedRetentionDays = 8;
|
||||||
|
|
||||||
|
public static double CountedThreshold(double durationSeconds) =>
|
||||||
|
Math.Min(CountedThresholdSeconds, durationSeconds * CountedThresholdFraction);
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,44 @@
|
|||||||
|
using ErrorOr;
|
||||||
|
using Mediator;
|
||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
|
using Musify.Application.Contracts;
|
||||||
|
|
||||||
|
namespace Musify.Application.Tracks
|
||||||
|
{
|
||||||
|
public record RecordListeningProgressCommand(
|
||||||
|
long UserId,
|
||||||
|
Guid ListenId,
|
||||||
|
double PlayedSeconds)
|
||||||
|
: ICommand<ErrorOr<Success>>;
|
||||||
|
|
||||||
|
public class RecordListeningProgressCommandHandler(IDatabase database)
|
||||||
|
: ICommandHandler<RecordListeningProgressCommand, ErrorOr<Success>>
|
||||||
|
{
|
||||||
|
public async ValueTask<ErrorOr<Success>> Handle(RecordListeningProgressCommand request, CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
if (!double.IsFinite(request.PlayedSeconds) || request.PlayedSeconds < 0)
|
||||||
|
return Error.Validation(description: "playedSeconds must be a non-negative number.");
|
||||||
|
|
||||||
|
var listen = await database.ListeningHistories
|
||||||
|
.Include(l => l.Track)
|
||||||
|
.SingleOrDefaultAsync(l => l.Id == request.ListenId && l.UserId == request.UserId, cancellationToken);
|
||||||
|
|
||||||
|
if (listen == null)
|
||||||
|
return Error.NotFound();
|
||||||
|
|
||||||
|
var now = DateTime.UtcNow;
|
||||||
|
var durationCap = listen.Track.DurationSeconds + ListeningRules.DurationToleranceSeconds;
|
||||||
|
var clockCap = (now - listen.ListenedAt).TotalSeconds + ListeningRules.ClockToleranceSeconds;
|
||||||
|
var accepted = Math.Min(request.PlayedSeconds, Math.Min(durationCap, clockCap));
|
||||||
|
|
||||||
|
listen.PlayedSeconds = Math.Max(listen.PlayedSeconds ?? 0, accepted);
|
||||||
|
listen.LastProgressAt = now;
|
||||||
|
listen.IsCounted = listen.IsCounted
|
||||||
|
|| listen.PlayedSeconds >= ListeningRules.CountedThreshold(listen.Track.DurationSeconds);
|
||||||
|
|
||||||
|
await database.SaveChangesAsync(cancellationToken);
|
||||||
|
|
||||||
|
return Result.Success;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -15,7 +15,8 @@ namespace Musify.Application.Tracks.Responses
|
|||||||
DateTime CreatedAt,
|
DateTime CreatedAt,
|
||||||
DateTime UpdatedAt,
|
DateTime UpdatedAt,
|
||||||
[property: JsonConverter(typeof(LongAsStringConverter))] long OwnerUserId,
|
[property: JsonConverter(typeof(LongAsStringConverter))] long OwnerUserId,
|
||||||
IReadOnlyCollection<Genre> Tags)
|
IReadOnlyCollection<Genre> Tags,
|
||||||
|
bool IsExplicit)
|
||||||
{
|
{
|
||||||
public static TrackApplicationResponse FromEntity(Track track, int listensCount) =>
|
public static TrackApplicationResponse FromEntity(Track track, int listensCount) =>
|
||||||
new(
|
new(
|
||||||
@@ -28,6 +29,7 @@ namespace Musify.Application.Tracks.Responses
|
|||||||
track.CreatedAt,
|
track.CreatedAt,
|
||||||
track.UpdatedAt,
|
track.UpdatedAt,
|
||||||
track.OwnerUserId,
|
track.OwnerUserId,
|
||||||
track.Tags.Select(tag => tag.Tag).ToList());
|
track.Tags.Select(tag => tag.Tag).ToList(),
|
||||||
|
track.IsExplicit);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -3,5 +3,6 @@ namespace Musify.Application.Tracks.Responses
|
|||||||
public record TrackStreamResponse(
|
public record TrackStreamResponse(
|
||||||
string ManifestUrl,
|
string ManifestUrl,
|
||||||
string Ticket,
|
string Ticket,
|
||||||
int ExpiresInSeconds);
|
int ExpiresInSeconds,
|
||||||
|
Guid? ListenId = null);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,30 @@
|
|||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
|
using Musify.Application.Contracts;
|
||||||
|
|
||||||
|
namespace Musify.Application.Users
|
||||||
|
{
|
||||||
|
public static class FollowVisibility
|
||||||
|
{
|
||||||
|
public static async Task<bool> CanViewFollowersAsync(
|
||||||
|
IDatabase database,
|
||||||
|
long? viewerId,
|
||||||
|
long userId,
|
||||||
|
CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
if (viewerId is not { } viewer)
|
||||||
|
return false;
|
||||||
|
|
||||||
|
if (viewer == userId)
|
||||||
|
return true;
|
||||||
|
|
||||||
|
var mutualRows = await database.UserFollows
|
||||||
|
.AsNoTracking()
|
||||||
|
.CountAsync(f =>
|
||||||
|
(f.FollowerId == viewer && f.FollowedId == userId) ||
|
||||||
|
(f.FollowerId == userId && f.FollowedId == viewer),
|
||||||
|
cancellationToken);
|
||||||
|
|
||||||
|
return mutualRows == 2;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -20,17 +20,18 @@ namespace Musify.Application.Users
|
|||||||
.Where(l => l.UserId == request.UserId && l.ListenedAt >= weekAgo);
|
.Where(l => l.UserId == request.UserId && l.ListenedAt >= weekAgo);
|
||||||
|
|
||||||
var tracksThisWeek = await thisWeekHistory
|
var tracksThisWeek = await thisWeekHistory
|
||||||
|
.Where(l => l.IsCounted)
|
||||||
.Select(l => l.TrackId)
|
.Select(l => l.TrackId)
|
||||||
.Distinct()
|
.Distinct()
|
||||||
.CountAsync(cancellationToken);
|
.CountAsync(cancellationToken);
|
||||||
|
|
||||||
var secondsThisWeek = await thisWeekHistory
|
var secondsThisWeek = await thisWeekHistory
|
||||||
.Join(database.Tracks.AsNoTracking(), l => l.TrackId, t => t.Id, (l, t) => t.DurationSeconds)
|
.Join(database.Tracks.AsNoTracking(), l => l.TrackId, t => t.Id, (l, t) => l.PlayedSeconds ?? (l.IsCounted ? t.DurationSeconds : 0))
|
||||||
.SumAsync(cancellationToken);
|
.SumAsync(cancellationToken);
|
||||||
|
|
||||||
var listenedDates = await database.ListeningHistories
|
var listenedDates = await database.ListeningHistories
|
||||||
.AsNoTracking()
|
.AsNoTracking()
|
||||||
.Where(l => l.UserId == request.UserId)
|
.Where(l => l.UserId == request.UserId && l.IsCounted)
|
||||||
.Select(l => l.ListenedAt.Date)
|
.Select(l => l.ListenedAt.Date)
|
||||||
.Distinct()
|
.Distinct()
|
||||||
.ToListAsync(cancellationToken);
|
.ToListAsync(cancellationToken);
|
||||||
|
|||||||
@@ -0,0 +1,46 @@
|
|||||||
|
using ErrorOr;
|
||||||
|
using Mediator;
|
||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
|
using Musify.Application.Contracts;
|
||||||
|
using Musify.Application.Shared;
|
||||||
|
using Musify.Application.Users.Responses;
|
||||||
|
using X.PagedList.EF;
|
||||||
|
|
||||||
|
namespace Musify.Application.Users
|
||||||
|
{
|
||||||
|
public record GetUserFollowersQuery(long? ViewerId, long UserId, int PageNumber, int PageSize)
|
||||||
|
: IQuery<ErrorOr<PaginatedResponse<UserSummaryResponse>>>;
|
||||||
|
|
||||||
|
public class GetUserFollowersQueryHandler(IDatabase database)
|
||||||
|
: IQueryHandler<GetUserFollowersQuery, ErrorOr<PaginatedResponse<UserSummaryResponse>>>
|
||||||
|
{
|
||||||
|
public async ValueTask<ErrorOr<PaginatedResponse<UserSummaryResponse>>> Handle(GetUserFollowersQuery request, CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
if (!await database.Users.AnyAsync(u => u.Id == request.UserId, cancellationToken))
|
||||||
|
return Error.NotFound();
|
||||||
|
|
||||||
|
if (!await FollowVisibility.CanViewFollowersAsync(database, request.ViewerId, request.UserId, cancellationToken))
|
||||||
|
return Error.Forbidden();
|
||||||
|
|
||||||
|
var followsQuery = database.UserFollows
|
||||||
|
.AsNoTracking()
|
||||||
|
.Where(f => f.FollowedId == request.UserId);
|
||||||
|
|
||||||
|
var totalCount = await followsQuery.CountAsync(cancellationToken);
|
||||||
|
var viewerId = request.ViewerId;
|
||||||
|
|
||||||
|
var page = await followsQuery
|
||||||
|
.OrderByDescending(f => f.CreatedAt)
|
||||||
|
.Select(f => new UserSummaryResponse(
|
||||||
|
f.Follower.Id,
|
||||||
|
f.Follower.Name,
|
||||||
|
f.Follower.FirstName,
|
||||||
|
f.Follower.SecondName,
|
||||||
|
f.Follower.ProfilePictureUrl,
|
||||||
|
viewerId != null && database.UserFollows.Any(v => v.FollowerId == viewerId && v.FollowedId == f.FollowerId)))
|
||||||
|
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
||||||
|
|
||||||
|
return PaginatedResponse<UserSummaryResponse>.FromPagedList(page);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,43 @@
|
|||||||
|
using ErrorOr;
|
||||||
|
using Mediator;
|
||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
|
using Musify.Application.Contracts;
|
||||||
|
using Musify.Application.Shared;
|
||||||
|
using Musify.Application.Users.Responses;
|
||||||
|
using X.PagedList.EF;
|
||||||
|
|
||||||
|
namespace Musify.Application.Users
|
||||||
|
{
|
||||||
|
public record GetUserFollowingQuery(long? ViewerId, long UserId, int PageNumber, int PageSize)
|
||||||
|
: IQuery<ErrorOr<PaginatedResponse<UserSummaryResponse>>>;
|
||||||
|
|
||||||
|
public class GetUserFollowingQueryHandler(IDatabase database)
|
||||||
|
: IQueryHandler<GetUserFollowingQuery, ErrorOr<PaginatedResponse<UserSummaryResponse>>>
|
||||||
|
{
|
||||||
|
public async ValueTask<ErrorOr<PaginatedResponse<UserSummaryResponse>>> Handle(GetUserFollowingQuery request, CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
if (!await database.Users.AnyAsync(u => u.Id == request.UserId, cancellationToken))
|
||||||
|
return Error.NotFound();
|
||||||
|
|
||||||
|
var followsQuery = database.UserFollows
|
||||||
|
.AsNoTracking()
|
||||||
|
.Where(f => f.FollowerId == request.UserId);
|
||||||
|
|
||||||
|
var totalCount = await followsQuery.CountAsync(cancellationToken);
|
||||||
|
var viewerId = request.ViewerId;
|
||||||
|
|
||||||
|
var page = await followsQuery
|
||||||
|
.OrderByDescending(f => f.CreatedAt)
|
||||||
|
.Select(f => new UserSummaryResponse(
|
||||||
|
f.Followed.Id,
|
||||||
|
f.Followed.Name,
|
||||||
|
f.Followed.FirstName,
|
||||||
|
f.Followed.SecondName,
|
||||||
|
f.Followed.ProfilePictureUrl,
|
||||||
|
viewerId != null && database.UserFollows.Any(v => v.FollowerId == viewerId && v.FollowedId == f.FollowedId)))
|
||||||
|
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
||||||
|
|
||||||
|
return PaginatedResponse<UserSummaryResponse>.FromPagedList(page);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -7,7 +7,7 @@ using Musify.Application.Serialization;
|
|||||||
|
|
||||||
namespace Musify.Application.Users
|
namespace Musify.Application.Users
|
||||||
{
|
{
|
||||||
public record GetUserProfileQuery(long UserId) : IQuery<ErrorOr<UserProfileResponse>>;
|
public record GetUserProfileQuery(long UserId, long? ViewerId = null) : IQuery<ErrorOr<UserProfileResponse>>;
|
||||||
|
|
||||||
public record UserProfileResponse(
|
public record UserProfileResponse(
|
||||||
[property: JsonConverter(typeof(LongAsStringConverter))] long Id,
|
[property: JsonConverter(typeof(LongAsStringConverter))] long Id,
|
||||||
@@ -17,7 +17,9 @@ namespace Musify.Application.Users
|
|||||||
string? ProfilePictureUrl,
|
string? ProfilePictureUrl,
|
||||||
DateTime CreatedAt,
|
DateTime CreatedAt,
|
||||||
int FollowersCount,
|
int FollowersCount,
|
||||||
int FollowingCount);
|
int FollowingCount,
|
||||||
|
bool IsFollowing,
|
||||||
|
bool CanViewFollowers);
|
||||||
|
|
||||||
public class GetUserProfileQueryHandler(IDatabase database)
|
public class GetUserProfileQueryHandler(IDatabase database)
|
||||||
: IQueryHandler<GetUserProfileQuery, ErrorOr<UserProfileResponse>>
|
: IQueryHandler<GetUserProfileQuery, ErrorOr<UserProfileResponse>>
|
||||||
@@ -49,6 +51,14 @@ namespace Musify.Application.Users
|
|||||||
.AsNoTracking()
|
.AsNoTracking()
|
||||||
.CountAsync(f => f.FollowerId == request.UserId, cancellationToken);
|
.CountAsync(f => f.FollowerId == request.UserId, cancellationToken);
|
||||||
|
|
||||||
|
var isFollowing = request.ViewerId is { } viewerId && viewerId != request.UserId &&
|
||||||
|
await database.UserFollows
|
||||||
|
.AsNoTracking()
|
||||||
|
.AnyAsync(f => f.FollowerId == viewerId && f.FollowedId == request.UserId, cancellationToken);
|
||||||
|
|
||||||
|
var canViewFollowers = await FollowVisibility.CanViewFollowersAsync(
|
||||||
|
database, request.ViewerId, request.UserId, cancellationToken);
|
||||||
|
|
||||||
return new UserProfileResponse(
|
return new UserProfileResponse(
|
||||||
user.Id,
|
user.Id,
|
||||||
user.Name,
|
user.Name,
|
||||||
@@ -57,7 +67,9 @@ namespace Musify.Application.Users
|
|||||||
user.ProfilePictureUrl,
|
user.ProfilePictureUrl,
|
||||||
user.CreatedAt,
|
user.CreatedAt,
|
||||||
followersCount,
|
followersCount,
|
||||||
followingCount);
|
followingCount,
|
||||||
|
isFollowing,
|
||||||
|
canViewFollowers);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -8,13 +8,13 @@ using Musify.Application.Contracts;
|
|||||||
|
|
||||||
namespace Musify.Application.Users
|
namespace Musify.Application.Users
|
||||||
{
|
{
|
||||||
public record GetUsersQuery(int PageNumber, int PageSize, string? UsernameSearch)
|
public record GetUsersQuery(int PageNumber, int PageSize, string? UsernameSearch, long? ViewerId = null)
|
||||||
: IQuery<ErrorOr<PaginatedResponse<UserApplicationResponse>>>;
|
: IQuery<ErrorOr<PaginatedResponse<UserSummaryResponse>>>;
|
||||||
|
|
||||||
public class GetUsersQueryHandler(IDatabase database)
|
public class GetUsersQueryHandler(IDatabase database)
|
||||||
: IQueryHandler<GetUsersQuery, ErrorOr<PaginatedResponse<UserApplicationResponse>>>
|
: IQueryHandler<GetUsersQuery, ErrorOr<PaginatedResponse<UserSummaryResponse>>>
|
||||||
{
|
{
|
||||||
public async ValueTask<ErrorOr<PaginatedResponse<UserApplicationResponse>>> Handle(GetUsersQuery request, CancellationToken cancellationToken)
|
public async ValueTask<ErrorOr<PaginatedResponse<UserSummaryResponse>>> Handle(GetUsersQuery request, CancellationToken cancellationToken)
|
||||||
{
|
{
|
||||||
var usersQuery = database.Users
|
var usersQuery = database.Users
|
||||||
.AsNoTracking()
|
.AsNoTracking()
|
||||||
@@ -27,11 +27,19 @@ namespace Musify.Application.Users
|
|||||||
}
|
}
|
||||||
|
|
||||||
var totalCount = await usersQuery.CountAsync(cancellationToken);
|
var totalCount = await usersQuery.CountAsync(cancellationToken);
|
||||||
|
var viewerId = request.ViewerId;
|
||||||
var pagedUsers = await usersQuery
|
var pagedUsers = await usersQuery
|
||||||
.Select(u => UserApplicationResponse.FromEntity(u))
|
.OrderBy(u => u.Name)
|
||||||
|
.Select(u => new UserSummaryResponse(
|
||||||
|
u.Id,
|
||||||
|
u.Name,
|
||||||
|
u.FirstName,
|
||||||
|
u.SecondName,
|
||||||
|
u.ProfilePictureUrl,
|
||||||
|
viewerId != null && database.UserFollows.Any(f => f.FollowerId == viewerId && f.FollowedId == u.Id)))
|
||||||
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
.ToPagedListAsync(request.PageNumber, request.PageSize, totalCount, cancellationToken);
|
||||||
|
|
||||||
return PaginatedResponse<UserApplicationResponse>.FromPagedList(pagedUsers);
|
return PaginatedResponse<UserSummaryResponse>.FromPagedList(pagedUsers);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,13 @@
|
|||||||
|
using System.Text.Json.Serialization;
|
||||||
|
using Musify.Application.Serialization;
|
||||||
|
|
||||||
|
namespace Musify.Application.Users.Responses
|
||||||
|
{
|
||||||
|
public record UserSummaryResponse(
|
||||||
|
[property: JsonConverter(typeof(LongAsStringConverter))] long Id,
|
||||||
|
string Name,
|
||||||
|
string? FirstName,
|
||||||
|
string? SecondName,
|
||||||
|
string? ProfilePictureUrl,
|
||||||
|
bool IsFollowedByViewer);
|
||||||
|
}
|
||||||
@@ -19,6 +19,12 @@ namespace Musify.Domain.Entities
|
|||||||
[Required]
|
[Required]
|
||||||
public DateTime ListenedAt { get; set; } = DateTime.UtcNow;
|
public DateTime ListenedAt { get; set; } = DateTime.UtcNow;
|
||||||
|
|
||||||
|
public double? PlayedSeconds { get; set; }
|
||||||
|
|
||||||
|
public DateTime? LastProgressAt { get; set; }
|
||||||
|
|
||||||
|
public bool IsCounted { get; set; }
|
||||||
|
|
||||||
[ForeignKey(nameof(UserId) )]
|
[ForeignKey(nameof(UserId) )]
|
||||||
public User User { get; set; }
|
public User User { get; set; }
|
||||||
|
|
||||||
|
|||||||
@@ -31,6 +31,9 @@ namespace Musify.Domain.Entities
|
|||||||
[Required]
|
[Required]
|
||||||
public LifeCycleStatus LifeCycleStatus { get; set; } = LifeCycleStatus.Active;
|
public LifeCycleStatus LifeCycleStatus { get; set; } = LifeCycleStatus.Active;
|
||||||
|
|
||||||
|
[Required]
|
||||||
|
public bool IsExplicit { get; set; }
|
||||||
|
|
||||||
[Required]
|
[Required]
|
||||||
public DateTime CreatedAt { get; set; } = DateTime.UtcNow;
|
public DateTime CreatedAt { get; set; } = DateTime.UtcNow;
|
||||||
|
|
||||||
|
|||||||
@@ -15,6 +15,7 @@ namespace Musify.Infrastructure.Jobs
|
|||||||
{
|
{
|
||||||
var userIds = await database.ListeningHistories
|
var userIds = await database.ListeningHistories
|
||||||
.AsNoTracking()
|
.AsNoTracking()
|
||||||
|
.Where(history => history.IsCounted)
|
||||||
.Select(history => history.UserId)
|
.Select(history => history.UserId)
|
||||||
.Distinct()
|
.Distinct()
|
||||||
.ToListAsync(cancellationToken);
|
.ToListAsync(cancellationToken);
|
||||||
|
|||||||
@@ -0,0 +1,13 @@
|
|||||||
|
using Mediator;
|
||||||
|
using Musify.Application.Tracks;
|
||||||
|
|
||||||
|
namespace Musify.Infrastructure.Jobs
|
||||||
|
{
|
||||||
|
public class ListeningHistoryCleanupJob(IMediator mediator)
|
||||||
|
{
|
||||||
|
public async Task RunAsync(CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
await mediator.Send(new DeleteStaleUncountedListensCommand(), cancellationToken);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -37,7 +37,7 @@ namespace Musify.Infrastructure.Persistence
|
|||||||
}
|
}
|
||||||
else if (entry.State == EntityState.Modified)
|
else if (entry.State == EntityState.Modified)
|
||||||
{
|
{
|
||||||
entry.Entity.UpdatedAt = now;
|
entry.Property(nameof(IAuditable.UpdatedAt)).CurrentValue = now;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -13,9 +13,13 @@ namespace Musify.Infrastructure.Persistence
|
|||||||
public class Database(DatabaseConfiguration configuration)
|
public class Database(DatabaseConfiguration configuration)
|
||||||
: DbContext, AppIDatabase
|
: DbContext, AppIDatabase
|
||||||
{
|
{
|
||||||
|
private static readonly AuditableEntityInterceptor AuditInterceptor = new();
|
||||||
|
|
||||||
protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
|
protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
|
||||||
{
|
{
|
||||||
optionsBuilder.UseNpgsql(configuration.ConnectionString, o => o.MapEnum<Genre>("genre"));
|
optionsBuilder
|
||||||
|
.UseNpgsql(configuration.ConnectionString, o => o.MapEnum<Genre>("genre"))
|
||||||
|
.AddInterceptors(AuditInterceptor);
|
||||||
}
|
}
|
||||||
|
|
||||||
protected override void OnModelCreating(ModelBuilder modelBuilder)
|
protected override void OnModelCreating(ModelBuilder modelBuilder)
|
||||||
|
|||||||
@@ -18,10 +18,7 @@ namespace Musify.Infrastructure.Persistence
|
|||||||
serviceDescriptors.AddSingleton(serviceProvider =>
|
serviceDescriptors.AddSingleton(serviceProvider =>
|
||||||
serviceProvider.GetRequiredService<IOptions<DatabaseConfiguration>>().Value);
|
serviceProvider.GetRequiredService<IOptions<DatabaseConfiguration>>().Value);
|
||||||
|
|
||||||
serviceDescriptors.AddSingleton<AuditableEntityInterceptor>();
|
serviceDescriptors.AddDbContext<Database>();
|
||||||
|
|
||||||
serviceDescriptors.AddDbContext<Database>((serviceProvider, options) =>
|
|
||||||
options.AddInterceptors(serviceProvider.GetRequiredService<AuditableEntityInterceptor>()));
|
|
||||||
|
|
||||||
serviceDescriptors.AddScoped<IDatabase>(serviceProvider => serviceProvider.GetRequiredService<Database>());
|
serviceDescriptors.AddScoped<IDatabase>(serviceProvider => serviceProvider.GetRequiredService<Database>());
|
||||||
|
|
||||||
|
|||||||
+1173
File diff suppressed because it is too large
Load Diff
+52
@@ -0,0 +1,52 @@
|
|||||||
|
using System;
|
||||||
|
using Microsoft.EntityFrameworkCore.Migrations;
|
||||||
|
|
||||||
|
#nullable disable
|
||||||
|
|
||||||
|
namespace Musify.Infrastructure.Persistence.Migrations
|
||||||
|
{
|
||||||
|
/// <inheritdoc />
|
||||||
|
public partial class TrackPlayedSeconds : Migration
|
||||||
|
{
|
||||||
|
/// <inheritdoc />
|
||||||
|
protected override void Up(MigrationBuilder migrationBuilder)
|
||||||
|
{
|
||||||
|
migrationBuilder.AddColumn<bool>(
|
||||||
|
name: "IsCounted",
|
||||||
|
table: "ListeningHistory",
|
||||||
|
type: "boolean",
|
||||||
|
nullable: false,
|
||||||
|
defaultValue: false);
|
||||||
|
|
||||||
|
migrationBuilder.AddColumn<DateTime>(
|
||||||
|
name: "LastProgressAt",
|
||||||
|
table: "ListeningHistory",
|
||||||
|
type: "timestamp with time zone",
|
||||||
|
nullable: true);
|
||||||
|
|
||||||
|
migrationBuilder.AddColumn<double>(
|
||||||
|
name: "PlayedSeconds",
|
||||||
|
table: "ListeningHistory",
|
||||||
|
type: "double precision",
|
||||||
|
nullable: true);
|
||||||
|
|
||||||
|
migrationBuilder.Sql("UPDATE \"ListeningHistory\" SET \"IsCounted\" = TRUE;");
|
||||||
|
}
|
||||||
|
|
||||||
|
/// <inheritdoc />
|
||||||
|
protected override void Down(MigrationBuilder migrationBuilder)
|
||||||
|
{
|
||||||
|
migrationBuilder.DropColumn(
|
||||||
|
name: "IsCounted",
|
||||||
|
table: "ListeningHistory");
|
||||||
|
|
||||||
|
migrationBuilder.DropColumn(
|
||||||
|
name: "LastProgressAt",
|
||||||
|
table: "ListeningHistory");
|
||||||
|
|
||||||
|
migrationBuilder.DropColumn(
|
||||||
|
name: "PlayedSeconds",
|
||||||
|
table: "ListeningHistory");
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
+1167
File diff suppressed because it is too large
Load Diff
+29
@@ -0,0 +1,29 @@
|
|||||||
|
using Microsoft.EntityFrameworkCore.Migrations;
|
||||||
|
|
||||||
|
#nullable disable
|
||||||
|
|
||||||
|
namespace Musify.Infrastructure.Persistence.Migrations
|
||||||
|
{
|
||||||
|
/// <inheritdoc />
|
||||||
|
public partial class AddTrackIsExplicit : Migration
|
||||||
|
{
|
||||||
|
/// <inheritdoc />
|
||||||
|
protected override void Up(MigrationBuilder migrationBuilder)
|
||||||
|
{
|
||||||
|
migrationBuilder.AddColumn<bool>(
|
||||||
|
name: "IsExplicit",
|
||||||
|
table: "Tracks",
|
||||||
|
type: "boolean",
|
||||||
|
nullable: false,
|
||||||
|
defaultValue: false);
|
||||||
|
}
|
||||||
|
|
||||||
|
/// <inheritdoc />
|
||||||
|
protected override void Down(MigrationBuilder migrationBuilder)
|
||||||
|
{
|
||||||
|
migrationBuilder.DropColumn(
|
||||||
|
name: "IsExplicit",
|
||||||
|
table: "Tracks");
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
+38
-30
@@ -1,4 +1,4 @@
|
|||||||
|
// <auto-generated />
|
||||||
using System;
|
using System;
|
||||||
using Microsoft.EntityFrameworkCore;
|
using Microsoft.EntityFrameworkCore;
|
||||||
using Microsoft.EntityFrameworkCore.Infrastructure;
|
using Microsoft.EntityFrameworkCore.Infrastructure;
|
||||||
@@ -68,7 +68,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasIndex("Delivered");
|
b.HasIndex("Delivered");
|
||||||
|
|
||||||
b.ToTable("InboxState", (string)null);
|
b.ToTable("InboxState");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("MassTransit.EntityFrameworkCoreIntegration.OutboxMessage", b =>
|
modelBuilder.Entity("MassTransit.EntityFrameworkCoreIntegration.OutboxMessage", b =>
|
||||||
@@ -159,7 +159,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
b.HasIndex("InboxMessageId", "InboxConsumerId", "SequenceNumber")
|
b.HasIndex("InboxMessageId", "InboxConsumerId", "SequenceNumber")
|
||||||
.IsUnique();
|
.IsUnique();
|
||||||
|
|
||||||
b.ToTable("OutboxMessage", (string)null);
|
b.ToTable("OutboxMessage");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("MassTransit.EntityFrameworkCoreIntegration.OutboxState", b =>
|
modelBuilder.Entity("MassTransit.EntityFrameworkCoreIntegration.OutboxState", b =>
|
||||||
@@ -189,7 +189,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasIndex("Created");
|
b.HasIndex("Created");
|
||||||
|
|
||||||
b.ToTable("OutboxState", (string)null);
|
b.ToTable("OutboxState");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.Album", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.Album", b =>
|
||||||
@@ -231,7 +231,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasIndex("OwnerUserId");
|
b.HasIndex("OwnerUserId");
|
||||||
|
|
||||||
b.ToTable("Albums", (string)null);
|
b.ToTable("Albums");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.AlbumHasTrack", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.AlbumHasTrack", b =>
|
||||||
@@ -259,7 +259,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
b.HasIndex("AlbumId", "TrackId")
|
b.HasIndex("AlbumId", "TrackId")
|
||||||
.IsUnique();
|
.IsUnique();
|
||||||
|
|
||||||
b.ToTable("AlbumHasTrack", (string)null);
|
b.ToTable("AlbumHasTrack");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.ListeningHistory", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.ListeningHistory", b =>
|
||||||
@@ -268,9 +268,18 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
.ValueGeneratedOnAdd()
|
.ValueGeneratedOnAdd()
|
||||||
.HasColumnType("uuid");
|
.HasColumnType("uuid");
|
||||||
|
|
||||||
|
b.Property<bool>("IsCounted")
|
||||||
|
.HasColumnType("boolean");
|
||||||
|
|
||||||
|
b.Property<DateTime?>("LastProgressAt")
|
||||||
|
.HasColumnType("timestamp with time zone");
|
||||||
|
|
||||||
b.Property<DateTime>("ListenedAt")
|
b.Property<DateTime>("ListenedAt")
|
||||||
.HasColumnType("timestamp with time zone");
|
.HasColumnType("timestamp with time zone");
|
||||||
|
|
||||||
|
b.Property<double?>("PlayedSeconds")
|
||||||
|
.HasColumnType("double precision");
|
||||||
|
|
||||||
b.Property<Guid>("TrackId")
|
b.Property<Guid>("TrackId")
|
||||||
.HasColumnType("uuid");
|
.HasColumnType("uuid");
|
||||||
|
|
||||||
@@ -283,7 +292,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasIndex("UserId");
|
b.HasIndex("UserId");
|
||||||
|
|
||||||
b.ToTable("ListeningHistory", (string)null);
|
b.ToTable("ListeningHistory");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.Mix", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.Mix", b =>
|
||||||
@@ -317,7 +326,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasIndex("UserId", "Position");
|
b.HasIndex("UserId", "Position");
|
||||||
|
|
||||||
b.ToTable("Mixes", (string)null);
|
b.ToTable("Mixes");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.MixItem", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.MixItem", b =>
|
||||||
@@ -341,7 +350,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasIndex("MixId", "Position");
|
b.HasIndex("MixId", "Position");
|
||||||
|
|
||||||
b.ToTable("MixItems", (string)null);
|
b.ToTable("MixItems");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.PlayList", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.PlayList", b =>
|
||||||
@@ -385,7 +394,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasIndex("UserId");
|
b.HasIndex("UserId");
|
||||||
|
|
||||||
b.ToTable("PlayLists", (string)null);
|
b.ToTable("PlayLists");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.PlayListHasTrack", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.PlayListHasTrack", b =>
|
||||||
@@ -412,7 +421,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasIndex("TrackId");
|
b.HasIndex("TrackId");
|
||||||
|
|
||||||
b.ToTable("PlayListHasTrack", (string)null);
|
b.ToTable("PlayListHasTrack");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.Track", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.Track", b =>
|
||||||
@@ -427,6 +436,9 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
b.Property<double>("DurationSeconds")
|
b.Property<double>("DurationSeconds")
|
||||||
.HasColumnType("double precision");
|
.HasColumnType("double precision");
|
||||||
|
|
||||||
|
b.Property<bool>("IsExplicit")
|
||||||
|
.HasColumnType("boolean");
|
||||||
|
|
||||||
b.Property<int>("LifeCycleStatus")
|
b.Property<int>("LifeCycleStatus")
|
||||||
.HasColumnType("integer");
|
.HasColumnType("integer");
|
||||||
|
|
||||||
@@ -450,7 +462,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasIndex("OwnerUserId");
|
b.HasIndex("OwnerUserId");
|
||||||
|
|
||||||
b.ToTable("Tracks", (string)null);
|
b.ToTable("Tracks");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.TrackLike", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.TrackLike", b =>
|
||||||
@@ -475,7 +487,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
b.HasIndex("UserId", "TrackId")
|
b.HasIndex("UserId", "TrackId")
|
||||||
.IsUnique();
|
.IsUnique();
|
||||||
|
|
||||||
b.ToTable("TrackLikes", (string)null);
|
b.ToTable("TrackLikes");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.TrackTag", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.TrackTag", b =>
|
||||||
@@ -495,7 +507,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
b.HasIndex("TrackId", "Tag")
|
b.HasIndex("TrackId", "Tag")
|
||||||
.IsUnique();
|
.IsUnique();
|
||||||
|
|
||||||
b.ToTable("TrackTags", (string)null);
|
b.ToTable("TrackTags");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.Upload", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.Upload", b =>
|
||||||
@@ -524,7 +536,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasKey("Id");
|
b.HasKey("Id");
|
||||||
|
|
||||||
b.ToTable("Upload", (string)null);
|
b.ToTable("Upload");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.UploadIntent", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.UploadIntent", b =>
|
||||||
@@ -575,7 +587,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasIndex("UserId");
|
b.HasIndex("UserId");
|
||||||
|
|
||||||
b.ToTable("UploadIntents", (string)null);
|
b.ToTable("UploadIntents");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.User", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.User", b =>
|
||||||
@@ -613,7 +625,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasKey("Id");
|
b.HasKey("Id");
|
||||||
|
|
||||||
b.ToTable("User", (string)null);
|
b.ToTable("User");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.UserFollow", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.UserFollow", b =>
|
||||||
@@ -638,7 +650,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
b.HasIndex("FollowerId", "FollowedId")
|
b.HasIndex("FollowerId", "FollowedId")
|
||||||
.IsUnique();
|
.IsUnique();
|
||||||
|
|
||||||
b.ToTable("UserFollows", (string)null);
|
b.ToTable("UserFollows");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Domain.Entities.UserHasTrack", b =>
|
modelBuilder.Entity("Musify.Domain.Entities.UserHasTrack", b =>
|
||||||
@@ -659,7 +671,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasIndex("UserId");
|
b.HasIndex("UserId");
|
||||||
|
|
||||||
b.ToTable("UserHasTrack", (string)null);
|
b.ToTable("UserHasTrack");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Infrastructure.MassTransit.Sagas.AlbumProcessingState", b =>
|
modelBuilder.Entity("Musify.Infrastructure.MassTransit.Sagas.AlbumProcessingState", b =>
|
||||||
@@ -686,7 +698,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasKey("CorrelationId");
|
b.HasKey("CorrelationId");
|
||||||
|
|
||||||
b.ToTable("AlbumProcessingState", (string)null);
|
b.ToTable("AlbumProcessingState");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Infrastructure.MassTransit.Sagas.PlayListProcessingState", b =>
|
modelBuilder.Entity("Musify.Infrastructure.MassTransit.Sagas.PlayListProcessingState", b =>
|
||||||
@@ -713,7 +725,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasKey("CorrelationId");
|
b.HasKey("CorrelationId");
|
||||||
|
|
||||||
b.ToTable("PlayListProcessingState", (string)null);
|
b.ToTable("PlayListProcessingState");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("Musify.Infrastructure.MassTransit.Sagas.TrackProcessingState", b =>
|
modelBuilder.Entity("Musify.Infrastructure.MassTransit.Sagas.TrackProcessingState", b =>
|
||||||
@@ -749,7 +761,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b.HasKey("CorrelationId");
|
b.HasKey("CorrelationId");
|
||||||
|
|
||||||
b.ToTable("TrackProcessingState", (string)null);
|
b.ToTable("TrackProcessingState");
|
||||||
});
|
});
|
||||||
|
|
||||||
modelBuilder.Entity("MassTransit.EntityFrameworkCoreIntegration.OutboxMessage", b =>
|
modelBuilder.Entity("MassTransit.EntityFrameworkCoreIntegration.OutboxMessage", b =>
|
||||||
@@ -803,7 +815,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b1.HasKey("AlbumId");
|
b1.HasKey("AlbumId");
|
||||||
|
|
||||||
b1.ToTable("Albums", (string)null);
|
b1.ToTable("Albums");
|
||||||
|
|
||||||
b1.WithOwner()
|
b1.WithOwner()
|
||||||
.HasForeignKey("AlbumId");
|
.HasForeignKey("AlbumId");
|
||||||
@@ -896,32 +908,28 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
.HasColumnType("uuid");
|
.HasColumnType("uuid");
|
||||||
|
|
||||||
b1.Property<string>("LargeName")
|
b1.Property<string>("LargeName")
|
||||||
.IsRequired()
|
|
||||||
.HasMaxLength(64)
|
.HasMaxLength(64)
|
||||||
.HasColumnType("character varying(64)")
|
.HasColumnType("character varying(64)")
|
||||||
.HasColumnName("LargePictureName");
|
.HasColumnName("LargePictureName");
|
||||||
|
|
||||||
b1.Property<string>("MediumName")
|
b1.Property<string>("MediumName")
|
||||||
.IsRequired()
|
|
||||||
.HasMaxLength(64)
|
.HasMaxLength(64)
|
||||||
.HasColumnType("character varying(64)")
|
.HasColumnType("character varying(64)")
|
||||||
.HasColumnName("MediumPictureName");
|
.HasColumnName("MediumPictureName");
|
||||||
|
|
||||||
b1.Property<string>("OriginalName")
|
b1.Property<string>("OriginalName")
|
||||||
.IsRequired()
|
|
||||||
.HasMaxLength(64)
|
.HasMaxLength(64)
|
||||||
.HasColumnType("character varying(64)")
|
.HasColumnType("character varying(64)")
|
||||||
.HasColumnName("OriginalPictureName");
|
.HasColumnName("OriginalPictureName");
|
||||||
|
|
||||||
b1.Property<string>("SmallName")
|
b1.Property<string>("SmallName")
|
||||||
.IsRequired()
|
|
||||||
.HasMaxLength(64)
|
.HasMaxLength(64)
|
||||||
.HasColumnType("character varying(64)")
|
.HasColumnType("character varying(64)")
|
||||||
.HasColumnName("SmallPictureName");
|
.HasColumnName("SmallPictureName");
|
||||||
|
|
||||||
b1.HasKey("PlayListId");
|
b1.HasKey("PlayListId");
|
||||||
|
|
||||||
b1.ToTable("PlayLists", (string)null);
|
b1.ToTable("PlayLists");
|
||||||
|
|
||||||
b1.WithOwner()
|
b1.WithOwner()
|
||||||
.HasForeignKey("PlayListId");
|
.HasForeignKey("PlayListId");
|
||||||
@@ -992,7 +1000,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b1.HasKey("TrackId");
|
b1.HasKey("TrackId");
|
||||||
|
|
||||||
b1.ToTable("Tracks", (string)null);
|
b1.ToTable("Tracks");
|
||||||
|
|
||||||
b1.WithOwner()
|
b1.WithOwner()
|
||||||
.HasForeignKey("TrackId");
|
.HasForeignKey("TrackId");
|
||||||
@@ -1029,7 +1037,7 @@ namespace Musify.Infrastructure.Persistence.Migrations
|
|||||||
|
|
||||||
b1.HasKey("TrackId");
|
b1.HasKey("TrackId");
|
||||||
|
|
||||||
b1.ToTable("Tracks", (string)null);
|
b1.ToTable("Tracks");
|
||||||
|
|
||||||
b1.WithOwner()
|
b1.WithOwner()
|
||||||
.HasForeignKey("TrackId");
|
.HasForeignKey("TrackId");
|
||||||
|
|||||||
@@ -74,6 +74,7 @@ namespace Musify.Infrastructure.Services
|
|||||||
|
|
||||||
serviceDescriptors.AddHangfireServer();
|
serviceDescriptors.AddHangfireServer();
|
||||||
serviceDescriptors.AddScoped<DailyMixGenerationJob>();
|
serviceDescriptors.AddScoped<DailyMixGenerationJob>();
|
||||||
|
serviceDescriptors.AddScoped<ListeningHistoryCleanupJob>();
|
||||||
|
|
||||||
return serviceDescriptors;
|
return serviceDescriptors;
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -6,4 +6,5 @@ public record CreateTrackRequest(
|
|||||||
string Title,
|
string Title,
|
||||||
Guid PictureIntentId,
|
Guid PictureIntentId,
|
||||||
Guid AudioIntentId,
|
Guid AudioIntentId,
|
||||||
IReadOnlyCollection<Genre> Tags);
|
IReadOnlyCollection<Genre> Tags,
|
||||||
|
bool IsExplicit = false);
|
||||||
|
|||||||
@@ -0,0 +1,3 @@
|
|||||||
|
namespace Musify.Api.DataTransferObjects.Tracks;
|
||||||
|
|
||||||
|
public record RecordListeningProgressRequest(double PlayedSeconds);
|
||||||
@@ -0,0 +1,39 @@
|
|||||||
|
using Mediator;
|
||||||
|
using Musify.Application.Genres;
|
||||||
|
|
||||||
|
namespace Musify.Api.Endpoints;
|
||||||
|
|
||||||
|
public static class GenreEndpoints
|
||||||
|
{
|
||||||
|
public static IEndpointRouteBuilder MapGenreEndpoints(this IEndpointRouteBuilder app)
|
||||||
|
{
|
||||||
|
var group = app.MapGroup("/genres")
|
||||||
|
.WithTags("Genres");
|
||||||
|
|
||||||
|
group.MapGet("/", GetGenres)
|
||||||
|
.WithName("GetGenres")
|
||||||
|
.WithSummary("Get The Genres That Have At Least One Track, Most Populated First. Works Anonymously.")
|
||||||
|
.Produces<IReadOnlyList<GenreResponse>>();
|
||||||
|
|
||||||
|
group.MapGet("/available", GetAvailableGenres)
|
||||||
|
.WithName("GetAvailableGenres")
|
||||||
|
.WithSummary("Get Every Genre A Track Can Be Tagged With, Including The Genres It Cannot Be Combined With. Works Anonymously.")
|
||||||
|
.Produces<IReadOnlyList<AvailableGenreResponse>>();
|
||||||
|
|
||||||
|
return app;
|
||||||
|
}
|
||||||
|
|
||||||
|
private static async Task<IReadOnlyList<AvailableGenreResponse>> GetAvailableGenres(
|
||||||
|
IMediator mediator,
|
||||||
|
CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
return await mediator.Send(new GetAvailableGenresQuery(), cancellationToken);
|
||||||
|
}
|
||||||
|
|
||||||
|
private static async Task<IReadOnlyList<GenreResponse>> GetGenres(
|
||||||
|
IMediator mediator,
|
||||||
|
CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
return await mediator.Send(new GetGenresQuery(), cancellationToken);
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -94,6 +94,14 @@ public static class PlayListEndpoints
|
|||||||
.Produces(StatusCodes.Status404NotFound)
|
.Produces(StatusCodes.Status404NotFound)
|
||||||
.Produces(StatusCodes.Status409Conflict);
|
.Produces(StatusCodes.Status409Conflict);
|
||||||
|
|
||||||
|
group.MapPost("/{playlistId}/albums/{albumId}", AddAlbumToPlayList)
|
||||||
|
.WithName("AddAlbumToPlayList")
|
||||||
|
.WithSummary("Add All The Tracks Of An Album To One Of Your PlayLists, Skipping Those Already In It.")
|
||||||
|
.RequireAuthorization()
|
||||||
|
.Produces<AddAlbumToPlayListResponse>()
|
||||||
|
.Produces(StatusCodes.Status401Unauthorized)
|
||||||
|
.Produces(StatusCodes.Status404NotFound);
|
||||||
|
|
||||||
group.MapDelete("/{playlistId}/tracks/{trackId}", RemoveTrackFromPlayList)
|
group.MapDelete("/{playlistId}/tracks/{trackId}", RemoveTrackFromPlayList)
|
||||||
.WithName("RemoveTrackFromPlayList")
|
.WithName("RemoveTrackFromPlayList")
|
||||||
.WithSummary("Remove A Track From A PlayList.")
|
.WithSummary("Remove A Track From A PlayList.")
|
||||||
@@ -234,6 +242,20 @@ public static class PlayListEndpoints
|
|||||||
return trackResult.ToCreatedResult(track => $"/tracks/{track.Id}");
|
return trackResult.ToCreatedResult(track => $"/tracks/{track.Id}");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private static async Task<IResult> AddAlbumToPlayList(
|
||||||
|
IMediator mediator,
|
||||||
|
CurrentUser currentUser,
|
||||||
|
Guid playlistId,
|
||||||
|
Guid albumId,
|
||||||
|
CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
var result = await mediator.Send(
|
||||||
|
new AddAlbumToPlayListCommand(currentUser.RequiredId, playlistId, albumId),
|
||||||
|
cancellationToken);
|
||||||
|
|
||||||
|
return result.ToHttpResult();
|
||||||
|
}
|
||||||
|
|
||||||
private static async Task<IResult> RemoveTrackFromPlayList(
|
private static async Task<IResult> RemoveTrackFromPlayList(
|
||||||
IMediator mediator,
|
IMediator mediator,
|
||||||
CurrentUser currentUser,
|
CurrentUser currentUser,
|
||||||
|
|||||||
@@ -1,4 +1,5 @@
|
|||||||
using Mediator;
|
using Mediator;
|
||||||
|
using Musify.Domain.ValueObjects;
|
||||||
using Musify.Application.Tracks;
|
using Musify.Application.Tracks;
|
||||||
using Musify.Application.Tracks.Responses;
|
using Musify.Application.Tracks.Responses;
|
||||||
using Musify.Application.Shared;
|
using Musify.Application.Shared;
|
||||||
@@ -19,8 +20,9 @@ public static class TrackEndpoints
|
|||||||
|
|
||||||
group.MapGet("/", GetTracks)
|
group.MapGet("/", GetTracks)
|
||||||
.WithName("GetTracks")
|
.WithName("GetTracks")
|
||||||
.WithSummary("Get Paginated Tracks, Optionally Filtered By Name.")
|
.WithSummary("Get Paginated Tracks, Optionally Filtered By Name And Genre.")
|
||||||
.Produces<TracksSearchResponse>();
|
.Produces<TracksSearchResponse>()
|
||||||
|
.ProducesValidationProblem();
|
||||||
|
|
||||||
group.MapGet("/{id}", GetTrackById)
|
group.MapGet("/{id}", GetTrackById)
|
||||||
.WithName("GetTrackById")
|
.WithName("GetTrackById")
|
||||||
@@ -41,6 +43,14 @@ public static class TrackEndpoints
|
|||||||
.Produces(StatusCodes.Status401Unauthorized)
|
.Produces(StatusCodes.Status401Unauthorized)
|
||||||
.Produces(StatusCodes.Status404NotFound);
|
.Produces(StatusCodes.Status404NotFound);
|
||||||
|
|
||||||
|
group.MapPut("/listens/{listenId}/progress", RecordListeningProgress)
|
||||||
|
.WithName("RecordListeningProgress")
|
||||||
|
.WithSummary("Report The Total Seconds Actually Played For A Listen Started By The Stream Endpoint.")
|
||||||
|
.RequireAuthorization()
|
||||||
|
.Produces(StatusCodes.Status204NoContent)
|
||||||
|
.Produces(StatusCodes.Status401Unauthorized)
|
||||||
|
.Produces(StatusCodes.Status404NotFound);
|
||||||
|
|
||||||
group.MapGet("/users/{userId}", GetTracksByUserId)
|
group.MapGet("/users/{userId}", GetTracksByUserId)
|
||||||
.WithName("GetTracksByUserId")
|
.WithName("GetTracksByUserId")
|
||||||
.WithSummary("Get Paginated Tracks For A User.")
|
.WithSummary("Get Paginated Tracks For A User.")
|
||||||
@@ -84,10 +94,25 @@ public static class TrackEndpoints
|
|||||||
IMediator mediator,
|
IMediator mediator,
|
||||||
CancellationToken cancellationToken,
|
CancellationToken cancellationToken,
|
||||||
string? name,
|
string? name,
|
||||||
|
string? genre,
|
||||||
int pageNumber = 1,
|
int pageNumber = 1,
|
||||||
int pageSize = 10)
|
int pageSize = 10)
|
||||||
{
|
{
|
||||||
var result = await mediator.Send(new GetTracksQuery(name, pageNumber, pageSize), cancellationToken);
|
Genre? parsedGenre = null;
|
||||||
|
if (!string.IsNullOrWhiteSpace(genre))
|
||||||
|
{
|
||||||
|
if (!Enum.TryParse<Genre>(genre, ignoreCase: true, out var value) || !Enum.IsDefined(value))
|
||||||
|
{
|
||||||
|
return Results.ValidationProblem(new Dictionary<string, string[]>
|
||||||
|
{
|
||||||
|
[nameof(genre)] = [$"'{genre}' is not a valid genre."]
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
parsedGenre = value;
|
||||||
|
}
|
||||||
|
|
||||||
|
var result = await mediator.Send(new GetTracksQuery(name, pageNumber, pageSize, parsedGenre), cancellationToken);
|
||||||
return result.ToHttpResult();
|
return result.ToHttpResult();
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -128,6 +153,19 @@ public static class TrackEndpoints
|
|||||||
return result.ToHttpResult();
|
return result.ToHttpResult();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private static async Task<IResult> RecordListeningProgress(
|
||||||
|
IMediator mediator,
|
||||||
|
CurrentUser currentUser,
|
||||||
|
Guid listenId,
|
||||||
|
RecordListeningProgressRequest request,
|
||||||
|
CancellationToken cancellationToken)
|
||||||
|
{
|
||||||
|
var result = await mediator.Send(
|
||||||
|
new RecordListeningProgressCommand(currentUser.RequiredId, listenId, request.PlayedSeconds),
|
||||||
|
cancellationToken);
|
||||||
|
return result.ToHttpResult();
|
||||||
|
}
|
||||||
|
|
||||||
private static async Task<IResult> GetTracksByUserId(
|
private static async Task<IResult> GetTracksByUserId(
|
||||||
IMediator mediator,
|
IMediator mediator,
|
||||||
long userId,
|
long userId,
|
||||||
@@ -147,7 +185,7 @@ public static class TrackEndpoints
|
|||||||
CancellationToken cancellationToken)
|
CancellationToken cancellationToken)
|
||||||
{
|
{
|
||||||
var result = await mediator.Send(
|
var result = await mediator.Send(
|
||||||
new CreateTrackCommand(currentUser.RequiredId, request.Title, request.PictureIntentId, request.AudioIntentId, request.Tags),
|
new CreateTrackCommand(currentUser.RequiredId, request.Title, request.PictureIntentId, request.AudioIntentId, request.Tags, request.IsExplicit),
|
||||||
cancellationToken);
|
cancellationToken);
|
||||||
|
|
||||||
return result.ToCreatedResult(track => $"/tracks/{track.Id}");
|
return result.ToCreatedResult(track => $"/tracks/{track.Id}");
|
||||||
|
|||||||
@@ -21,7 +21,7 @@ public static class UserEndpoints
|
|||||||
group.MapGet("/", GetUsers)
|
group.MapGet("/", GetUsers)
|
||||||
.WithName("GetUsers")
|
.WithName("GetUsers")
|
||||||
.WithSummary("Get Paginated Users.")
|
.WithSummary("Get Paginated Users.")
|
||||||
.Produces<PaginatedResponse<UserApplicationResponse>>();
|
.Produces<PaginatedResponse<UserSummaryResponse>>();
|
||||||
|
|
||||||
group.MapGet("/{id}", GetUserById)
|
group.MapGet("/{id}", GetUserById)
|
||||||
.WithName("GetUserById")
|
.WithName("GetUserById")
|
||||||
@@ -52,6 +52,19 @@ public static class UserEndpoints
|
|||||||
.Produces<UserProfileResponse>()
|
.Produces<UserProfileResponse>()
|
||||||
.Produces(StatusCodes.Status404NotFound);
|
.Produces(StatusCodes.Status404NotFound);
|
||||||
|
|
||||||
|
group.MapGet("/{id}/followers", GetFollowers)
|
||||||
|
.WithName("GetUserFollowers")
|
||||||
|
.WithSummary("Get The Users Following A User. Visible To The User And To Mutual Followers.")
|
||||||
|
.Produces<PaginatedResponse<UserSummaryResponse>>()
|
||||||
|
.Produces(StatusCodes.Status403Forbidden)
|
||||||
|
.Produces(StatusCodes.Status404NotFound);
|
||||||
|
|
||||||
|
group.MapGet("/{id}/following", GetFollowing)
|
||||||
|
.WithName("GetUserFollowing")
|
||||||
|
.WithSummary("Get The Users A User Follows.")
|
||||||
|
.Produces<PaginatedResponse<UserSummaryResponse>>()
|
||||||
|
.Produces(StatusCodes.Status404NotFound);
|
||||||
|
|
||||||
group.MapGet("/{id}/is-following", IsFollowing)
|
group.MapGet("/{id}/is-following", IsFollowing)
|
||||||
.WithName("IsFollowingUser")
|
.WithName("IsFollowingUser")
|
||||||
.WithSummary("Check Whether The Current User Follows Another User.")
|
.WithSummary("Check Whether The Current User Follows Another User.")
|
||||||
@@ -88,12 +101,13 @@ public static class UserEndpoints
|
|||||||
|
|
||||||
private static async Task<IResult> GetUsers(
|
private static async Task<IResult> GetUsers(
|
||||||
IMediator mediator,
|
IMediator mediator,
|
||||||
|
CurrentUser currentUser,
|
||||||
CancellationToken cancellationToken,
|
CancellationToken cancellationToken,
|
||||||
string? usernameSearch,
|
string? usernameSearch,
|
||||||
int pageNumber = 1,
|
int pageNumber = 1,
|
||||||
int pageSize = 10)
|
int pageSize = 10)
|
||||||
{
|
{
|
||||||
var result = await mediator.Send(new GetUsersQuery(pageNumber, pageSize, usernameSearch), cancellationToken);
|
var result = await mediator.Send(new GetUsersQuery(pageNumber, pageSize, usernameSearch, currentUser.Id), cancellationToken);
|
||||||
return result.ToHttpResult();
|
return result.ToHttpResult();
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -133,10 +147,35 @@ public static class UserEndpoints
|
|||||||
|
|
||||||
private static async Task<IResult> GetUserProfile(
|
private static async Task<IResult> GetUserProfile(
|
||||||
IMediator mediator,
|
IMediator mediator,
|
||||||
|
CurrentUser currentUser,
|
||||||
long id,
|
long id,
|
||||||
CancellationToken cancellationToken)
|
CancellationToken cancellationToken)
|
||||||
{
|
{
|
||||||
var result = await mediator.Send(new GetUserProfileQuery(id), cancellationToken);
|
var result = await mediator.Send(new GetUserProfileQuery(id, currentUser.Id), cancellationToken);
|
||||||
|
return result.ToHttpResult();
|
||||||
|
}
|
||||||
|
|
||||||
|
private static async Task<IResult> GetFollowers(
|
||||||
|
IMediator mediator,
|
||||||
|
CurrentUser currentUser,
|
||||||
|
long id,
|
||||||
|
CancellationToken cancellationToken,
|
||||||
|
int pageNumber = 1,
|
||||||
|
int pageSize = 20)
|
||||||
|
{
|
||||||
|
var result = await mediator.Send(new GetUserFollowersQuery(currentUser.Id, id, pageNumber, pageSize), cancellationToken);
|
||||||
|
return result.ToHttpResult();
|
||||||
|
}
|
||||||
|
|
||||||
|
private static async Task<IResult> GetFollowing(
|
||||||
|
IMediator mediator,
|
||||||
|
CurrentUser currentUser,
|
||||||
|
long id,
|
||||||
|
CancellationToken cancellationToken,
|
||||||
|
int pageNumber = 1,
|
||||||
|
int pageSize = 20)
|
||||||
|
{
|
||||||
|
var result = await mediator.Send(new GetUserFollowingQuery(currentUser.Id, id, pageNumber, pageSize), cancellationToken);
|
||||||
return result.ToHttpResult();
|
return result.ToHttpResult();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -40,6 +40,7 @@ app.UseAuthorization();
|
|||||||
app
|
app
|
||||||
.MapAlbumEndpoints()
|
.MapAlbumEndpoints()
|
||||||
.MapConfigEndpoints()
|
.MapConfigEndpoints()
|
||||||
|
.MapGenreEndpoints()
|
||||||
.MapLikeEndpoints()
|
.MapLikeEndpoints()
|
||||||
.MapMixEndpoints()
|
.MapMixEndpoints()
|
||||||
.MapPlayListEndpoints()
|
.MapPlayListEndpoints()
|
||||||
|
|||||||
@@ -1,6 +0,0 @@
|
|||||||
@WebApi_HostAddress = http://localhost:5111
|
|
||||||
|
|
||||||
GET {{WebApi_HostAddress}}/weatherforecast/
|
|
||||||
Accept: application/json
|
|
||||||
|
|
||||||
###
|
|
||||||
@@ -41,6 +41,11 @@ using (var scope = host.Services.CreateScope())
|
|||||||
job => job.RunAsync(CancellationToken.None),
|
job => job.RunAsync(CancellationToken.None),
|
||||||
Cron.Daily(),
|
Cron.Daily(),
|
||||||
new RecurringJobOptions { TimeZone = TimeZoneInfo.Utc });
|
new RecurringJobOptions { TimeZone = TimeZoneInfo.Utc });
|
||||||
|
recurringJobManager.AddOrUpdate<ListeningHistoryCleanupJob>(
|
||||||
|
"listening-history-cleanup",
|
||||||
|
job => job.RunAsync(CancellationToken.None),
|
||||||
|
Cron.Daily(3),
|
||||||
|
new RecurringJobOptions { TimeZone = TimeZoneInfo.Utc });
|
||||||
}
|
}
|
||||||
|
|
||||||
host.Run();
|
host.Run();
|
||||||
|
|||||||
@@ -12,10 +12,6 @@ RUN dotnet restore Hosts/Musify.Api/Api.csproj
|
|||||||
|
|
||||||
COPY . .
|
COPY . .
|
||||||
RUN dotnet publish Hosts/Musify.Api/Api.csproj -c Release -o /app --no-restore
|
RUN dotnet publish Hosts/Musify.Api/Api.csproj -c Release -o /app --no-restore
|
||||||
# Config files are AppSettings*.json (PascalCase); on case-sensitive Linux the host
|
|
||||||
# looks for appsettings*.json. Add lowercase copies so they get loaded.
|
|
||||||
RUN cp /app/AppSettings.json /app/appsettings.json \
|
|
||||||
&& cp /app/AppSettings.Development.json /app/appsettings.Development.json
|
|
||||||
|
|
||||||
FROM mcr.microsoft.com/dotnet/aspnet:10.0 AS runtime
|
FROM mcr.microsoft.com/dotnet/aspnet:10.0 AS runtime
|
||||||
WORKDIR /app
|
WORKDIR /app
|
||||||
|
|||||||
@@ -11,9 +11,6 @@ RUN dotnet restore Hosts/Musify.Worker/Worker.csproj
|
|||||||
|
|
||||||
COPY . .
|
COPY . .
|
||||||
RUN dotnet publish Hosts/Musify.Worker/Worker.csproj -c Release -o /app --no-restore
|
RUN dotnet publish Hosts/Musify.Worker/Worker.csproj -c Release -o /app --no-restore
|
||||||
# AppSettings*.json (PascalCase) -> lowercase copies for case-sensitive Linux.
|
|
||||||
RUN cp /app/AppSettings.json /app/appsettings.json \
|
|
||||||
&& cp /app/AppSettings.Development.json /app/appsettings.Development.json
|
|
||||||
|
|
||||||
FROM mcr.microsoft.com/dotnet/aspnet:10.0 AS runtime
|
FROM mcr.microsoft.com/dotnet/aspnet:10.0 AS runtime
|
||||||
WORKDIR /app
|
WORKDIR /app
|
||||||
|
|||||||
@@ -0,0 +1,106 @@
|
|||||||
|
using System.Net;
|
||||||
|
using System.Net.Http.Json;
|
||||||
|
using System.Text.Json;
|
||||||
|
using System.Text.Json.Serialization;
|
||||||
|
using Microsoft.Extensions.DependencyInjection;
|
||||||
|
using Musify.Api.Tests.TestSupport;
|
||||||
|
using Musify.Application.Genres;
|
||||||
|
using Musify.Application.Tracks.Responses;
|
||||||
|
using Musify.Domain.Entities;
|
||||||
|
using Musify.Domain.ValueObjects;
|
||||||
|
using Musify.Infrastructure.Persistence;
|
||||||
|
using Xunit;
|
||||||
|
|
||||||
|
namespace Musify.Api.Tests.Endpoints
|
||||||
|
{
|
||||||
|
[Collection(ApiCollection.Name)]
|
||||||
|
public sealed class GenreEndpointsTests(ApiTestFixture fixture)
|
||||||
|
{
|
||||||
|
private static readonly JsonSerializerOptions JsonOptions = new(JsonSerializerDefaults.Web)
|
||||||
|
{
|
||||||
|
Converters = { new JsonStringEnumConverter() }
|
||||||
|
};
|
||||||
|
|
||||||
|
private async Task<string> SeedTrackAsync(Genre genre)
|
||||||
|
{
|
||||||
|
using var scope = fixture.Services.CreateScope();
|
||||||
|
var database = scope.ServiceProvider.GetRequiredService<Database>();
|
||||||
|
var owner = new User { Id = Random.Shared.NextInt64(1, long.MaxValue), Name = "genre-owner", NormalizedName = "GENRE-OWNER" };
|
||||||
|
var title = $"Genre track {Guid.NewGuid():N}";
|
||||||
|
var track = new Track
|
||||||
|
{
|
||||||
|
Title = title,
|
||||||
|
NormalizedTitle = title.ToUpperInvariant(),
|
||||||
|
OwnerUserId = owner.Id,
|
||||||
|
Owner = owner,
|
||||||
|
Tags = [new TrackTag { TrackId = Guid.Empty, Tag = genre }]
|
||||||
|
};
|
||||||
|
foreach (var tag in track.Tags)
|
||||||
|
tag.TrackId = track.Id;
|
||||||
|
database.Users.Add(owner);
|
||||||
|
database.Tracks.Add(track);
|
||||||
|
database.UserHasTracks.Add(new UserHasTrack { UserId = owner.Id, TrackId = track.Id });
|
||||||
|
await database.SaveChangesAsync(TestContext.Current.CancellationToken);
|
||||||
|
return title;
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task GetGenres_Anonymous_ListsGenresInUse()
|
||||||
|
{
|
||||||
|
await SeedTrackAsync(Genre.Funk);
|
||||||
|
|
||||||
|
var response = await fixture.CreateAnonymousClient().GetAsync("/genres", TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(HttpStatusCode.OK, response.StatusCode);
|
||||||
|
var body = await response.Content.ReadFromJsonAsync<List<GenreResponse>>(JsonOptions, TestContext.Current.CancellationToken);
|
||||||
|
Assert.Contains(body!, g => g.Genre == Genre.Funk && g.TrackCount >= 1);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task GetAvailableGenres_Anonymous_ListsEveryGenreWithIncompatibilities()
|
||||||
|
{
|
||||||
|
var response = await fixture.CreateAnonymousClient().GetAsync("/genres/available", TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(HttpStatusCode.OK, response.StatusCode);
|
||||||
|
var body = await response.Content.ReadFromJsonAsync<List<AvailableGenreResponse>>(JsonOptions, TestContext.Current.CancellationToken);
|
||||||
|
Assert.Equal(Enum.GetValues<Genre>().Length, body!.Count);
|
||||||
|
Assert.Contains(Genre.Lofi, body.Single(g => g.Genre == Genre.Metal).IncompatibleWith);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Theory]
|
||||||
|
[InlineData("Nope")]
|
||||||
|
[InlineData("999")]
|
||||||
|
public async Task GetTracks_UnknownGenre_ReturnsBadRequest(string genre)
|
||||||
|
{
|
||||||
|
var response = await fixture.CreateAnonymousClient().GetAsync($"/tracks?genre={genre}", TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(HttpStatusCode.BadRequest, response.StatusCode);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task GetTracks_GenreQuery_IsCaseInsensitive()
|
||||||
|
{
|
||||||
|
var title = await SeedTrackAsync(Genre.Blues);
|
||||||
|
|
||||||
|
var response = await fixture.CreateAnonymousClient().GetAsync("/tracks?genre=blues&pageSize=100", TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(HttpStatusCode.OK, response.StatusCode);
|
||||||
|
var body = await response.Content.ReadFromJsonAsync<TracksSearchResponse>(JsonOptions, TestContext.Current.CancellationToken);
|
||||||
|
Assert.Contains(body!.Items, item => item.Track.Title == title);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task GetTracks_GenreQuery_FiltersByTag()
|
||||||
|
{
|
||||||
|
var title = await SeedTrackAsync(Genre.Ambient);
|
||||||
|
await SeedTrackAsync(Genre.Techno);
|
||||||
|
|
||||||
|
var response = await fixture.CreateAnonymousClient().GetAsync("/tracks?genre=Ambient&pageSize=100", TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(HttpStatusCode.OK, response.StatusCode);
|
||||||
|
var body = await response.Content.ReadFromJsonAsync<TracksSearchResponse>(JsonOptions, TestContext.Current.CancellationToken);
|
||||||
|
Assert.Contains(body!.Items, item => item.Track.Title == title);
|
||||||
|
Assert.All(body.Items, item => Assert.Contains(Genre.Ambient, item.Track.Tags));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,109 @@
|
|||||||
|
using System.Net;
|
||||||
|
using System.Net.Http.Json;
|
||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
|
using Microsoft.Extensions.DependencyInjection;
|
||||||
|
using Musify.Api.DataTransferObjects.Tracks;
|
||||||
|
using Musify.Api.DataTransferObjects.Users;
|
||||||
|
using Musify.Domain.Entities;
|
||||||
|
using Musify.Infrastructure.Persistence;
|
||||||
|
using Xunit;
|
||||||
|
using Musify.Api.Tests.TestSupport;
|
||||||
|
|
||||||
|
namespace Musify.Api.Tests.Endpoints
|
||||||
|
{
|
||||||
|
[Collection(ApiCollection.Name)]
|
||||||
|
public sealed class ListeningProgressEndpointsTests(ApiTestFixture fixture)
|
||||||
|
{
|
||||||
|
private async Task<long> CreateUserAsync()
|
||||||
|
{
|
||||||
|
var userId = Random.Shared.NextInt64(1, long.MaxValue);
|
||||||
|
var response = await fixture.CreateAnonymousClient().PostAsJsonAsync(
|
||||||
|
"/users", new CreateUserRequest(userId, $"user-{userId}", null, null), TestContext.Current.CancellationToken);
|
||||||
|
response.EnsureSuccessStatusCode();
|
||||||
|
return userId;
|
||||||
|
}
|
||||||
|
|
||||||
|
private async Task<Guid> SeedListenAsync(long userId, double durationSeconds)
|
||||||
|
{
|
||||||
|
using var scope = fixture.Services.CreateScope();
|
||||||
|
var database = scope.ServiceProvider.GetRequiredService<Database>();
|
||||||
|
var owner = await database.Users.SingleAsync(u => u.Id == userId, TestContext.Current.CancellationToken);
|
||||||
|
var track = new Track
|
||||||
|
{
|
||||||
|
Title = "Progress track",
|
||||||
|
NormalizedTitle = "PROGRESS TRACK",
|
||||||
|
OwnerUserId = owner.Id,
|
||||||
|
Owner = owner,
|
||||||
|
DurationSeconds = durationSeconds
|
||||||
|
};
|
||||||
|
var listen = new ListeningHistory
|
||||||
|
{
|
||||||
|
UserId = userId,
|
||||||
|
TrackId = track.Id,
|
||||||
|
ListenedAt = DateTime.UtcNow.AddMinutes(-10)
|
||||||
|
};
|
||||||
|
database.Tracks.Add(track);
|
||||||
|
database.ListeningHistories.Add(listen);
|
||||||
|
await database.SaveChangesAsync(TestContext.Current.CancellationToken);
|
||||||
|
return listen.Id;
|
||||||
|
}
|
||||||
|
|
||||||
|
private async Task<ListeningHistory> LoadListenAsync(Guid listenId)
|
||||||
|
{
|
||||||
|
using var scope = fixture.Services.CreateScope();
|
||||||
|
var database = scope.ServiceProvider.GetRequiredService<Database>();
|
||||||
|
return await database.ListeningHistories.AsNoTracking()
|
||||||
|
.SingleAsync(l => l.Id == listenId, TestContext.Current.CancellationToken);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task PutProgress_Anonymous_ReturnsUnauthorized()
|
||||||
|
{
|
||||||
|
var response = await fixture.CreateAnonymousClient().PutAsJsonAsync(
|
||||||
|
$"/tracks/listens/{Guid.NewGuid()}/progress", new RecordListeningProgressRequest(10), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(HttpStatusCode.Unauthorized, response.StatusCode);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task PutProgress_OwnListen_StoresSecondsAndMarksCounted()
|
||||||
|
{
|
||||||
|
var userId = await CreateUserAsync();
|
||||||
|
var listenId = await SeedListenAsync(userId, durationSeconds: 200);
|
||||||
|
|
||||||
|
var response = await fixture.CreateAuthenticatedClient(userId).PutAsJsonAsync(
|
||||||
|
$"/tracks/listens/{listenId}/progress", new RecordListeningProgressRequest(45), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(HttpStatusCode.NoContent, response.StatusCode);
|
||||||
|
var stored = await LoadListenAsync(listenId);
|
||||||
|
Assert.Equal(45, stored.PlayedSeconds);
|
||||||
|
Assert.True(stored.IsCounted);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task PutProgress_OtherUsersListen_ReturnsNotFound()
|
||||||
|
{
|
||||||
|
var ownerId = await CreateUserAsync();
|
||||||
|
var otherId = await CreateUserAsync();
|
||||||
|
var listenId = await SeedListenAsync(ownerId, durationSeconds: 200);
|
||||||
|
|
||||||
|
var response = await fixture.CreateAuthenticatedClient(otherId).PutAsJsonAsync(
|
||||||
|
$"/tracks/listens/{listenId}/progress", new RecordListeningProgressRequest(45), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(HttpStatusCode.NotFound, response.StatusCode);
|
||||||
|
Assert.Null((await LoadListenAsync(listenId)).PlayedSeconds);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task PutProgress_NegativeSeconds_ReturnsBadRequest()
|
||||||
|
{
|
||||||
|
var userId = await CreateUserAsync();
|
||||||
|
var listenId = await SeedListenAsync(userId, durationSeconds: 200);
|
||||||
|
|
||||||
|
var response = await fixture.CreateAuthenticatedClient(userId).PutAsJsonAsync(
|
||||||
|
$"/tracks/listens/{listenId}/progress", new RecordListeningProgressRequest(-5), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(HttpStatusCode.BadRequest, response.StatusCode);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
+20
@@ -57,5 +57,25 @@ namespace Musify.Application.Tests.Albums
|
|||||||
Assert.False(result.IsError);
|
Assert.False(result.IsError);
|
||||||
Assert.Equal(2, result.Value.Count);
|
Assert.Equal(2, result.Value.Count);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_UncountedListens_AreIgnored()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User();
|
||||||
|
var countedAlbum = TestEntities.Album(owner.Id, "Counted album");
|
||||||
|
var skippedAlbum = TestEntities.Album(owner.Id, "Skipped album");
|
||||||
|
var countedTrack = TestEntities.Track(owner, "In counted");
|
||||||
|
var skippedTrack = TestEntities.Track(owner, "In skipped");
|
||||||
|
await SeedAsync(
|
||||||
|
owner, countedAlbum, skippedAlbum, countedTrack, skippedTrack,
|
||||||
|
new AlbumHasTrack { AlbumId = countedAlbum.Id, TrackId = countedTrack.Id, TrackNumber = 1 },
|
||||||
|
new AlbumHasTrack { AlbumId = skippedAlbum.Id, TrackId = skippedTrack.Id, TrackNumber = 1 },
|
||||||
|
TestEntities.ListeningHistory(owner.Id, countedTrack.Id, DateTime.UtcNow.AddDays(-1), playedSeconds: 120),
|
||||||
|
TestEntities.ListeningHistory(owner.Id, skippedTrack.Id, DateTime.UtcNow, playedSeconds: 1, isCounted: false));
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetRecentlyListenedAlbumsQuery(owner.Id, Limit: 10), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(["Counted album"], result.Value.Select(album => album.Title));
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,32 @@
|
|||||||
|
using Musify.Application.Genres;
|
||||||
|
using Musify.Domain.ValueObjects;
|
||||||
|
using Xunit;
|
||||||
|
|
||||||
|
namespace Musify.Application.Tests.Genres
|
||||||
|
{
|
||||||
|
public sealed class GetAvailableGenresQueryHandlerTests
|
||||||
|
{
|
||||||
|
private static async Task<IReadOnlyList<AvailableGenreResponse>> RunAsync() =>
|
||||||
|
await new GetAvailableGenresQueryHandler().Handle(new GetAvailableGenresQuery(), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_ReturnsEveryGenre()
|
||||||
|
{
|
||||||
|
var result = await RunAsync();
|
||||||
|
|
||||||
|
Assert.Equal(Enum.GetValues<Genre>(), result.Select(r => r.Genre));
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_ListsIncompatibleGenresSymmetrically()
|
||||||
|
{
|
||||||
|
var result = await RunAsync();
|
||||||
|
|
||||||
|
var metal = result.Single(r => r.Genre == Genre.Metal);
|
||||||
|
var lofi = result.Single(r => r.Genre == Genre.Lofi);
|
||||||
|
Assert.Contains(Genre.Lofi, metal.IncompatibleWith);
|
||||||
|
Assert.Contains(Genre.Metal, lofi.IncompatibleWith);
|
||||||
|
Assert.Empty(result.Single(r => r.Genre == Genre.Pop).IncompatibleWith);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,69 @@
|
|||||||
|
using Musify.Application.Genres;
|
||||||
|
using Musify.Application.Tests.TestSupport;
|
||||||
|
using Musify.Domain.Entities;
|
||||||
|
using Musify.Domain.ValueObjects;
|
||||||
|
using Xunit;
|
||||||
|
|
||||||
|
namespace Musify.Application.Tests.Genres
|
||||||
|
{
|
||||||
|
public sealed class GetGenresQueryHandlerTests : HandlerTestBase
|
||||||
|
{
|
||||||
|
private GetGenresQueryHandler CreateHandler() => new(Database);
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_NoTracks_ReturnsEmpty()
|
||||||
|
{
|
||||||
|
var result = await CreateHandler().Handle(new GetGenresQuery(), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Empty(result);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_ReturnsOnlyGenresInUse_MostPopulatedFirst()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User();
|
||||||
|
var rockA = TestEntities.Track(owner, "Rock A", tags: [Genre.Rock]);
|
||||||
|
var rockB = TestEntities.Track(owner, "Rock B", tags: [Genre.Rock, Genre.Indie]);
|
||||||
|
var jazz = TestEntities.Track(owner, "Jazz", tags: [Genre.Jazz]);
|
||||||
|
await SeedAsync(
|
||||||
|
owner, rockA, rockB, jazz,
|
||||||
|
new UserHasTrack { UserId = owner.Id, TrackId = rockA.Id },
|
||||||
|
new UserHasTrack { UserId = owner.Id, TrackId = rockB.Id },
|
||||||
|
new UserHasTrack { UserId = owner.Id, TrackId = jazz.Id });
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetGenresQuery(), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(
|
||||||
|
[(Genre.Rock, 2), (Genre.Jazz, 1), (Genre.Indie, 1)],
|
||||||
|
result.Select(r => (r.Genre, r.TrackCount)));
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_TrackInSeveralLibraries_IsCountedOnce()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User();
|
||||||
|
var other = TestEntities.User(id: 2, name: "other-user");
|
||||||
|
var track = TestEntities.Track(owner, tags: [Genre.Pop]);
|
||||||
|
await SeedAsync(
|
||||||
|
owner, other, track,
|
||||||
|
new UserHasTrack { UserId = owner.Id, TrackId = track.Id },
|
||||||
|
new UserHasTrack { UserId = other.Id, TrackId = track.Id });
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetGenresQuery(), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(1, Assert.Single(result).TrackCount);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_TrackNotInAnyLibrary_IsIgnored()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User();
|
||||||
|
var track = TestEntities.Track(owner, tags: [Genre.Metal]);
|
||||||
|
await SeedAsync(owner, track);
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetGenresQuery(), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Empty(result);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
+16
@@ -72,5 +72,21 @@ namespace Musify.Application.Tests.Mixes
|
|||||||
.ToListAsync(TestContext.Current.CancellationToken);
|
.ToListAsync(TestContext.Current.CancellationToken);
|
||||||
Assert.All(discoveryItems, item => Assert.Equal(unheardTrack.Id, item.TrackId));
|
Assert.All(discoveryItems, item => Assert.Equal(unheardTrack.Id, item.TrackId));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_TrackWithOnlyUncountedListens_IsTreatedAsUnheard()
|
||||||
|
{
|
||||||
|
var user = TestEntities.User();
|
||||||
|
var skipped = TestEntities.Track(user, "Only skipped");
|
||||||
|
await SeedAsync(
|
||||||
|
user, skipped,
|
||||||
|
new UserHasTrack { UserId = user.Id, TrackId = skipped.Id },
|
||||||
|
TestEntities.ListeningHistory(user.Id, skipped.Id, playedSeconds: 3, isCounted: false));
|
||||||
|
|
||||||
|
await CreateHandler().Handle(new GenerateMixesForUserCommand(user.Id), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
var mixes = await Database.Mixes.ToListAsync(TestContext.Current.CancellationToken);
|
||||||
|
Assert.Contains(mixes, mix => mix.Title == "Descubrimiento");
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
+91
@@ -0,0 +1,91 @@
|
|||||||
|
using ErrorOr;
|
||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
|
using Musify.Application.PlayLists;
|
||||||
|
using Musify.Application.Tests.TestSupport;
|
||||||
|
using Musify.Domain.Entities;
|
||||||
|
using Xunit;
|
||||||
|
|
||||||
|
namespace Musify.Application.Tests.PlayLists
|
||||||
|
{
|
||||||
|
public sealed class AddAlbumToPlayListCommandHandlerTests : HandlerTestBase
|
||||||
|
{
|
||||||
|
private AddAlbumToPlayListCommandHandler CreateHandler() => new(Database, NoOpLogger<AddAlbumToPlayListCommandHandler>());
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_AddsAlbumTracksInAlbumOrder()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User();
|
||||||
|
var playList = TestEntities.PlayList(owner.Id);
|
||||||
|
var album = TestEntities.Album(owner.Id);
|
||||||
|
var first = TestEntities.Track(owner, "First");
|
||||||
|
var second = TestEntities.Track(owner, "Second");
|
||||||
|
await SeedAsync(
|
||||||
|
owner, playList, album, first, second,
|
||||||
|
new AlbumHasTrack { AlbumId = album.Id, TrackId = second.Id, TrackNumber = 2 },
|
||||||
|
new AlbumHasTrack { AlbumId = album.Id, TrackId = first.Id, TrackNumber = 1 });
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new AddAlbumToPlayListCommand(owner.Id, playList.Id, album.Id), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.False(result.IsError);
|
||||||
|
Assert.Equal(2, result.Value.AddedCount);
|
||||||
|
var links = await Database.PlayListHasTracks.OrderBy(link => link.Position).ToListAsync(TestContext.Current.CancellationToken);
|
||||||
|
Assert.Equal([first.Id, second.Id], links.Select(link => link.TrackId));
|
||||||
|
Assert.Equal([0, 1], links.Select(link => link.Position));
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_SkipsTracksAlreadyInPlayList_AndAppendsAfterThem()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User();
|
||||||
|
var playList = TestEntities.PlayList(owner.Id);
|
||||||
|
var album = TestEntities.Album(owner.Id);
|
||||||
|
var existing = TestEntities.Track(owner, "Existing");
|
||||||
|
var fresh = TestEntities.Track(owner, "Fresh");
|
||||||
|
await SeedAsync(
|
||||||
|
owner, playList, album, existing, fresh,
|
||||||
|
new AlbumHasTrack { AlbumId = album.Id, TrackId = existing.Id, TrackNumber = 1 },
|
||||||
|
new AlbumHasTrack { AlbumId = album.Id, TrackId = fresh.Id, TrackNumber = 2 },
|
||||||
|
new PlayListHasTrack { PlayListId = playList.Id, TrackId = existing.Id, Position = 4 });
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new AddAlbumToPlayListCommand(owner.Id, playList.Id, album.Id), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(1, result.Value.AddedCount);
|
||||||
|
var link = await Database.PlayListHasTracks.SingleAsync(l => l.TrackId == fresh.Id, TestContext.Current.CancellationToken);
|
||||||
|
Assert.Equal(5, link.Position);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_PlayListMissing_ReturnsNotFound()
|
||||||
|
{
|
||||||
|
var result = await CreateHandler().Handle(new AddAlbumToPlayListCommand(1, Guid.NewGuid(), Guid.NewGuid()), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(ErrorType.NotFound, result.FirstError.Type);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_NotOwner_ReturnsUnauthorized()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User(1, "owner");
|
||||||
|
var stranger = TestEntities.User(2, "stranger");
|
||||||
|
var playList = TestEntities.PlayList(owner.Id);
|
||||||
|
var album = TestEntities.Album(owner.Id);
|
||||||
|
await SeedAsync(owner, stranger, playList, album);
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new AddAlbumToPlayListCommand(stranger.Id, playList.Id, album.Id), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(ErrorType.Unauthorized, result.FirstError.Type);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_AlbumMissing_ReturnsNotFound()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User();
|
||||||
|
var playList = TestEntities.PlayList(owner.Id);
|
||||||
|
await SeedAsync(owner, playList);
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new AddAlbumToPlayListCommand(owner.Id, playList.Id, Guid.NewGuid()), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(ErrorType.NotFound, result.FirstError.Type);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -17,6 +17,17 @@ namespace Musify.Application.Tests.Shared
|
|||||||
Assert.Equal("The Owner", response.Artist);
|
Assert.Equal("The Owner", response.Artist);
|
||||||
Assert.Equal(owner.Id, response.OwnerUserId);
|
Assert.Equal(owner.Id, response.OwnerUserId);
|
||||||
Assert.Equal(3, response.ListensCount);
|
Assert.Equal(3, response.ListensCount);
|
||||||
|
Assert.False(response.IsExplicit);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public void FromEntity_CopiesExplicitFlag()
|
||||||
|
{
|
||||||
|
var track = TestEntities.Track(TestEntities.User(), isExplicit: true);
|
||||||
|
|
||||||
|
var response = TrackApplicationResponse.FromEntity(track, listensCount: 0);
|
||||||
|
|
||||||
|
Assert.True(response.IsExplicit);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -56,7 +56,8 @@ namespace Musify.Application.Tests.TestSupport
|
|||||||
TrackPictures? pictures = null,
|
TrackPictures? pictures = null,
|
||||||
TrackAudio? audio = null,
|
TrackAudio? audio = null,
|
||||||
LifeCycleStatus lifeCycleStatus = LifeCycleStatus.Active,
|
LifeCycleStatus lifeCycleStatus = LifeCycleStatus.Active,
|
||||||
IEnumerable<Genre>? tags = null)
|
IEnumerable<Genre>? tags = null,
|
||||||
|
bool isExplicit = false)
|
||||||
{
|
{
|
||||||
var track = new Track
|
var track = new Track
|
||||||
{
|
{
|
||||||
@@ -64,6 +65,7 @@ namespace Musify.Application.Tests.TestSupport
|
|||||||
NormalizedTitle = title.ToUpperInvariant(),
|
NormalizedTitle = title.ToUpperInvariant(),
|
||||||
DurationSeconds = durationSeconds,
|
DurationSeconds = durationSeconds,
|
||||||
LifeCycleStatus = lifeCycleStatus,
|
LifeCycleStatus = lifeCycleStatus,
|
||||||
|
IsExplicit = isExplicit,
|
||||||
OwnerUserId = owner.Id,
|
OwnerUserId = owner.Id,
|
||||||
Owner = owner,
|
Owner = owner,
|
||||||
Pictures = pictures ?? ProcessedPictures(),
|
Pictures = pictures ?? ProcessedPictures(),
|
||||||
@@ -177,12 +179,16 @@ namespace Musify.Application.Tests.TestSupport
|
|||||||
public static ListeningHistory ListeningHistory(
|
public static ListeningHistory ListeningHistory(
|
||||||
long userId,
|
long userId,
|
||||||
Guid trackId,
|
Guid trackId,
|
||||||
DateTime? listenedAt = null) =>
|
DateTime? listenedAt = null,
|
||||||
|
double? playedSeconds = null,
|
||||||
|
bool isCounted = true) =>
|
||||||
new()
|
new()
|
||||||
{
|
{
|
||||||
UserId = userId,
|
UserId = userId,
|
||||||
TrackId = trackId,
|
TrackId = trackId,
|
||||||
ListenedAt = listenedAt ?? DateTime.UtcNow
|
ListenedAt = listenedAt ?? DateTime.UtcNow,
|
||||||
|
PlayedSeconds = playedSeconds,
|
||||||
|
IsCounted = isCounted
|
||||||
};
|
};
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -58,6 +58,26 @@ namespace Musify.Application.Tests.Tracks
|
|||||||
Assert.Equal(ProcessingStatus.Pending, stored.Audio.TranscodeStatus);
|
Assert.Equal(ProcessingStatus.Pending, stored.Audio.TranscodeStatus);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Theory]
|
||||||
|
[InlineData(true)]
|
||||||
|
[InlineData(false)]
|
||||||
|
public async Task Handle_StoresExplicitFlag(bool isExplicit)
|
||||||
|
{
|
||||||
|
var user = TestEntities.User();
|
||||||
|
var (picture, audio) = SeedIntents(user.Id);
|
||||||
|
await SeedAsync(user, picture, audio);
|
||||||
|
|
||||||
|
var command = new CreateTrackCommand(user.Id, "My Song", picture.Id, audio.Id, [Genre.Pop], isExplicit);
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(command, TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.False(result.IsError);
|
||||||
|
Assert.Equal(isExplicit, result.Value.IsExplicit);
|
||||||
|
var stored = await Database.Tracks.FindAsync([result.Value.Id], TestContext.Current.CancellationToken);
|
||||||
|
Assert.NotNull(stored);
|
||||||
|
Assert.Equal(isExplicit, stored.IsExplicit);
|
||||||
|
}
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task Handle_NoTags_ReturnsValidationError()
|
public async Task Handle_NoTags_ReturnsValidationError()
|
||||||
{
|
{
|
||||||
|
|||||||
+32
@@ -0,0 +1,32 @@
|
|||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
|
using Musify.Application.Tests.TestSupport;
|
||||||
|
using Musify.Application.Tracks;
|
||||||
|
using Xunit;
|
||||||
|
|
||||||
|
namespace Musify.Application.Tests.Tracks
|
||||||
|
{
|
||||||
|
public sealed class DeleteStaleUncountedListensCommandHandlerTests : HandlerTestBase
|
||||||
|
{
|
||||||
|
private DeleteStaleUncountedListensCommandHandler CreateHandler() =>
|
||||||
|
new(Database, NoOpLogger<DeleteStaleUncountedListensCommandHandler>());
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_DeletesOnlyOldUncountedListens()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User();
|
||||||
|
var track = TestEntities.Track(owner);
|
||||||
|
var oldUncounted = TestEntities.ListeningHistory(owner.Id, track.Id, DateTime.UtcNow.AddDays(-9), playedSeconds: 2, isCounted: false);
|
||||||
|
var oldCounted = TestEntities.ListeningHistory(owner.Id, track.Id, DateTime.UtcNow.AddDays(-9), playedSeconds: 100);
|
||||||
|
var recentUncounted = TestEntities.ListeningHistory(owner.Id, track.Id, DateTime.UtcNow.AddDays(-2), playedSeconds: 2, isCounted: false);
|
||||||
|
await SeedAsync(owner, track, oldUncounted, oldCounted, recentUncounted);
|
||||||
|
|
||||||
|
var deleted = await CreateHandler().Handle(new DeleteStaleUncountedListensCommand(), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(1, deleted);
|
||||||
|
var remaining = await Database.ListeningHistories.Select(l => l.Id).ToListAsync(TestContext.Current.CancellationToken);
|
||||||
|
Assert.DoesNotContain(oldUncounted.Id, remaining);
|
||||||
|
Assert.Contains(oldCounted.Id, remaining);
|
||||||
|
Assert.Contains(recentUncounted.Id, remaining);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -35,5 +35,21 @@ namespace Musify.Application.Tests.Tracks
|
|||||||
|
|
||||||
Assert.Empty(result);
|
Assert.Empty(result);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_UncountedListens_AreIgnored()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User();
|
||||||
|
var counted = TestEntities.Track(owner, "Counted");
|
||||||
|
var skipped = TestEntities.Track(owner, "Skipped");
|
||||||
|
await SeedAsync(
|
||||||
|
owner, counted, skipped,
|
||||||
|
TestEntities.ListeningHistory(owner.Id, counted.Id, DateTime.UtcNow.AddMinutes(-5), playedSeconds: 90),
|
||||||
|
TestEntities.ListeningHistory(owner.Id, skipped.Id, DateTime.UtcNow, playedSeconds: 2, isCounted: false));
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetListeningHistoryQuery(owner.Id), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(["Counted"], result.Select(track => track.Title));
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -33,5 +33,21 @@ namespace Musify.Application.Tests.Tracks
|
|||||||
|
|
||||||
Assert.Equal(ErrorType.NotFound, result.FirstError.Type);
|
Assert.Equal(ErrorType.NotFound, result.FirstError.Type);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_ListensCount_OnlyCountsCountedListens()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User();
|
||||||
|
var track = TestEntities.Track(owner);
|
||||||
|
await SeedAsync(
|
||||||
|
owner, track,
|
||||||
|
TestEntities.ListeningHistory(owner.Id, track.Id, playedSeconds: 100),
|
||||||
|
TestEntities.ListeningHistory(owner.Id, track.Id, playedSeconds: 100),
|
||||||
|
TestEntities.ListeningHistory(owner.Id, track.Id, playedSeconds: 1, isCounted: false));
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetTrackByIdQuery(track.Id), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(2, result.Value.ListensCount);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -45,6 +45,9 @@ namespace Musify.Application.Tests.Tracks
|
|||||||
var history = Assert.Single(await Database.ListeningHistories.ToListAsync(TestContext.Current.CancellationToken));
|
var history = Assert.Single(await Database.ListeningHistories.ToListAsync(TestContext.Current.CancellationToken));
|
||||||
Assert.Equal(owner.Id, history.UserId);
|
Assert.Equal(owner.Id, history.UserId);
|
||||||
Assert.Equal(track.Id, history.TrackId);
|
Assert.Equal(track.Id, history.TrackId);
|
||||||
|
Assert.Equal(history.Id, result.Value.ListenId);
|
||||||
|
Assert.False(history.IsCounted);
|
||||||
|
Assert.Null(history.PlayedSeconds);
|
||||||
}
|
}
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
|
|||||||
@@ -1,5 +1,6 @@
|
|||||||
using Musify.Application.Tests.TestSupport;
|
using Musify.Application.Tests.TestSupport;
|
||||||
using Musify.Application.Tracks;
|
using Musify.Application.Tracks;
|
||||||
|
using Musify.Domain.ValueObjects;
|
||||||
using Xunit;
|
using Xunit;
|
||||||
|
|
||||||
namespace Musify.Application.Tests.Tracks
|
namespace Musify.Application.Tests.Tracks
|
||||||
@@ -43,5 +44,23 @@ namespace Musify.Application.Tests.Tracks
|
|||||||
var item = Assert.Single(result.Value.Items);
|
var item = Assert.Single(result.Value.Items);
|
||||||
Assert.Equal("Bohemian Rhapsody", item.Track.Title);
|
Assert.Equal("Bohemian Rhapsody", item.Track.Title);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_GenreFilter_ReturnsOnlyTracksWithThatTag()
|
||||||
|
{
|
||||||
|
var owner = TestEntities.User();
|
||||||
|
var rock = TestEntities.Track(owner, "Loud", tags: [Genre.Rock, Genre.Indie]);
|
||||||
|
var jazz = TestEntities.Track(owner, "Smooth", tags: [Genre.Jazz]);
|
||||||
|
await SeedAsync(
|
||||||
|
owner, rock, jazz,
|
||||||
|
new Musify.Domain.Entities.UserHasTrack { UserId = owner.Id, TrackId = rock.Id },
|
||||||
|
new Musify.Domain.Entities.UserHasTrack { UserId = owner.Id, TrackId = jazz.Id });
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetTracksQuery(null, 1, 10, Genre.Indie), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.False(result.IsError);
|
||||||
|
var item = Assert.Single(result.Value.Items);
|
||||||
|
Assert.Equal("Loud", item.Track.Title);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
+127
@@ -0,0 +1,127 @@
|
|||||||
|
using ErrorOr;
|
||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
|
using Musify.Application.Tests.TestSupport;
|
||||||
|
using Musify.Application.Tracks;
|
||||||
|
using Musify.Domain.Entities;
|
||||||
|
using Xunit;
|
||||||
|
|
||||||
|
namespace Musify.Application.Tests.Tracks
|
||||||
|
{
|
||||||
|
public sealed class RecordListeningProgressCommandHandlerTests : HandlerTestBase
|
||||||
|
{
|
||||||
|
private RecordListeningProgressCommandHandler CreateHandler() => new(Database);
|
||||||
|
|
||||||
|
private async Task<ListeningHistory> ReloadAsync(Guid id) =>
|
||||||
|
await Database.ListeningHistories.AsNoTracking()
|
||||||
|
.SingleAsync(l => l.Id == id, TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
private async Task<(User User, ListeningHistory Listen)> SeedListenAsync(
|
||||||
|
double durationSeconds, DateTime listenedAt)
|
||||||
|
{
|
||||||
|
var user = TestEntities.User();
|
||||||
|
var track = TestEntities.Track(user, durationSeconds: durationSeconds);
|
||||||
|
var listen = TestEntities.ListeningHistory(user.Id, track.Id, listenedAt, isCounted: false);
|
||||||
|
await SeedAsync(user, track, listen);
|
||||||
|
return (user, listen);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_BelowThreshold_StoresSecondsWithoutCounting()
|
||||||
|
{
|
||||||
|
var (user, listen) = await SeedListenAsync(200, DateTime.UtcNow.AddSeconds(-60));
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(
|
||||||
|
new RecordListeningProgressCommand(user.Id, listen.Id, 10), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.False(result.IsError);
|
||||||
|
var stored = await ReloadAsync(listen.Id);
|
||||||
|
Assert.Equal(10, stored.PlayedSeconds);
|
||||||
|
Assert.False(stored.IsCounted);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_ReachesThirtySeconds_MarksAsCounted()
|
||||||
|
{
|
||||||
|
var (user, listen) = await SeedListenAsync(200, DateTime.UtcNow.AddSeconds(-60));
|
||||||
|
|
||||||
|
await CreateHandler().Handle(
|
||||||
|
new RecordListeningProgressCommand(user.Id, listen.Id, 30), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.True((await ReloadAsync(listen.Id)).IsCounted);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_ShortTrack_CountsAtHalfDuration()
|
||||||
|
{
|
||||||
|
var (user, listen) = await SeedListenAsync(20, DateTime.UtcNow.AddSeconds(-60));
|
||||||
|
|
||||||
|
await CreateHandler().Handle(
|
||||||
|
new RecordListeningProgressCommand(user.Id, listen.Id, 10), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.True((await ReloadAsync(listen.Id)).IsCounted);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_LowerValueThanStored_KeepsHighest()
|
||||||
|
{
|
||||||
|
var (user, listen) = await SeedListenAsync(200, DateTime.UtcNow.AddSeconds(-60));
|
||||||
|
var handler = CreateHandler();
|
||||||
|
|
||||||
|
await handler.Handle(new RecordListeningProgressCommand(user.Id, listen.Id, 40), TestContext.Current.CancellationToken);
|
||||||
|
await handler.Handle(new RecordListeningProgressCommand(user.Id, listen.Id, 15), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
var stored = await ReloadAsync(listen.Id);
|
||||||
|
Assert.Equal(40, stored.PlayedSeconds);
|
||||||
|
Assert.True(stored.IsCounted);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_MoreThanElapsedClock_IsCappedToElapsedTime()
|
||||||
|
{
|
||||||
|
var (user, listen) = await SeedListenAsync(200, DateTime.UtcNow.AddSeconds(-20));
|
||||||
|
|
||||||
|
await CreateHandler().Handle(
|
||||||
|
new RecordListeningProgressCommand(user.Id, listen.Id, 150), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
var stored = await ReloadAsync(listen.Id);
|
||||||
|
Assert.InRange(stored.PlayedSeconds!.Value, 24, 27);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_MoreThanTrackDuration_IsCappedToDuration()
|
||||||
|
{
|
||||||
|
var (user, listen) = await SeedListenAsync(50, DateTime.UtcNow.AddMinutes(-30));
|
||||||
|
|
||||||
|
await CreateHandler().Handle(
|
||||||
|
new RecordListeningProgressCommand(user.Id, listen.Id, 900), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(52, (await ReloadAsync(listen.Id)).PlayedSeconds);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_OtherUsersListen_ReturnsNotFound()
|
||||||
|
{
|
||||||
|
var (_, listen) = await SeedListenAsync(200, DateTime.UtcNow.AddSeconds(-60));
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(
|
||||||
|
new RecordListeningProgressCommand(long.MaxValue, listen.Id, 10), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.True(result.IsError);
|
||||||
|
Assert.Equal(ErrorType.NotFound, result.FirstError.Type);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Theory]
|
||||||
|
[InlineData(-1)]
|
||||||
|
[InlineData(double.NaN)]
|
||||||
|
[InlineData(double.PositiveInfinity)]
|
||||||
|
public async Task Handle_InvalidSeconds_ReturnsValidationError(double seconds)
|
||||||
|
{
|
||||||
|
var (user, listen) = await SeedListenAsync(200, DateTime.UtcNow.AddSeconds(-60));
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(
|
||||||
|
new RecordListeningProgressCommand(user.Id, listen.Id, seconds), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(ErrorType.Validation, result.FirstError.Type);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -40,6 +40,37 @@ namespace Musify.Application.Tests.Users
|
|||||||
Assert.Equal(500, result.SecondsThisWeek);
|
Assert.Equal(500, result.SecondsThisWeek);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_PlayedSeconds_SumsPlayedTimeInsteadOfDuration()
|
||||||
|
{
|
||||||
|
var user = TestEntities.User();
|
||||||
|
var track = TestEntities.Track(user, durationSeconds: 200);
|
||||||
|
await SeedAsync(
|
||||||
|
user, track,
|
||||||
|
TestEntities.ListeningHistory(user.Id, track.Id, playedSeconds: 200),
|
||||||
|
TestEntities.ListeningHistory(user.Id, track.Id, playedSeconds: 12, isCounted: false));
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetListeningStatsQuery(user.Id), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(212, result.SecondsThisWeek);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_UncountedListens_AreExcludedFromTracksAndStreak()
|
||||||
|
{
|
||||||
|
var user = TestEntities.User();
|
||||||
|
var track = TestEntities.Track(user);
|
||||||
|
await SeedAsync(
|
||||||
|
user, track,
|
||||||
|
TestEntities.ListeningHistory(user.Id, track.Id, playedSeconds: 3, isCounted: false));
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetListeningStatsQuery(user.Id), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(0, result.TracksThisWeek);
|
||||||
|
Assert.Equal(0, result.StreakDays);
|
||||||
|
Assert.Equal(3, result.SecondsThisWeek);
|
||||||
|
}
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task Handle_ListenedTodayAndYesterday_StreakIsTwo()
|
public async Task Handle_ListenedTodayAndYesterday_StreakIsTwo()
|
||||||
{
|
{
|
||||||
|
|||||||
@@ -0,0 +1,95 @@
|
|||||||
|
using ErrorOr;
|
||||||
|
using Musify.Application.Shared;
|
||||||
|
using Musify.Application.Tests.TestSupport;
|
||||||
|
using Musify.Application.Users;
|
||||||
|
using Musify.Application.Users.Responses;
|
||||||
|
using Musify.Domain.Entities;
|
||||||
|
using Xunit;
|
||||||
|
|
||||||
|
namespace Musify.Application.Tests.Users
|
||||||
|
{
|
||||||
|
public sealed class GetUserFollowersQueryHandlerTests : HandlerTestBase
|
||||||
|
{
|
||||||
|
private GetUserFollowersQueryHandler CreateHandler() => new(Database);
|
||||||
|
|
||||||
|
private async Task<ErrorOr<PaginatedResponse<UserSummaryResponse>>> Send(long? viewerId, long userId) =>
|
||||||
|
await CreateHandler().Handle(new GetUserFollowersQuery(viewerId, userId, 1, 20), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_OwnProfile_ReturnsFollowers()
|
||||||
|
{
|
||||||
|
var user = TestEntities.User(1, "user");
|
||||||
|
var follower = TestEntities.User(2, "follower");
|
||||||
|
await SeedAsync(user, follower, new UserFollow { FollowerId = follower.Id, FollowedId = user.Id });
|
||||||
|
|
||||||
|
var result = await Send(user.Id, user.Id);
|
||||||
|
|
||||||
|
Assert.False(result.IsError);
|
||||||
|
var item = Assert.Single(result.Value.Items);
|
||||||
|
Assert.Equal(follower.Id, item.Id);
|
||||||
|
Assert.False(item.IsFollowedByViewer);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_MutualFollow_ReturnsFollowersWithViewerFollowState()
|
||||||
|
{
|
||||||
|
var viewer = TestEntities.User(1, "viewer");
|
||||||
|
var user = TestEntities.User(2, "user");
|
||||||
|
var other = TestEntities.User(3, "other");
|
||||||
|
await SeedAsync(
|
||||||
|
viewer, user, other,
|
||||||
|
new UserFollow { FollowerId = viewer.Id, FollowedId = user.Id },
|
||||||
|
new UserFollow { FollowerId = user.Id, FollowedId = viewer.Id },
|
||||||
|
new UserFollow { FollowerId = other.Id, FollowedId = user.Id },
|
||||||
|
new UserFollow { FollowerId = viewer.Id, FollowedId = other.Id });
|
||||||
|
|
||||||
|
var result = await Send(viewer.Id, user.Id);
|
||||||
|
|
||||||
|
Assert.False(result.IsError);
|
||||||
|
Assert.Equal(2, result.Value.TotalItemCount);
|
||||||
|
Assert.All(result.Value.Items, item => Assert.Equal(item.Id == other.Id, item.IsFollowedByViewer));
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_ViewerFollowsButNotFollowedBack_ReturnsForbidden()
|
||||||
|
{
|
||||||
|
var viewer = TestEntities.User(1, "viewer");
|
||||||
|
var user = TestEntities.User(2, "user");
|
||||||
|
await SeedAsync(viewer, user, new UserFollow { FollowerId = viewer.Id, FollowedId = user.Id });
|
||||||
|
|
||||||
|
var result = await Send(viewer.Id, user.Id);
|
||||||
|
|
||||||
|
Assert.Equal(ErrorType.Forbidden, result.FirstError.Type);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_FollowedByUserButNotFollowingBack_ReturnsForbidden()
|
||||||
|
{
|
||||||
|
var viewer = TestEntities.User(1, "viewer");
|
||||||
|
var user = TestEntities.User(2, "user");
|
||||||
|
await SeedAsync(viewer, user, new UserFollow { FollowerId = user.Id, FollowedId = viewer.Id });
|
||||||
|
|
||||||
|
var result = await Send(viewer.Id, user.Id);
|
||||||
|
|
||||||
|
Assert.Equal(ErrorType.Forbidden, result.FirstError.Type);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_Anonymous_ReturnsForbidden()
|
||||||
|
{
|
||||||
|
await SeedAsync(TestEntities.User(1, "user"));
|
||||||
|
|
||||||
|
var result = await Send(null, 1);
|
||||||
|
|
||||||
|
Assert.Equal(ErrorType.Forbidden, result.FirstError.Type);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_UserMissing_ReturnsNotFound()
|
||||||
|
{
|
||||||
|
var result = await Send(1, 404);
|
||||||
|
|
||||||
|
Assert.Equal(ErrorType.NotFound, result.FirstError.Type);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,75 @@
|
|||||||
|
using ErrorOr;
|
||||||
|
using Musify.Application.Tests.TestSupport;
|
||||||
|
using Musify.Application.Users;
|
||||||
|
using Musify.Domain.Entities;
|
||||||
|
using Xunit;
|
||||||
|
|
||||||
|
namespace Musify.Application.Tests.Users
|
||||||
|
{
|
||||||
|
public sealed class GetUserFollowingQueryHandlerTests : HandlerTestBase
|
||||||
|
{
|
||||||
|
private GetUserFollowingQueryHandler CreateHandler() => new(Database);
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_Anonymous_ReturnsFollowedUsers()
|
||||||
|
{
|
||||||
|
var user = TestEntities.User(1, "user");
|
||||||
|
var followedA = TestEntities.User(2, "followedA");
|
||||||
|
var followedB = TestEntities.User(3, "followedB");
|
||||||
|
await SeedAsync(
|
||||||
|
user, followedA, followedB,
|
||||||
|
new UserFollow { FollowerId = user.Id, FollowedId = followedA.Id },
|
||||||
|
new UserFollow { FollowerId = user.Id, FollowedId = followedB.Id });
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetUserFollowingQuery(null, user.Id, 1, 20), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.False(result.IsError);
|
||||||
|
Assert.Equal(2, result.Value.TotalItemCount);
|
||||||
|
Assert.All(result.Value.Items, item => Assert.False(item.IsFollowedByViewer));
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_Viewer_MarksUsersTheViewerFollows()
|
||||||
|
{
|
||||||
|
var viewer = TestEntities.User(1, "viewer");
|
||||||
|
var user = TestEntities.User(2, "user");
|
||||||
|
var followedA = TestEntities.User(3, "followedA");
|
||||||
|
var followedB = TestEntities.User(4, "followedB");
|
||||||
|
await SeedAsync(
|
||||||
|
viewer, user, followedA, followedB,
|
||||||
|
new UserFollow { FollowerId = user.Id, FollowedId = followedA.Id },
|
||||||
|
new UserFollow { FollowerId = user.Id, FollowedId = followedB.Id },
|
||||||
|
new UserFollow { FollowerId = viewer.Id, FollowedId = followedA.Id });
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetUserFollowingQuery(viewer.Id, user.Id, 1, 20), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.False(result.IsError);
|
||||||
|
Assert.All(result.Value.Items, item => Assert.Equal(item.Id == followedA.Id, item.IsFollowedByViewer));
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_Pagination_ReturnsRequestedPage()
|
||||||
|
{
|
||||||
|
await SeedAsync(
|
||||||
|
TestEntities.User(1, "user"), TestEntities.User(2, "a"), TestEntities.User(3, "b"), TestEntities.User(4, "c"),
|
||||||
|
new UserFollow { FollowerId = 1, FollowedId = 2 },
|
||||||
|
new UserFollow { FollowerId = 1, FollowedId = 3 },
|
||||||
|
new UserFollow { FollowerId = 1, FollowedId = 4 });
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetUserFollowingQuery(null, 1, 2, 2), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.False(result.IsError);
|
||||||
|
Assert.Single(result.Value.Items);
|
||||||
|
Assert.Equal(3, result.Value.TotalItemCount);
|
||||||
|
Assert.False(result.Value.HasNextPage);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_UserMissing_ReturnsNotFound()
|
||||||
|
{
|
||||||
|
var result = await CreateHandler().Handle(new GetUserFollowingQuery(null, 404, 1, 20), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.Equal(ErrorType.NotFound, result.FirstError.Type);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -30,6 +30,46 @@ namespace Musify.Application.Tests.Users
|
|||||||
Assert.Equal(1, result.Value.FollowingCount);
|
Assert.Equal(1, result.Value.FollowingCount);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_MutualFollow_ViewerCanSeeFollowersAndIsFollowing()
|
||||||
|
{
|
||||||
|
var viewer = TestEntities.User(1, "viewer");
|
||||||
|
var user = TestEntities.User(2, "user");
|
||||||
|
await SeedAsync(
|
||||||
|
viewer, user,
|
||||||
|
new UserFollow { FollowerId = viewer.Id, FollowedId = user.Id },
|
||||||
|
new UserFollow { FollowerId = user.Id, FollowedId = viewer.Id });
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetUserProfileQuery(user.Id, viewer.Id), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.True(result.Value.IsFollowing);
|
||||||
|
Assert.True(result.Value.CanViewFollowers);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_OneWayFollow_ViewerCannotSeeFollowers()
|
||||||
|
{
|
||||||
|
var viewer = TestEntities.User(1, "viewer");
|
||||||
|
var user = TestEntities.User(2, "user");
|
||||||
|
await SeedAsync(viewer, user, new UserFollow { FollowerId = viewer.Id, FollowedId = user.Id });
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetUserProfileQuery(user.Id, viewer.Id), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.True(result.Value.IsFollowing);
|
||||||
|
Assert.False(result.Value.CanViewFollowers);
|
||||||
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_OwnProfile_CanSeeFollowersButIsNotFollowing()
|
||||||
|
{
|
||||||
|
await SeedAsync(TestEntities.User(1, "user"));
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetUserProfileQuery(1, 1), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.False(result.Value.IsFollowing);
|
||||||
|
Assert.True(result.Value.CanViewFollowers);
|
||||||
|
}
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public async Task Handle_UserMissing_ReturnsNotFound()
|
public async Task Handle_UserMissing_ReturnsNotFound()
|
||||||
{
|
{
|
||||||
|
|||||||
@@ -1,5 +1,6 @@
|
|||||||
using Musify.Application.Tests.TestSupport;
|
using Musify.Application.Tests.TestSupport;
|
||||||
using Musify.Application.Users;
|
using Musify.Application.Users;
|
||||||
|
using Musify.Domain.Entities;
|
||||||
using Xunit;
|
using Xunit;
|
||||||
|
|
||||||
namespace Musify.Application.Tests.Users
|
namespace Musify.Application.Tests.Users
|
||||||
@@ -30,5 +31,18 @@ namespace Musify.Application.Tests.Users
|
|||||||
var user = Assert.Single(result.Value.Items);
|
var user = Assert.Single(result.Value.Items);
|
||||||
Assert.Equal(1, user.Id);
|
Assert.Equal(1, user.Id);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
[Fact]
|
||||||
|
public async Task Handle_Viewer_MarksFollowedUsers()
|
||||||
|
{
|
||||||
|
await SeedAsync(
|
||||||
|
TestEntities.User(1, "viewer"), TestEntities.User(2, "alice"), TestEntities.User(3, "bob"),
|
||||||
|
new UserFollow { FollowerId = 1, FollowedId = 2 });
|
||||||
|
|
||||||
|
var result = await CreateHandler().Handle(new GetUsersQuery(PageNumber: 1, PageSize: 10, UsernameSearch: null, ViewerId: 1), TestContext.Current.CancellationToken);
|
||||||
|
|
||||||
|
Assert.False(result.IsError);
|
||||||
|
Assert.All(result.Value.Items, item => Assert.Equal(item.Id == 2, item.IsFollowedByViewer));
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
+6
@@ -1,3 +1,4 @@
|
|||||||
|
using Microsoft.EntityFrameworkCore;
|
||||||
using Musify.Domain.Entities;
|
using Musify.Domain.Entities;
|
||||||
using Musify.Infrastructure.Tests.TestSupport;
|
using Musify.Infrastructure.Tests.TestSupport;
|
||||||
using Xunit;
|
using Xunit;
|
||||||
@@ -43,6 +44,11 @@ namespace Musify.Infrastructure.Tests.Persistence
|
|||||||
Assert.Equal(originalCreatedAt, user.CreatedAt);
|
Assert.Equal(originalCreatedAt, user.CreatedAt);
|
||||||
Assert.True(user.UpdatedAt > originalCreatedAt,
|
Assert.True(user.UpdatedAt > originalCreatedAt,
|
||||||
$"original-created-at={originalCreatedAt:O}, final-updated-at={user.UpdatedAt:O}");
|
$"original-created-at={originalCreatedAt:O}, final-updated-at={user.UpdatedAt:O}");
|
||||||
|
|
||||||
|
await using var reader = fixture.CreateDatabase();
|
||||||
|
var stored = await reader.Users.SingleAsync(u => u.Id == user.Id, TestContext.Current.CancellationToken);
|
||||||
|
Assert.True(stored.UpdatedAt > stored.CreatedAt,
|
||||||
|
$"stored-created-at={stored.CreatedAt:O}, stored-updated-at={stored.UpdatedAt:O}");
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
+29
-24
@@ -1,39 +1,39 @@
|
|||||||
# Development configuration for deploy/compose.yml + compose.dev.yml.
|
# Development configuration for deploy/compose.yml + compose.dev.yml.
|
||||||
# Copy to deploy/.env and adjust. Not versioned.
|
# Copy to deploy/.env and adjust.
|
||||||
#
|
#
|
||||||
# cp deploy/.env.example deploy/.env
|
# cp deploy/.env.example deploy/.env
|
||||||
|
#
|
||||||
|
# Requires ../../../Infrastructure running first (Postgres, Zitadel,
|
||||||
|
# SeaweedFS, RabbitMQ, Jaeger). See ../../../Infrastructure/README.md.
|
||||||
|
# Everything marked "must match Infrastructure's .env" below has to be the
|
||||||
|
# same value as the one in that file.
|
||||||
|
|
||||||
DOTNET_ENVIRONMENT=Development
|
DOTNET_ENVIRONMENT=Development
|
||||||
|
|
||||||
# --- Public URLs -------------------------------------------------------------
|
# --- Public URLs -------------------------------------------------------------
|
||||||
# How each service is reached from *outside* Docker. `host.docker.internal`
|
# How each service is reached from *outside* Docker. `host.docker.internal`
|
||||||
# resolves to the same address from the browser, from apps running on the host
|
# resolves to the same address from the browser, from apps running on the
|
||||||
# and from inside the containers, which is what makes the OIDC issuer and the
|
# host and from inside the containers, which is what makes the OIDC issuer
|
||||||
# presigned S3 URLs valid in both run modes.
|
# and the presigned S3 URLs valid in both run modes.
|
||||||
PUBLIC_WEB_URL=http://localhost:3000
|
PUBLIC_WEB_URL=http://localhost:3000
|
||||||
PUBLIC_API_URL=http://localhost:5111
|
PUBLIC_API_URL=http://localhost:5111
|
||||||
PUBLIC_STREAM_URL=http://localhost:8081
|
PUBLIC_STREAM_URL=http://localhost:8081
|
||||||
PUBLIC_AUTH_URL=http://host.docker.internal:8080
|
PUBLIC_AUTH_URL=http://host.docker.internal:8080
|
||||||
|
|
||||||
# --- PostgreSQL (shared by Musify and Zitadel) -------------------------------
|
# --- PostgreSQL (Infrastructure's shared instance, Musify's own database) ---
|
||||||
|
# POSTGRES_USER / POSTGRES_PASSWORD must match Infrastructure's .env.
|
||||||
|
POSTGRES_HOST=host.docker.internal
|
||||||
POSTGRES_USER=postgres
|
POSTGRES_USER=postgres
|
||||||
POSTGRES_PASSWORD=postgres
|
POSTGRES_PASSWORD=postgres
|
||||||
POSTGRES_DB=musify_db
|
POSTGRES_DB=musify_db
|
||||||
POSTGRES_HOST_PORT=59000
|
|
||||||
|
|
||||||
# --- Zitadel (OIDC) ----------------------------------------------------------
|
# --- Zitadel (OIDC, shared instance) -----------------------------------------
|
||||||
# The masterkey MUST be exactly 32 characters. The external domain/port form
|
# Path to the admin service-account PAT Infrastructure's Zitadel writes on
|
||||||
# the token issuer, so they have to match PUBLIC_AUTH_URL. Changing them needs
|
# init. Adjust if your Infrastructure checkout lives somewhere else relative
|
||||||
# a re-init: docker compose -f deploy/compose.yml -f deploy/compose.dev.yml down -v
|
# to this one.
|
||||||
ZITADEL_MASTERKEY=MasterkeyNeedsToHave32Characters
|
ZITADEL_ADMIN_PAT_DIR=../../../Infrastructure/zitadel/.output
|
||||||
ZITADEL_EXTERNAL_DOMAIN=host.docker.internal
|
# Allows the OIDC apps this project creates to use plain-http redirect URIs
|
||||||
ZITADEL_EXTERNAL_PORT=8080
|
# (dev only).
|
||||||
ZITADEL_EXTERNAL_SECURE=false
|
|
||||||
ZITADEL_DATABASE_PASSWORD=zitadel
|
|
||||||
# Console login: <username>@zitadel.<external domain>
|
|
||||||
ZITADEL_ADMIN_USERNAME=admin
|
|
||||||
ZITADEL_ADMIN_PASSWORD=Password1!
|
|
||||||
# Allows the OIDC apps to use plain-http redirect URIs (dev only).
|
|
||||||
ZITADEL_DEV_MODE=true
|
ZITADEL_DEV_MODE=true
|
||||||
|
|
||||||
# OIDC clients: the zitadel-init one-shot service (see compose.yml) fills
|
# OIDC clients: the zitadel-init one-shot service (see compose.yml) fills
|
||||||
@@ -42,17 +42,25 @@ AUTH_CLIENT_ID=
|
|||||||
WEB_CLIENT_ID=
|
WEB_CLIENT_ID=
|
||||||
WEB_CLIENT_SECRET=
|
WEB_CLIENT_SECRET=
|
||||||
|
|
||||||
# --- RabbitMQ (MassTransit) --------------------------------------------------
|
# --- RabbitMQ (shared instance, MassTransit) --------------------------------
|
||||||
|
# Must match Infrastructure's .env.
|
||||||
|
RABBITMQ_HOST=host.docker.internal
|
||||||
RABBITMQ_DEFAULT_USER=guest
|
RABBITMQ_DEFAULT_USER=guest
|
||||||
RABBITMQ_DEFAULT_PASS=guest
|
RABBITMQ_DEFAULT_PASS=guest
|
||||||
|
|
||||||
# --- SeaweedFS (S3) ----------------------------------------------------------
|
# --- SeaweedFS (S3, shared instance, Musify's own bucket) --------------------
|
||||||
|
# S3_ACCESS_KEY / S3_SECRET_KEY must match Infrastructure's .env.
|
||||||
|
SEAWEEDFS_HOST=host.docker.internal
|
||||||
S3_ACCESS_KEY=admin_access_key
|
S3_ACCESS_KEY=admin_access_key
|
||||||
S3_SECRET_KEY=admin_secret_key
|
S3_SECRET_KEY=admin_secret_key
|
||||||
S3_BUCKET=webapi-storage
|
S3_BUCKET=webapi-storage
|
||||||
S3_PUBLIC_URL=http://host.docker.internal:8333
|
S3_PUBLIC_URL=http://host.docker.internal:8333
|
||||||
|
S3_INTERNAL_URL=http://host.docker.internal:8333
|
||||||
S3_USE_HTTP=true
|
S3_USE_HTTP=true
|
||||||
|
|
||||||
|
# --- Jaeger (shared instance, tracing) ---------------------------------------
|
||||||
|
JAEGER_HOST=host.docker.internal
|
||||||
|
|
||||||
# --- Web player --------------------------------------------------------------
|
# --- Web player --------------------------------------------------------------
|
||||||
# Key material for the encrypted session cookie: any long random string.
|
# Key material for the encrypted session cookie: any long random string.
|
||||||
SESSION_SECRET=dev-session-secret-change-me-0123456789abcdef
|
SESSION_SECRET=dev-session-secret-change-me-0123456789abcdef
|
||||||
@@ -61,6 +69,3 @@ AUTH_ALLOW_INSECURE_HTTP=true
|
|||||||
|
|
||||||
# --- Misc --------------------------------------------------------------------
|
# --- Misc --------------------------------------------------------------------
|
||||||
AUTH_REQUIRE_HTTPS=false
|
AUTH_REQUIRE_HTTPS=false
|
||||||
PGADMIN_DEFAULT_EMAIL=admin@musify.dev
|
|
||||||
PGADMIN_DEFAULT_PASSWORD=admin
|
|
||||||
PGADMIN_HOST_PORT=5050
|
|
||||||
|
|||||||
+32
-28
@@ -1,41 +1,41 @@
|
|||||||
# Production configuration for deploy/compose.yml + compose.prod.yml.
|
# Production configuration for deploy/compose.yml + compose.prod.yml.
|
||||||
# Copy to deploy/.env.prod, then:
|
# Copy to deploy/.env.prod, then:
|
||||||
#
|
#
|
||||||
# 1. replace every `example.com` with your domain,
|
# 1. replace every `*.example.com` with your actual domains (each service
|
||||||
|
# below has its own, independent domain — they don't need to share a
|
||||||
|
# root, e.g. for DNS providers that don't allow multi-level
|
||||||
|
# subdomains),
|
||||||
# 2. replace every CHANGE_ME with a strong random value
|
# 2. replace every CHANGE_ME with a strong random value
|
||||||
# (`openssl rand -hex 24`; the Zitadel masterkey needs exactly 32 chars),
|
# (`openssl rand -hex 24`),
|
||||||
# 3. point these DNS records at the server, all proxied/TLS-terminated the
|
# 3. point each domain below at the server and add its vhost to the shared
|
||||||
# same way: example.com, www, api, auth, stream, s3.
|
# edge (Infrastructure's edge/conf.d).
|
||||||
#
|
#
|
||||||
# Not versioned. Keep it readable only by the deploy user (chmod 600).
|
# Requires ../../../Infrastructure running first, on the same Docker host,
|
||||||
|
# with its own `infra-net` network already created. See
|
||||||
|
# ../../../Infrastructure/README.md. Not versioned. Keep it readable only by
|
||||||
|
# the deploy user (chmod 600).
|
||||||
|
|
||||||
DOTNET_ENVIRONMENT=Production
|
DOTNET_ENVIRONMENT=Production
|
||||||
|
|
||||||
# --- Domain ------------------------------------------------------------------
|
# --- Domains -------------------------------------------------------------
|
||||||
# ROOT_DOMAIN drives the nginx vhosts; the URLs below must agree with it.
|
# Public URLs. They must match the vhosts of the shared edge nginx
|
||||||
ROOT_DOMAIN=example.com
|
# (Infrastructure's edge/conf.d), which is what serves them; the domains are
|
||||||
PUBLIC_WEB_URL=https://example.com
|
# independent, they don't need to share a root.
|
||||||
PUBLIC_API_URL=https://api.example.com
|
PUBLIC_WEB_URL=https://musify.example.com
|
||||||
PUBLIC_STREAM_URL=https://stream.example.com
|
PUBLIC_API_URL=https://musifyapi.example.com
|
||||||
|
PUBLIC_STREAM_URL=https://musifystream.example.com
|
||||||
PUBLIC_AUTH_URL=https://auth.example.com
|
PUBLIC_AUTH_URL=https://auth.example.com
|
||||||
S3_PUBLIC_URL=https://s3.example.com
|
S3_PUBLIC_URL=https://s3.example.com
|
||||||
|
|
||||||
# --- PostgreSQL (shared by Musify and Zitadel) -------------------------------
|
# --- PostgreSQL (Infrastructure's shared instance, Musify's own database) ---
|
||||||
|
# POSTGRES_USER / POSTGRES_PASSWORD must match Infrastructure's .env.prod.
|
||||||
|
POSTGRES_HOST=postgres
|
||||||
POSTGRES_USER=postgres
|
POSTGRES_USER=postgres
|
||||||
POSTGRES_PASSWORD=CHANGE_ME
|
POSTGRES_PASSWORD=CHANGE_ME
|
||||||
POSTGRES_DB=musify_db
|
POSTGRES_DB=musify_db
|
||||||
|
|
||||||
# --- Zitadel (OIDC) ----------------------------------------------------------
|
# --- Zitadel (OIDC, shared instance) -----------------------------------------
|
||||||
# The masterkey MUST be exactly 32 characters and can never be rotated in place.
|
ZITADEL_ADMIN_PAT_DIR=../../../Infrastructure/zitadel/.output
|
||||||
ZITADEL_MASTERKEY=CHANGE_ME_32_CHARACTERS_EXACTLY_
|
|
||||||
ZITADEL_EXTERNAL_DOMAIN=auth.example.com
|
|
||||||
ZITADEL_EXTERNAL_PORT=443
|
|
||||||
ZITADEL_EXTERNAL_SECURE=true
|
|
||||||
ZITADEL_DATABASE_PASSWORD=CHANGE_ME
|
|
||||||
# Console login: <username>@zitadel.<external domain>
|
|
||||||
ZITADEL_ADMIN_USERNAME=admin
|
|
||||||
ZITADEL_ADMIN_PASSWORD=CHANGE_ME
|
|
||||||
# Keep OIDC apps strict about https redirect URIs in production.
|
|
||||||
ZITADEL_DEV_MODE=false
|
ZITADEL_DEV_MODE=false
|
||||||
|
|
||||||
# OIDC clients: the zitadel-init one-shot service (see compose.yml) fills
|
# OIDC clients: the zitadel-init one-shot service (see compose.yml) fills
|
||||||
@@ -44,22 +44,26 @@ AUTH_CLIENT_ID=
|
|||||||
WEB_CLIENT_ID=
|
WEB_CLIENT_ID=
|
||||||
WEB_CLIENT_SECRET=
|
WEB_CLIENT_SECRET=
|
||||||
|
|
||||||
# --- RabbitMQ (MassTransit) --------------------------------------------------
|
# --- RabbitMQ (shared instance, MassTransit) --------------------------------
|
||||||
|
RABBITMQ_HOST=rabbitmq
|
||||||
RABBITMQ_DEFAULT_USER=musify
|
RABBITMQ_DEFAULT_USER=musify
|
||||||
RABBITMQ_DEFAULT_PASS=CHANGE_ME
|
RABBITMQ_DEFAULT_PASS=CHANGE_ME
|
||||||
|
|
||||||
# --- SeaweedFS (S3) ----------------------------------------------------------
|
# --- SeaweedFS (S3, shared instance, Musify's own bucket) --------------------
|
||||||
|
SEAWEEDFS_HOST=seaweedfs
|
||||||
S3_ACCESS_KEY=CHANGE_ME
|
S3_ACCESS_KEY=CHANGE_ME
|
||||||
S3_SECRET_KEY=CHANGE_ME
|
S3_SECRET_KEY=CHANGE_ME
|
||||||
S3_BUCKET=webapi-storage
|
S3_BUCKET=webapi-storage
|
||||||
|
S3_INTERNAL_URL=http://seaweedfs:8333
|
||||||
S3_USE_HTTP=false
|
S3_USE_HTTP=false
|
||||||
|
|
||||||
|
# --- Jaeger (shared instance, tracing) ---------------------------------------
|
||||||
|
JAEGER_HOST=jaeger
|
||||||
|
|
||||||
# --- Web player --------------------------------------------------------------
|
# --- Web player --------------------------------------------------------------
|
||||||
# Key material for the encrypted session cookie. Rotating it logs everyone out.
|
# Key material for the encrypted session cookie. Rotating it logs everyone out.
|
||||||
SESSION_SECRET=CHANGE_ME
|
SESSION_SECRET=CHANGE_ME
|
||||||
AUTH_ALLOW_INSECURE_HTTP=false
|
AUTH_ALLOW_INSECURE_HTTP=false
|
||||||
|
|
||||||
# --- Misc --------------------------------------------------------------------
|
# --- Misc --------------------------------------------------------------------
|
||||||
AUTH_REQUIRE_HTTPS=true
|
AUTH_REQUIRE_HTTPS=true
|
||||||
# Largest request body nginx accepts (audio uploads go through the web player).
|
|
||||||
MAX_UPLOAD_SIZE=1024m
|
|
||||||
+121
-87
@@ -1,51 +1,65 @@
|
|||||||
# Deploy
|
# Deploy
|
||||||
|
|
||||||
Everything needed to run Musify, in development and in production, from the
|
Everything needed to run Musify itself, in development and in production,
|
||||||
same set of compose files — no wrapper scripts, just `docker compose`.
|
from the same set of compose files, no wrapper scripts, just `docker
|
||||||
|
compose`. Postgres, Zitadel, SeaweedFS, RabbitMQ and Jaeger are **not**
|
||||||
|
started here: they're shared infrastructure that lives in the separate
|
||||||
|
`Infrastructure` repository and has to already be running (see its own
|
||||||
|
README). `../../Infrastructure` below is just an example path, assuming
|
||||||
|
that repo happens to be checked out as a sibling of this one. Adjust it to
|
||||||
|
wherever you actually cloned it.
|
||||||
|
|
||||||
| File | Role |
|
| File | Role |
|
||||||
|---|---|
|
|---|---|
|
||||||
| `compose.yml` | Every service, including the one-shot bootstrap jobs. No published ports, no domains, no environment-specific values. |
|
| `compose.yml` | Musify's own services, including the one-shot bootstrap jobs. No published ports, no domains, no environment-specific values. |
|
||||||
| `compose.dev.yml` | Development overlay: publishes the ports to the host, adds pgAdmin. |
|
| `compose.dev.yml` | Development overlay: publishes Musify's ports to the host. |
|
||||||
| `compose.prod.yml` | Production overlay: adds the nginx edge. |
|
| `compose.prod.yml` | Production overlay: joins Infrastructure's `infra-net` network. |
|
||||||
| `.env` / `.env.prod` | The values compose interpolates. Not versioned; templates in `.env.example` / `.env.prod.example`. |
|
| `.env` / `.env.prod` | The values compose interpolates. Not versioned; templates in `.env.example` / `.env.prod.example`. |
|
||||||
|
|
||||||
The four applications (`api`, `worker`, `gateway`, `web-player`) sit behind the
|
The four applications (`api`, `worker`, `gateway`, `web-player`) sit behind
|
||||||
`apps` compose profile, so a plain `up` starts infrastructure only — the usual
|
the `apps` compose profile, so a plain `up` starts only the one-shot
|
||||||
development setup, with the apps running from the IDE.
|
bootstrap jobs, the usual development setup, with the apps running from the
|
||||||
|
IDE.
|
||||||
|
|
||||||
### The one-shot bootstrap jobs
|
### The one-shot bootstrap jobs
|
||||||
|
|
||||||
Three services in `compose.yml` run once and exit, so a plain `docker compose
|
Four services in `compose.yml` run once and exit, so a plain `docker compose
|
||||||
up -d` leaves the stack fully ready with no manual steps:
|
up -d` leaves the stack fully ready with no manual steps:
|
||||||
|
|
||||||
| Service | Does |
|
| Service | Does |
|
||||||
|---|---|
|
|---|---|
|
||||||
|
| `wait-for-postgres` | Waits for Infrastructure's Postgres to accept connections. Replaces an in-file health-check `depends_on`, which only works within the same compose file. |
|
||||||
| `keys-init` | Generates the RS256 stream-ticket key pair in `./keys`, if it is not there already. |
|
| `keys-init` | Generates the RS256 stream-ticket key pair in `./keys`, if it is not there already. |
|
||||||
| `migrate` | Applies the EF Core migrations (built from `backend/Musify.Migrator.Dockerfile`). |
|
| `migrate` | Applies the EF Core migrations (built from `backend/Musify.Migrator.Dockerfile`). |
|
||||||
| `zitadel-init` | Creates the Musify project and its two OIDC apps in Zitadel (or reuses them), then writes `AUTH_CLIENT_ID` / `WEB_CLIENT_ID` / `WEB_CLIENT_SECRET` into the env file. |
|
| `seaweedfs-init` | Creates Musify's own bucket on the shared SeaweedFS, if it doesn't exist yet. |
|
||||||
|
| `zitadel-init` | Creates the Musify project and its OIDC apps in the shared Zitadel (or reuses them), then writes `AUTH_CLIENT_ID` / `WEB_CLIENT_ID` / `WEB_CLIENT_SECRET` / `NATIVE_CLIENT_ID` into the env file. |
|
||||||
|
|
||||||
`api` and `worker` wait for `migrate`; `api` and `gateway` wait for
|
`api` and `worker` wait for `migrate`; `api` waits for `keys-init`
|
||||||
`keys-init` (`depends_on: condition: service_completed_successfully`). They do
|
(`depends_on: condition: service_completed_successfully`). Nothing waits on
|
||||||
**not** wait for `zitadel-init` — compose interpolates `${AUTH_CLIENT_ID}` once,
|
`zitadel-init`, `seaweedfs-init` or the shared infrastructure services
|
||||||
when a command starts, so a container that starts in the *same* `up` as
|
directly. Compose interpolates `${AUTH_CLIENT_ID}` once, when a command
|
||||||
`zitadel-init` would still get the empty value it had before provisioning ran.
|
starts, so a container that started in the *same* `up` as `zitadel-init`
|
||||||
That's why bringing up the apps is a second command: by the time you run it,
|
would still get the empty value it had before provisioning ran. That's why
|
||||||
|
bringing up the apps is a second command: by the time you run it,
|
||||||
`zitadel-init` has already finished and the env file has real values.
|
`zitadel-init` has already finished and the env file has real values.
|
||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
## Development
|
## Development
|
||||||
|
|
||||||
|
Start the separate `Infrastructure` repository first, from wherever you
|
||||||
|
cloned it (see its own README for the exact command). Then, from this
|
||||||
|
repository's root:
|
||||||
|
|
||||||
```sh
|
```sh
|
||||||
docker compose -f deploy/compose.yml -f deploy/compose.dev.yml up -d
|
docker compose -f deploy/compose.yml -f deploy/compose.dev.yml up -d
|
||||||
docker compose -f deploy/compose.yml -f deploy/compose.dev.yml --profile apps up -d --build
|
docker compose -f deploy/compose.yml -f deploy/compose.dev.yml --profile apps up -d --build
|
||||||
```
|
```
|
||||||
|
|
||||||
The first command starts the infrastructure and runs the bootstrap jobs
|
The first command starts Musify's own bootstrap jobs (Postgres wait, keys,
|
||||||
(keys, migrations, Zitadel project/apps). The second builds and runs `api`,
|
migrations, the bucket, the Zitadel project/apps). The second builds and runs
|
||||||
`worker`, `gateway` and `web-player` in Docker too — skip it to run the apps
|
`api`, `worker`, `gateway` and `web-player` in Docker too. Skip it to run the
|
||||||
from the IDE instead (the usual dev loop):
|
apps from the IDE instead (the usual dev loop):
|
||||||
|
|
||||||
```sh
|
```sh
|
||||||
dotnet run --project backend/Hosts/Musify.Api # :5111
|
dotnet run --project backend/Hosts/Musify.Api # :5111
|
||||||
@@ -55,86 +69,117 @@ npm --prefix web-player run dev # :5173
|
|||||||
```
|
```
|
||||||
|
|
||||||
When running the apps from the IDE, `Musify.Api` reads `Authentication:*`
|
When running the apps from the IDE, `Musify.Api` reads `Authentication:*`
|
||||||
from user-secrets and `web-player` reads its own `web-player/.env` — neither
|
from user-secrets and `web-player` reads its own `web-player/.env`, neither
|
||||||
looks at `deploy/.env`. Copy the three values `zitadel-init` wrote
|
looks at `deploy/.env`. Copy the four values `zitadel-init` wrote
|
||||||
(`AUTH_CLIENT_ID`, `WEB_CLIENT_ID`, `WEB_CLIENT_SECRET`) into
|
(`AUTH_CLIENT_ID`, `WEB_CLIENT_ID`, `WEB_CLIENT_SECRET`, `NATIVE_CLIENT_ID`)
|
||||||
`dotnet user-secrets set Authentication:ClientId <value> --project
|
into `dotnet user-secrets set Authentication:ClientId <value> --project
|
||||||
backend/Hosts/Musify.Api` and into `web-player/.env` once; they stay valid
|
backend/Hosts/Musify.Api` and into `web-player/.env` once. They stay valid
|
||||||
until you wipe the Zitadel volume.
|
until Infrastructure's Zitadel volume is wiped.
|
||||||
|
|
||||||
Add `--profile tools` to also start pgAdmin.
|
|
||||||
|
|
||||||
### Ports
|
### Ports
|
||||||
|
|
||||||
| Service | Port | Used by |
|
| Service | Port | Used by |
|
||||||
|---|---|---|
|
|---|---|---|
|
||||||
| PostgreSQL | `59000` → 5432 | API + Worker (EF Core) and Zitadel |
|
| Musify.Api | `5111` | `/scalar/v1`, `/openapi/v1.json` |
|
||||||
| Zitadel | `8080` | OIDC/OAuth2, console at `/ui/console` |
|
| Musify.StreamingGateway | `8081` | `/health`, `/media/...` |
|
||||||
| RabbitMQ | `5672` / `15672` | MassTransit / management UI |
|
| web-player | `5173` (`npm run dev`) or `3000` (container) | |
|
||||||
| SeaweedFS | `8333` / `8888` / `9333` | S3 / filer (→ gateway) / master |
|
|
||||||
| Jaeger | `16686` / `4317` / `4318` | traces UI / OTLP gRPC / OTLP HTTP |
|
|
||||||
| pgAdmin (`--profile tools`) | `5050` | Postgres UI |
|
|
||||||
| API, gateway, web player (`--profile apps`) | `5111` / `8081` / `3000` | |
|
|
||||||
|
|
||||||
Host requirements: .NET 10 SDK, Node 22, Docker Desktop. (ffmpeg is only
|
Postgres, Zitadel, SeaweedFS, RabbitMQ and Jaeger ports come from
|
||||||
needed on the host if you run the Worker outside Docker — inside Docker its
|
Infrastructure. See its own README for that table.
|
||||||
image already has it.)
|
|
||||||
|
Host requirements: .NET 10 SDK, Node 22, Docker Desktop. ffmpeg is only
|
||||||
|
needed on the host if you run the Worker outside Docker; inside Docker its
|
||||||
|
image already has it.
|
||||||
|
|
||||||
### Why `host.docker.internal`
|
### Why `host.docker.internal`
|
||||||
|
|
||||||
`PUBLIC_AUTH_URL` and `S3_PUBLIC_URL` point at `host.docker.internal`, which
|
`PUBLIC_AUTH_URL` and `S3_PUBLIC_URL` point at `host.docker.internal`, which
|
||||||
resolves to the same address from the browser, from apps running on the host
|
resolves to the same address from the browser, from apps running on the host
|
||||||
and from inside the containers. That keeps the OIDC token issuer and the
|
and from inside the containers. That keeps the OIDC token issuer and the
|
||||||
presigned S3 URLs valid whichever way the apps are running. Zitadel routes by
|
presigned S3 URLs valid whichever way the apps are running, and is the same
|
||||||
the `Host` header, so the domain has to be identical everywhere; changing it
|
address Infrastructure's own dev ports are published on.
|
||||||
requires wiping the Postgres/Zitadel volumes and starting over (`docker
|
|
||||||
compose -f compose.yml -f compose.dev.yml down -v`, then `up -d` again).
|
|
||||||
|
|
||||||
### Resetting
|
### Resetting
|
||||||
|
|
||||||
```sh
|
```sh
|
||||||
docker compose -f deploy/compose.yml -f deploy/compose.dev.yml down # stop, keep data
|
docker compose -f deploy/compose.yml -f deploy/compose.dev.yml down # stop, keep data
|
||||||
docker compose -f deploy/compose.yml -f deploy/compose.dev.yml down -v # stop, wipe volumes
|
docker compose -f deploy/compose.yml -f deploy/compose.dev.yml down -v # stop, wipe Musify's own volumes
|
||||||
```
|
```
|
||||||
|
|
||||||
|
This only touches Musify's own containers and volumes. Resetting Zitadel or
|
||||||
|
SeaweedFS is done from the `Infrastructure` repo instead.
|
||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
## Production
|
## Production
|
||||||
|
|
||||||
On a server with a domain, from the repository root:
|
Musify doesn't run its own nginx: the shared edge in Infrastructure
|
||||||
|
(`edge/`) owns host ports 80/443, terminates TLS for every project's domains
|
||||||
|
and proxies to Musify's containers over `infra-net`. On a server with
|
||||||
|
domains:
|
||||||
|
|
||||||
|
1. Bring up the `Infrastructure` repo first, with its own `.env.prod` filled
|
||||||
|
in. It creates the `infra-net` network this stack joins.
|
||||||
|
2. Bring up its `edge/` (see Infrastructure's README) and add Musify's
|
||||||
|
vhosts and certificate there (`edge/conf.d/musify.conf`). The five
|
||||||
|
public hostnames (web, api, stream, auth, s3) are independent: they
|
||||||
|
don't need to share a root domain.
|
||||||
|
3. From this repository's root:
|
||||||
|
|
||||||
```sh
|
```sh
|
||||||
cp deploy/.env.prod.example deploy/.env.prod
|
cp deploy/.env.prod.example deploy/.env.prod
|
||||||
```
|
```
|
||||||
|
|
||||||
Fill it in:
|
Replace every `*.example.com` with your domains (they must match the
|
||||||
|
edge's vhosts) and every `CHANGE_ME` with a random value
|
||||||
|
(`openssl rand -hex 24`). `chmod 600 deploy/.env.prod`.
|
||||||
|
|
||||||
1. Replace every `example.com` with your domain and every `CHANGE_ME` with a
|
Then:
|
||||||
random value (`openssl rand -hex 24`; the Zitadel masterkey needs exactly
|
|
||||||
32 characters). `chmod 600 deploy/.env.prod`.
|
|
||||||
2. Drop **`deploy/nginx/certs/origin.pem` and `origin.key`** — the origin
|
|
||||||
certificate nginx serves. The stack assumes Cloudflare in *Full (strict)*
|
|
||||||
mode in front of it, so a Cloudflare Origin Certificate is enough; any
|
|
||||||
certificate valid for the five hostnames works.
|
|
||||||
3. Point `example.com`, `www`, `api`, `auth`, `stream` and `s3` at the server,
|
|
||||||
proxied/TLS-terminated the same way.
|
|
||||||
|
|
||||||
Then, same two commands as development, with the prod env file and overlay:
|
|
||||||
|
|
||||||
```sh
|
```sh
|
||||||
docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml up -d
|
docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml up -d
|
||||||
docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml --profile apps up -d --build
|
docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml --profile apps up -d --build
|
||||||
```
|
```
|
||||||
|
|
||||||
The first command starts the infrastructure, the nginx edge and the bootstrap
|
The first command starts Musify's bootstrap jobs (`zitadel-init` and
|
||||||
jobs; the second builds and starts the applications, already wired to the
|
`seaweedfs-init` reach Zitadel/SeaweedFS through the public `auth.`/`s3.`
|
||||||
OIDC client ids the first one produced. Both are idempotent — run them again
|
URLs, so the edge has to be serving them by then); the second builds and
|
||||||
to deploy a new version (`docker compose ... --profile apps up -d --build`
|
starts the applications, already wired to the OIDC client ids the first one
|
||||||
alone is enough for an update once the stack already exists; it reuses the
|
produced. Both are idempotent, so running them again deploys a new version
|
||||||
infra containers and only rebuilds and restarts the apps).
|
(`... --profile apps up -d --build` alone is enough for an update once the
|
||||||
|
stack already exists).
|
||||||
|
|
||||||
Only nginx publishes ports (80/443); everything else is reachable only on the
|
Nothing in this stack publishes a port; everything is reachable only on the
|
||||||
internal compose network.
|
internal `infra-net` network, through the edge.
|
||||||
|
|
||||||
|
### Continuous deployment
|
||||||
|
|
||||||
|
`.gitea/workflows/ci-cd.yml` runs the backend and web-player checks on every
|
||||||
|
push/PR, and on a successful push to `master` deploys automatically. The
|
||||||
|
deploy job runs on `gitea-runner`, the self-hosted Gitea Actions runner on
|
||||||
|
the production host (`~/gitea/docker-compose.yml`), configured by
|
||||||
|
`~/gitea/runner-data/config.yaml` (`CONFIG_FILE`): job containers are pinned
|
||||||
|
to the `gitea_gitea` network so `actions/checkout` can resolve `gitea`,
|
||||||
|
`ubuntu-latest` is `node:22` (JS actions need Node inside the job
|
||||||
|
container), and a `production:host` label runs steps natively.
|
||||||
|
|
||||||
|
Native mode means the label's steps execute inside the runner container
|
||||||
|
itself rather than a fresh one. That container has `/var/run/docker.sock`
|
||||||
|
mounted, so `docker`/`docker compose` there still control the host's real
|
||||||
|
Docker daemon (sibling containers, "Docker outside of Docker"). Its compose
|
||||||
|
service also bind-mounts `/home/ubuntu/deploys` into itself at the *same*
|
||||||
|
path, which matters because compose's own relative bind mounts (`./keys`)
|
||||||
|
are resolved by the host daemon against that literal path; mounting it
|
||||||
|
anywhere else would silently bind the wrong (empty) directory.
|
||||||
|
|
||||||
|
`DEPLOY_PATH` (hardcoded in the workflow as `/home/ubuntu/deploys/musify`)
|
||||||
|
is a persistent clone on that host, reused instead of a fresh checkout each
|
||||||
|
run since `deploy/.env.prod` is untracked and only lives there.
|
||||||
|
`/home/ubuntu/deploys/SharedServices` is the sibling clone of the
|
||||||
|
`Infrastructure` repo (shared services + `edge/`), already running; its
|
||||||
|
`.env.prod` shares the Postgres/RabbitMQ/S3 credentials with Musify's own
|
||||||
|
`deploy/.env.prod` by design (see that repo's README). The edge is deployed
|
||||||
|
from that repo, not from this pipeline.
|
||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
@@ -142,17 +187,12 @@ internal compose network.
|
|||||||
|
|
||||||
```
|
```
|
||||||
deploy/
|
deploy/
|
||||||
├─ compose.yml # all services, incl. keys-init / migrate / zitadel-init
|
├─ compose.yml # Musify's own services: wait-for-postgres, keys-init,
|
||||||
├─ compose.dev.yml # dev overlay: host ports, pgAdmin
|
│ # migrate, seaweedfs-init, zitadel-init, api/worker/gateway/web-player
|
||||||
├─ compose.prod.yml # prod overlay: nginx edge
|
├─ compose.dev.yml # dev overlay: host ports
|
||||||
|
├─ compose.prod.yml # prod overlay: joins infra-net
|
||||||
├─ .env.example / .env.prod.example
|
├─ .env.example / .env.prod.example
|
||||||
├─ keys/ # RS256 stream-ticket key pair (generated)
|
└─ keys/ # RS256 stream-ticket key pair (generated)
|
||||||
├─ nginx/
|
|
||||||
│ ├─ templates/default.conf.template # vhosts, rendered with the domain by envsubst
|
|
||||||
│ ├─ snippets/proxy.conf # proxy headers shared by every vhost
|
|
||||||
│ └─ certs/ # origin.pem + origin.key (not versioned)
|
|
||||||
└─ zitadel/
|
|
||||||
└─ .output/ # PAT written by Zitadel on init (not versioned)
|
|
||||||
```
|
```
|
||||||
|
|
||||||
The application images are built from `backend/Musify.*.Dockerfile` (build
|
The application images are built from `backend/Musify.*.Dockerfile` (build
|
||||||
@@ -161,18 +201,12 @@ and `web-player/Dockerfile`.
|
|||||||
|
|
||||||
## Notes
|
## Notes
|
||||||
|
|
||||||
- **S3 credentials** are rendered into the SeaweedFS identity file from
|
- **S3 credentials, the Postgres admin user, the RabbitMQ user and the
|
||||||
`S3_ACCESS_KEY` / `S3_SECRET_KEY` when the container starts, so they cannot
|
Zitadel masterkey** all live in Infrastructure's own `.env`/`.env.prod` now.
|
||||||
drift from the ones the apps are given.
|
This project's env files only duplicate the values that have to match (see
|
||||||
|
the comments in `.env.example`).
|
||||||
- **Zitadel** runs `start-from-init`, which is idempotent: the same command
|
- **Zitadel** runs `start-from-init`, which is idempotent: the same command
|
||||||
works on an empty volume and on an existing one.
|
works on an empty volume and on an existing one.
|
||||||
- **`latest` Zitadel images** require the standalone Login UI v2 container,
|
- **Re-running `zitadel-init`, `seaweedfs-init` or `migrate`** is safe: they
|
||||||
which this stack does not run. `zitadel-init` turns that requirement off so
|
reuse the existing project/apps/bucket by name, and `migrate` is a normal
|
||||||
the built-in `/ui/login` is used.
|
`dotnet ef database update`, a no-op once the schema is current.
|
||||||
- **`AppSettings*.json` are PascalCase** in `Musify.Api` and `Musify.Worker`;
|
|
||||||
on case-sensitive Linux the host looks for `appsettings*.json`, so their
|
|
||||||
Dockerfiles create lowercase copies at publish time.
|
|
||||||
- **Re-running `zitadel-init` or `migrate`** is safe: the former reuses the
|
|
||||||
existing project/apps by name (regenerating the web app's secret only if it
|
|
||||||
is missing from the env file), the latter is a normal `dotnet ef database
|
|
||||||
update`, a no-op once the schema is current.
|
|
||||||
|
|||||||
+16
-50
@@ -1,67 +1,36 @@
|
|||||||
name: musify-dev
|
name: musify-dev
|
||||||
|
|
||||||
# Development overlay: publishes the infrastructure ports to the host so the
|
# Development overlay: publishes Musify's own ports to the host so the .NET
|
||||||
# .NET apps and the web player can run from the IDE, and adds pgAdmin.
|
# apps and the web player can run from the IDE.
|
||||||
#
|
#
|
||||||
# docker compose -f deploy/compose.yml -f deploy/compose.dev.yml up -d
|
# docker compose -f deploy/compose.yml -f deploy/compose.dev.yml up -d
|
||||||
# docker compose -f deploy/compose.yml -f deploy/compose.dev.yml --profile apps up -d --build
|
# docker compose -f deploy/compose.yml -f deploy/compose.dev.yml --profile apps up -d --build
|
||||||
#
|
#
|
||||||
# `host.docker.internal` (see ZITADEL_EXTERNAL_DOMAIN / *_PUBLIC_URL in .env)
|
# Postgres/Zitadel/SeaweedFS/RabbitMQ/Jaeger ports come from Infrastructure's
|
||||||
# resolves to the same address from the browser, from apps on the host and from
|
# own compose.dev.yml, not from here. See ../../../Infrastructure/README.md.
|
||||||
# inside the containers, so the OIDC issuer and the presigned S3 URLs are valid
|
# `host.docker.internal` resolves to the same address from the browser, from
|
||||||
# in both modes. `extra_hosts` gives Linux containers that same name.
|
# apps on the host and from inside these containers, which is what makes the
|
||||||
|
# OIDC issuer and the presigned S3 URLs valid in both run modes. `extra_hosts`
|
||||||
|
# gives Linux containers that same name.
|
||||||
|
|
||||||
x-host-alias: &host-alias
|
x-host-alias: &host-alias
|
||||||
- "${ZITADEL_EXTERNAL_DOMAIN}:host-gateway"
|
- "host.docker.internal:host-gateway"
|
||||||
|
|
||||||
services:
|
services:
|
||||||
postgres:
|
wait-for-postgres:
|
||||||
ports:
|
extra_hosts: *host-alias
|
||||||
- "${POSTGRES_HOST_PORT}:5432"
|
|
||||||
|
|
||||||
zitadel:
|
seaweedfs-init:
|
||||||
ports:
|
extra_hosts: *host-alias
|
||||||
- "${ZITADEL_EXTERNAL_PORT}:8080"
|
|
||||||
|
|
||||||
rabbitmq:
|
|
||||||
ports:
|
|
||||||
- "5672:5672" # AMQP -> MassTransit
|
|
||||||
- "15672:15672" # management UI
|
|
||||||
|
|
||||||
seaweedfs:
|
|
||||||
ports:
|
|
||||||
- "9333:9333" # master / admin UI
|
|
||||||
- "8888:8888" # filer -> StreamingGateway
|
|
||||||
- "8333:8333" # S3 API -> API, Worker and presigned uploads
|
|
||||||
|
|
||||||
jaeger:
|
|
||||||
ports:
|
|
||||||
- "16686:16686" # UI
|
|
||||||
- "4317:4317" # OTLP gRPC
|
|
||||||
- "4318:4318" # OTLP HTTP
|
|
||||||
|
|
||||||
# zitadel-init (compose.yml) edits whichever env file is mounted here.
|
# zitadel-init (compose.yml) edits whichever env file is mounted here.
|
||||||
zitadel-init:
|
zitadel-init:
|
||||||
|
extra_hosts: *host-alias
|
||||||
volumes:
|
volumes:
|
||||||
- ./.env:/workspace/.env
|
- ./.env:/workspace/.env
|
||||||
|
|
||||||
# Optional Postgres UI: docker compose ... --profile tools up -d
|
migrate:
|
||||||
pgadmin:
|
extra_hosts: *host-alias
|
||||||
profiles: [tools]
|
|
||||||
image: dpage/pgadmin4:latest
|
|
||||||
container_name: musify-pgadmin
|
|
||||||
environment:
|
|
||||||
PGADMIN_DEFAULT_EMAIL: ${PGADMIN_DEFAULT_EMAIL}
|
|
||||||
PGADMIN_DEFAULT_PASSWORD: ${PGADMIN_DEFAULT_PASSWORD}
|
|
||||||
PGADMIN_CONFIG_SERVER_MODE: "False"
|
|
||||||
depends_on:
|
|
||||||
postgres:
|
|
||||||
condition: service_healthy
|
|
||||||
ports:
|
|
||||||
- "${PGADMIN_HOST_PORT}:80"
|
|
||||||
volumes:
|
|
||||||
- pgadmin_data:/var/lib/pgadmin
|
|
||||||
restart: unless-stopped
|
|
||||||
|
|
||||||
api:
|
api:
|
||||||
ports:
|
ports:
|
||||||
@@ -80,6 +49,3 @@ services:
|
|||||||
ports:
|
ports:
|
||||||
- "3000:3000"
|
- "3000:3000"
|
||||||
extra_hosts: *host-alias
|
extra_hosts: *host-alias
|
||||||
|
|
||||||
volumes:
|
|
||||||
pgadmin_data:
|
|
||||||
|
|||||||
+10
-39
@@ -1,50 +1,21 @@
|
|||||||
name: musify-prod
|
name: musify-prod
|
||||||
|
|
||||||
# Production overlay: adds the nginx edge and the host-mounted secrets.
|
# Production overlay: joins Infrastructure's shared `infra-net` network so
|
||||||
# Nothing but nginx publishes a port; every other service is reachable only on
|
# this project's containers can reach Postgres, Zitadel, SeaweedFS, RabbitMQ
|
||||||
# the internal `musify-prod_default` network.
|
# and Jaeger by container name, and so the shared edge nginx (Infrastructure's
|
||||||
|
# `edge/`, which owns host ports 80/443 and terminates TLS for every domain)
|
||||||
|
# can reach api, gateway and web-player. Nothing here publishes a port.
|
||||||
#
|
#
|
||||||
# docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml up -d
|
# docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml up -d
|
||||||
# docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml --profile apps up -d --build
|
# docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml --profile apps up -d --build
|
||||||
#
|
|
||||||
# The first command starts the infrastructure, the nginx edge and the
|
networks:
|
||||||
# one-shot bootstrap jobs (stream keys, migrations, Zitadel project/apps); the
|
default:
|
||||||
# second builds and starts the applications, picking up the OIDC client ids
|
name: infra-net
|
||||||
# the first command just wrote into deploy/.env.prod. See deploy/README.md.
|
external: true
|
||||||
|
|
||||||
services:
|
services:
|
||||||
# zitadel-init (compose.yml) edits whichever env file is mounted here.
|
# zitadel-init (compose.yml) edits whichever env file is mounted here.
|
||||||
zitadel-init:
|
zitadel-init:
|
||||||
volumes:
|
volumes:
|
||||||
- ./.env.prod:/workspace/.env
|
- ./.env.prod:/workspace/.env
|
||||||
|
|
||||||
# Every vhost proxies through `set $upstream ...; proxy_pass $upstream;`
|
|
||||||
# with the embedded Docker DNS resolver (see nginx/templates), so nginx
|
|
||||||
# does not need api/gateway/web-player to exist at its own startup — only
|
|
||||||
# at request time. That is what lets it start in the FIRST command
|
|
||||||
# (profile-gated services aren't part of that invocation yet) and still be
|
|
||||||
# reachable at the auth domain for zitadel-init to provision through.
|
|
||||||
nginx:
|
|
||||||
image: nginx:1.27-alpine
|
|
||||||
container_name: musify-nginx
|
|
||||||
depends_on:
|
|
||||||
zitadel:
|
|
||||||
condition: service_started
|
|
||||||
environment:
|
|
||||||
# The entrypoint renders /etc/nginx/templates/*.template with envsubst.
|
|
||||||
# The filter keeps it from eating nginx's own $host / $remote_addr.
|
|
||||||
MUSIFY_ROOT_DOMAIN: ${ROOT_DOMAIN}
|
|
||||||
MUSIFY_MAX_UPLOAD_SIZE: ${MAX_UPLOAD_SIZE:-1024m}
|
|
||||||
NGINX_ENVSUBST_FILTER: ^MUSIFY_
|
|
||||||
ports:
|
|
||||||
- "80:80"
|
|
||||||
- "443:443"
|
|
||||||
volumes:
|
|
||||||
- ./nginx/templates:/etc/nginx/templates:ro
|
|
||||||
- ./nginx/snippets:/etc/nginx/snippets:ro
|
|
||||||
- ./nginx/certs:/etc/nginx/certs:ro
|
|
||||||
- nginx_cache:/var/cache/nginx
|
|
||||||
restart: unless-stopped
|
|
||||||
|
|
||||||
volumes:
|
|
||||||
nginx_cache:
|
|
||||||
|
|||||||
+65
-169
@@ -1,33 +1,32 @@
|
|||||||
# Musify — base stack, shared by development and production.
|
# Musify: base stack, shared by development and production.
|
||||||
#
|
#
|
||||||
# This file publishes no host ports and hardcodes no domain: everything that
|
# Postgres, Zitadel, SeaweedFS, RabbitMQ and Jaeger are NOT part of this
|
||||||
# differs between environments comes from the .env file (see .env.example) or
|
# file. They're shared infrastructure that lives in ../../../Infrastructure
|
||||||
# from the overlay that is merged on top of it.
|
# and must already be running (see Infrastructure/README.md). This file only
|
||||||
|
# has what's specific to Musify: its own database, its own OIDC apps and
|
||||||
|
# storage bucket (created against the shared instances below), and its own
|
||||||
|
# apps.
|
||||||
#
|
#
|
||||||
# dev docker compose -f deploy/compose.yml -f deploy/compose.dev.yml up -d
|
# dev docker compose -f deploy/compose.yml -f deploy/compose.dev.yml up -d
|
||||||
# prod docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml up -d
|
# prod docker compose --env-file deploy/.env.prod -f deploy/compose.yml -f deploy/compose.prod.yml up -d
|
||||||
#
|
#
|
||||||
# The four application services sit behind the `apps` profile, so a plain `up`
|
# The four application services sit behind the `apps` profile, so a plain
|
||||||
# starts the infrastructure plus the one-shot bootstrap jobs (stream-ticket
|
# `up` runs only the one-shot bootstrap jobs (stream-ticket keys, database
|
||||||
# keys, database migrations, Zitadel project/apps) and stops there — the usual
|
# migrations, Zitadel project/apps, the storage bucket) and stops there. That's
|
||||||
# dev setup, with the apps running from the IDE. Add `--profile apps up -d
|
# the usual dev setup, with the apps running from the IDE. Add `--profile apps
|
||||||
# --build` as a SECOND command to also build and run them in Docker: by then
|
# up -d --build` as a SECOND command to also build and run them in Docker: by
|
||||||
# zitadel-init has already written the OIDC client ids into the env file, so
|
# then zitadel-init has already written the OIDC client ids into the env
|
||||||
# the apps start with real values. See deploy/README.md.
|
# file, so the apps start with real values. See deploy/README.md.
|
||||||
|
|
||||||
x-app-depends: &app-depends
|
|
||||||
postgres:
|
|
||||||
condition: service_healthy
|
|
||||||
rabbitmq:
|
|
||||||
condition: service_healthy
|
|
||||||
seaweedfs:
|
|
||||||
condition: service_healthy
|
|
||||||
|
|
||||||
# Backend settings that are identical for the API and the Worker. ASP.NET Core
|
# Backend settings that are identical for the API and the Worker. ASP.NET Core
|
||||||
# maps `__` to `:`, so these override the matching AppSettings.json sections.
|
# maps `__` to `:`, so these override the matching appsettings.json sections.
|
||||||
|
# POSTGRES_HOST / RABBITMQ_HOST / JAEGER_HOST differ between environments
|
||||||
|
# (host.docker.internal in dev, the container name over infra-net in prod,
|
||||||
|
# see .env.example / .env.prod.example), same as PUBLIC_AUTH_URL/S3_PUBLIC_URL
|
||||||
|
# already do.
|
||||||
x-backend-env: &backend-env
|
x-backend-env: &backend-env
|
||||||
Database__ConnectionString: "Host=postgres;Port=5432;Database=${POSTGRES_DB};Username=${POSTGRES_USER};Password=${POSTGRES_PASSWORD}"
|
Database__ConnectionString: "Host=${POSTGRES_HOST};Port=5432;Database=${POSTGRES_DB};Username=${POSTGRES_USER};Password=${POSTGRES_PASSWORD}"
|
||||||
MassTransit__Address: "rabbitmq://rabbitmq:5672"
|
MassTransit__Address: "rabbitmq://${RABBITMQ_HOST}:5672"
|
||||||
MassTransit__Username: ${RABBITMQ_DEFAULT_USER}
|
MassTransit__Username: ${RABBITMQ_DEFAULT_USER}
|
||||||
MassTransit__Password: ${RABBITMQ_DEFAULT_PASS}
|
MassTransit__Password: ${RABBITMQ_DEFAULT_PASS}
|
||||||
# The public S3 URL, not the internal one: the same address is used to sign
|
# The public S3 URL, not the internal one: the same address is used to sign
|
||||||
@@ -38,128 +37,35 @@ x-backend-env: &backend-env
|
|||||||
InfrastructureStorage__ForcePathStyle: "true"
|
InfrastructureStorage__ForcePathStyle: "true"
|
||||||
InfrastructureStorage__UseHttp: ${S3_USE_HTTP}
|
InfrastructureStorage__UseHttp: ${S3_USE_HTTP}
|
||||||
ApplicationStorage__Bucket: ${S3_BUCKET}
|
ApplicationStorage__Bucket: ${S3_BUCKET}
|
||||||
OpenTelemetry__OtlpEndpoint: "http://jaeger:4317"
|
OpenTelemetry__OtlpEndpoint: "http://${JAEGER_HOST}:4317"
|
||||||
|
|
||||||
services:
|
services:
|
||||||
# --- Infrastructure -------------------------------------------------------
|
# --- Shared-infrastructure bootstrap ---------------------------------------
|
||||||
|
|
||||||
postgres:
|
# Waits for Infrastructure's Postgres to accept connections before `migrate`
|
||||||
|
# runs. This replaces a `depends_on: postgres: condition: service_healthy`:
|
||||||
|
# that only works within the same compose file, and Postgres now lives in a
|
||||||
|
# separate one (../../../Infrastructure).
|
||||||
|
wait-for-postgres:
|
||||||
image: postgres:17-alpine
|
image: postgres:17-alpine
|
||||||
container_name: musify-postgres
|
container_name: musify-wait-for-postgres
|
||||||
environment:
|
|
||||||
POSTGRES_USER: ${POSTGRES_USER}
|
|
||||||
POSTGRES_PASSWORD: ${POSTGRES_PASSWORD}
|
|
||||||
POSTGRES_DB: ${POSTGRES_DB}
|
|
||||||
volumes:
|
|
||||||
- postgres_data:/var/lib/postgresql/data
|
|
||||||
healthcheck:
|
|
||||||
test: ["CMD-SHELL", "pg_isready -U ${POSTGRES_USER} -d ${POSTGRES_DB}"]
|
|
||||||
interval: 10s
|
|
||||||
timeout: 5s
|
|
||||||
retries: 10
|
|
||||||
restart: unless-stopped
|
|
||||||
|
|
||||||
# OIDC provider. `start-from-init` runs init + setup (both idempotent) and
|
|
||||||
# then starts, so the same command works on a cold and on a warm volume.
|
|
||||||
# It reuses the Postgres above: the admin user creates the `zitadel` database
|
|
||||||
# and role on first start.
|
|
||||||
zitadel:
|
|
||||||
image: ghcr.io/zitadel/zitadel:latest
|
|
||||||
container_name: musify-zitadel
|
|
||||||
command: start-from-init --masterkeyFromEnv --tlsMode disabled
|
|
||||||
environment:
|
|
||||||
ZITADEL_MASTERKEY: ${ZITADEL_MASTERKEY}
|
|
||||||
# The external domain/port forms the token issuer and must be identical
|
|
||||||
# for the browser, the apps and the containers (Zitadel routes by Host).
|
|
||||||
ZITADEL_EXTERNALSECURE: ${ZITADEL_EXTERNAL_SECURE}
|
|
||||||
ZITADEL_EXTERNALDOMAIN: ${ZITADEL_EXTERNAL_DOMAIN}
|
|
||||||
ZITADEL_EXTERNALPORT: ${ZITADEL_EXTERNAL_PORT}
|
|
||||||
ZITADEL_TLS_ENABLED: "false"
|
|
||||||
ZITADEL_DATABASE_POSTGRES_HOST: postgres
|
|
||||||
ZITADEL_DATABASE_POSTGRES_PORT: 5432
|
|
||||||
ZITADEL_DATABASE_POSTGRES_DATABASE: zitadel
|
|
||||||
ZITADEL_DATABASE_POSTGRES_USER_USERNAME: zitadel
|
|
||||||
ZITADEL_DATABASE_POSTGRES_USER_PASSWORD: ${ZITADEL_DATABASE_PASSWORD}
|
|
||||||
ZITADEL_DATABASE_POSTGRES_USER_SSL_MODE: disable
|
|
||||||
ZITADEL_DATABASE_POSTGRES_ADMIN_USERNAME: ${POSTGRES_USER}
|
|
||||||
ZITADEL_DATABASE_POSTGRES_ADMIN_PASSWORD: ${POSTGRES_PASSWORD}
|
|
||||||
ZITADEL_DATABASE_POSTGRES_ADMIN_SSL_MODE: disable
|
|
||||||
# Initial console admin. Login: <username>@zitadel.<external domain>
|
|
||||||
ZITADEL_FIRSTINSTANCE_ORG_HUMAN_USERNAME: ${ZITADEL_ADMIN_USERNAME}
|
|
||||||
ZITADEL_FIRSTINSTANCE_ORG_HUMAN_PASSWORD: ${ZITADEL_ADMIN_PASSWORD}
|
|
||||||
ZITADEL_FIRSTINSTANCE_ORG_HUMAN_PASSWORDCHANGEREQUIRED: "false"
|
|
||||||
# Service account + PAT written on init; the provisioning script uses it
|
|
||||||
# to create the Musify project and its OIDC apps.
|
|
||||||
ZITADEL_FIRSTINSTANCE_ORG_MACHINE_MACHINE_USERNAME: zitadel-admin-sa
|
|
||||||
ZITADEL_FIRSTINSTANCE_ORG_MACHINE_MACHINE_NAME: Admin Service Account
|
|
||||||
ZITADEL_FIRSTINSTANCE_ORG_MACHINE_PAT_EXPIRATIONDATE: "2099-01-01T00:00:00Z"
|
|
||||||
ZITADEL_FIRSTINSTANCE_PATPATH: /output/admin-sa.pat
|
|
||||||
depends_on:
|
|
||||||
postgres:
|
|
||||||
condition: service_healthy
|
|
||||||
volumes:
|
|
||||||
- ./zitadel/.output:/output
|
|
||||||
restart: unless-stopped
|
|
||||||
|
|
||||||
rabbitmq:
|
|
||||||
image: rabbitmq:4-management-alpine
|
|
||||||
container_name: musify-rabbitmq
|
|
||||||
environment:
|
|
||||||
RABBITMQ_DEFAULT_USER: ${RABBITMQ_DEFAULT_USER}
|
|
||||||
RABBITMQ_DEFAULT_PASS: ${RABBITMQ_DEFAULT_PASS}
|
|
||||||
volumes:
|
|
||||||
- rabbitmq_data:/var/lib/rabbitmq
|
|
||||||
healthcheck:
|
|
||||||
test: ["CMD", "rabbitmq-diagnostics", "-q", "ping"]
|
|
||||||
interval: 10s
|
|
||||||
timeout: 5s
|
|
||||||
retries: 10
|
|
||||||
restart: unless-stopped
|
|
||||||
|
|
||||||
# All-in-one object store: master + volume + filer + S3 gateway.
|
|
||||||
# The S3 identity file is rendered from the environment on start so the
|
|
||||||
# credentials can never drift from the ones handed to the apps.
|
|
||||||
seaweedfs:
|
|
||||||
image: chrislusf/seaweedfs:latest
|
|
||||||
container_name: musify-seaweedfs
|
|
||||||
entrypoint: ["/bin/sh", "-c"]
|
entrypoint: ["/bin/sh", "-c"]
|
||||||
command:
|
command:
|
||||||
- |
|
- |
|
||||||
cat > /tmp/s3.conf <<'JSON'
|
set -e
|
||||||
{
|
deadline=$$(( $$(date +%s) + 60 ))
|
||||||
"identities": [
|
until pg_isready -h ${POSTGRES_HOST} -p 5432 -U ${POSTGRES_USER} >/dev/null 2>&1; do
|
||||||
{
|
[ "$$(date +%s)" -lt "$$deadline" ] || { echo "postgres did not respond at ${POSTGRES_HOST}:5432. Is Infrastructure running?" >&2; exit 1; }
|
||||||
"name": "musify",
|
sleep 2
|
||||||
"credentials": [{ "accessKey": "${S3_ACCESS_KEY}", "secretKey": "${S3_SECRET_KEY}" }],
|
done
|
||||||
"actions": ["Admin", "Read", "Write"]
|
restart: "no"
|
||||||
}
|
|
||||||
]
|
|
||||||
}
|
|
||||||
JSON
|
|
||||||
exec weed server \
|
|
||||||
-ip.bind=0.0.0.0 \
|
|
||||||
-dir=/data \
|
|
||||||
-master.volumeSizeLimitMB=1024 \
|
|
||||||
-volume.max=0 \
|
|
||||||
-filer \
|
|
||||||
-s3 \
|
|
||||||
-s3.config=/tmp/s3.conf
|
|
||||||
volumes:
|
|
||||||
- seaweedfs_data:/data
|
|
||||||
healthcheck:
|
|
||||||
test: ["CMD", "wget", "-q", "-O", "-", "http://localhost:9333/cluster/healthz"]
|
|
||||||
interval: 10s
|
|
||||||
timeout: 5s
|
|
||||||
retries: 10
|
|
||||||
restart: unless-stopped
|
|
||||||
|
|
||||||
# One-shot: SeaweedFS does not auto-create buckets.
|
# One-shot: SeaweedFS does not auto-create buckets. `aws s3 ls` doubles as
|
||||||
|
# the readiness probe, since seaweedfs-init no longer has an in-file
|
||||||
|
# `depends_on: seaweedfs: condition: service_healthy` to wait for.
|
||||||
seaweedfs-init:
|
seaweedfs-init:
|
||||||
image: amazon/aws-cli:latest
|
image: amazon/aws-cli:latest
|
||||||
container_name: musify-seaweedfs-init
|
container_name: musify-seaweedfs-init
|
||||||
depends_on:
|
|
||||||
seaweedfs:
|
|
||||||
condition: service_healthy
|
|
||||||
environment:
|
environment:
|
||||||
AWS_ACCESS_KEY_ID: ${S3_ACCESS_KEY}
|
AWS_ACCESS_KEY_ID: ${S3_ACCESS_KEY}
|
||||||
AWS_SECRET_ACCESS_KEY: ${S3_SECRET_KEY}
|
AWS_SECRET_ACCESS_KEY: ${S3_SECRET_KEY}
|
||||||
@@ -167,13 +73,19 @@ services:
|
|||||||
entrypoint: ["/bin/sh", "-c"]
|
entrypoint: ["/bin/sh", "-c"]
|
||||||
command:
|
command:
|
||||||
- |
|
- |
|
||||||
aws --endpoint-url http://seaweedfs:8333 s3 mb s3://${S3_BUCKET} 2>/dev/null || true
|
set -e
|
||||||
aws --endpoint-url http://seaweedfs:8333 s3 ls
|
deadline=$$(( $$(date +%s) + 60 ))
|
||||||
|
until aws --endpoint-url ${S3_PUBLIC_URL} s3 ls >/dev/null 2>&1; do
|
||||||
|
[ "$$(date +%s)" -lt "$$deadline" ] || { echo "seaweedfs did not respond at ${S3_PUBLIC_URL}. Is Infrastructure running?" >&2; exit 1; }
|
||||||
|
sleep 2
|
||||||
|
done
|
||||||
|
aws --endpoint-url ${S3_PUBLIC_URL} s3 mb s3://${S3_BUCKET} 2>/dev/null || true
|
||||||
|
aws --endpoint-url ${S3_PUBLIC_URL} s3 ls
|
||||||
restart: "no"
|
restart: "no"
|
||||||
|
|
||||||
# One-shot: generates the RS256 key pair used to sign (API) and verify
|
# One-shot: generates the RS256 key pair used to sign (API) and verify
|
||||||
# (Gateway) stream tickets, if it is not already in ./keys. No host tools
|
# (Gateway) stream tickets, if it is not already in ./keys. No host tools
|
||||||
# required — openssl runs inside the container.
|
# required, since openssl runs inside the container.
|
||||||
keys-init:
|
keys-init:
|
||||||
image: alpine:3.20
|
image: alpine:3.20
|
||||||
container_name: musify-keys-init
|
container_name: musify-keys-init
|
||||||
@@ -203,30 +115,27 @@ services:
|
|||||||
dockerfile: Musify.Migrator.Dockerfile
|
dockerfile: Musify.Migrator.Dockerfile
|
||||||
container_name: musify-migrate
|
container_name: musify-migrate
|
||||||
depends_on:
|
depends_on:
|
||||||
postgres:
|
wait-for-postgres:
|
||||||
condition: service_healthy
|
condition: service_completed_successfully
|
||||||
environment:
|
environment:
|
||||||
Database__ConnectionString: "Host=postgres;Port=5432;Database=${POSTGRES_DB};Username=${POSTGRES_USER};Password=${POSTGRES_PASSWORD}"
|
Database__ConnectionString: "Host=${POSTGRES_HOST};Port=5432;Database=${POSTGRES_DB};Username=${POSTGRES_USER};Password=${POSTGRES_PASSWORD}"
|
||||||
restart: "no"
|
restart: "no"
|
||||||
|
|
||||||
# One-shot: creates the Musify project and its two OIDC apps in Zitadel (or
|
# One-shot: creates the Musify project and its two OIDC apps in the shared
|
||||||
# reuses them if they already exist), then writes AUTH_CLIENT_ID,
|
# Zitadel (or reuses them if they already exist), then writes
|
||||||
# WEB_CLIENT_ID and WEB_CLIENT_SECRET back into the env file mounted at
|
# AUTH_CLIENT_ID, WEB_CLIENT_ID and WEB_CLIENT_SECRET back into the env file
|
||||||
# /workspace/.env (see compose.dev.yml / compose.prod.yml) — so a later
|
# mounted at /workspace/.env (see compose.dev.yml / compose.prod.yml), so a
|
||||||
# `--profile apps up` picks up real values instead of empty ones.
|
# later `--profile apps up` picks up real values instead of empty ones.
|
||||||
zitadel-init:
|
zitadel-init:
|
||||||
image: alpine:3.20
|
image: alpine:3.20
|
||||||
container_name: musify-zitadel-init
|
container_name: musify-zitadel-init
|
||||||
depends_on:
|
|
||||||
zitadel:
|
|
||||||
condition: service_started
|
|
||||||
environment:
|
environment:
|
||||||
PUBLIC_AUTH_URL: ${PUBLIC_AUTH_URL}
|
PUBLIC_AUTH_URL: ${PUBLIC_AUTH_URL}
|
||||||
PUBLIC_API_URL: ${PUBLIC_API_URL}
|
PUBLIC_API_URL: ${PUBLIC_API_URL}
|
||||||
PUBLIC_WEB_URL: ${PUBLIC_WEB_URL}
|
PUBLIC_WEB_URL: ${PUBLIC_WEB_URL}
|
||||||
ZITADEL_DEV_MODE: ${ZITADEL_DEV_MODE:-false}
|
ZITADEL_DEV_MODE: ${ZITADEL_DEV_MODE:-false}
|
||||||
volumes:
|
volumes:
|
||||||
- ./zitadel/.output:/output:ro
|
- ${ZITADEL_ADMIN_PAT_DIR}:/output:ro
|
||||||
entrypoint: ["/bin/sh", "-c"]
|
entrypoint: ["/bin/sh", "-c"]
|
||||||
command:
|
command:
|
||||||
- |
|
- |
|
||||||
@@ -240,7 +149,7 @@ services:
|
|||||||
fi
|
fi
|
||||||
|
|
||||||
# The env file may have CRLF line endings (edited on Windows), so
|
# The env file may have CRLF line endings (edited on Windows), so
|
||||||
# strip \r on read and drop it on write — otherwise a "blank" value
|
# strip \r on read and drop it on write, otherwise a "blank" value
|
||||||
# is actually a lone \r, and `[ -z ... ]` lies.
|
# is actually a lone \r, and `[ -z ... ]` lies.
|
||||||
env_get() { grep -E "^$$2=" "$$1" 2>/dev/null | tail -n1 | cut -d= -f2- | tr -d '\r'; }
|
env_get() { grep -E "^$$2=" "$$1" 2>/dev/null | tail -n1 | cut -d= -f2- | tr -d '\r'; }
|
||||||
env_set() {
|
env_set() {
|
||||||
@@ -257,7 +166,7 @@ services:
|
|||||||
echo "==> waiting for the Zitadel service-account PAT"
|
echo "==> waiting for the Zitadel service-account PAT"
|
||||||
deadline=$$(( $$(date +%s) + 180 ))
|
deadline=$$(( $$(date +%s) + 180 ))
|
||||||
while [ ! -f /output/admin-sa.pat ]; do
|
while [ ! -f /output/admin-sa.pat ]; do
|
||||||
[ "$$(date +%s)" -lt "$$deadline" ] || { echo "PAT not found, check 'docker logs musify-zitadel'" >&2; exit 1; }
|
[ "$$(date +%s)" -lt "$$deadline" ] || { echo "PAT not found under /output. Is Infrastructure's Zitadel running? Check 'docker logs infra-zitadel'." >&2; exit 1; }
|
||||||
sleep 3
|
sleep 3
|
||||||
done
|
done
|
||||||
PAT=$$(tr -d '\r\n' </output/admin-sa.pat)
|
PAT=$$(tr -d '\r\n' </output/admin-sa.pat)
|
||||||
@@ -387,14 +296,6 @@ services:
|
|||||||
echo "==> wired into $$ENV_FILE"
|
echo "==> wired into $$ENV_FILE"
|
||||||
restart: "no"
|
restart: "no"
|
||||||
|
|
||||||
# OTLP collector + trace UI (MassTransit / EF Core / HTTP spans).
|
|
||||||
jaeger:
|
|
||||||
image: jaegertracing/all-in-one:latest
|
|
||||||
container_name: musify-jaeger
|
|
||||||
environment:
|
|
||||||
COLLECTOR_OTLP_ENABLED: "true"
|
|
||||||
restart: unless-stopped
|
|
||||||
|
|
||||||
# --- Applications (profile: apps) -----------------------------------------
|
# --- Applications (profile: apps) -----------------------------------------
|
||||||
|
|
||||||
api:
|
api:
|
||||||
@@ -404,7 +305,6 @@ services:
|
|||||||
dockerfile: Musify.Api.Dockerfile
|
dockerfile: Musify.Api.Dockerfile
|
||||||
container_name: musify-api
|
container_name: musify-api
|
||||||
depends_on:
|
depends_on:
|
||||||
<<: *app-depends
|
|
||||||
migrate:
|
migrate:
|
||||||
condition: service_completed_successfully
|
condition: service_completed_successfully
|
||||||
keys-init:
|
keys-init:
|
||||||
@@ -439,7 +339,6 @@ services:
|
|||||||
dockerfile: Musify.Worker.Dockerfile
|
dockerfile: Musify.Worker.Dockerfile
|
||||||
container_name: musify-worker
|
container_name: musify-worker
|
||||||
depends_on:
|
depends_on:
|
||||||
<<: *app-depends
|
|
||||||
migrate:
|
migrate:
|
||||||
condition: service_completed_successfully
|
condition: service_completed_successfully
|
||||||
environment:
|
environment:
|
||||||
@@ -458,15 +357,13 @@ services:
|
|||||||
dockerfile: Musify.StreamingGateway.Dockerfile
|
dockerfile: Musify.StreamingGateway.Dockerfile
|
||||||
container_name: musify-gateway
|
container_name: musify-gateway
|
||||||
depends_on:
|
depends_on:
|
||||||
seaweedfs:
|
|
||||||
condition: service_healthy
|
|
||||||
keys-init:
|
keys-init:
|
||||||
condition: service_completed_successfully
|
condition: service_completed_successfully
|
||||||
environment:
|
environment:
|
||||||
ASPNETCORE_ENVIRONMENT: ${DOTNET_ENVIRONMENT}
|
ASPNETCORE_ENVIRONMENT: ${DOTNET_ENVIRONMENT}
|
||||||
ASPNETCORE_URLS: http://+:8081
|
ASPNETCORE_URLS: http://+:8081
|
||||||
StreamTicket__PublicKeyPath: /keys/stream_public.pem
|
StreamTicket__PublicKeyPath: /keys/stream_public.pem
|
||||||
ReverseProxy__Clusters__seaweed-filer__Destinations__filer__Address: "http://seaweedfs:8888/"
|
ReverseProxy__Clusters__seaweed-filer__Destinations__filer__Address: "http://${SEAWEEDFS_HOST}:8888/"
|
||||||
ReverseProxy__Routes__media__Transforms__0__PathPattern: "/buckets/${S3_BUCKET}/{**rest}"
|
ReverseProxy__Routes__media__Transforms__0__PathPattern: "/buckets/${S3_BUCKET}/{**rest}"
|
||||||
Cors__AllowedOrigins__0: ${PUBLIC_WEB_URL}
|
Cors__AllowedOrigins__0: ${PUBLIC_WEB_URL}
|
||||||
volumes:
|
volumes:
|
||||||
@@ -484,6 +381,10 @@ services:
|
|||||||
environment:
|
environment:
|
||||||
NODE_ENV: production
|
NODE_ENV: production
|
||||||
PORT: 3000
|
PORT: 3000
|
||||||
|
# adapter-node rejects request bodies over 512K by default; audio
|
||||||
|
# uploads are posted to /upload. Keep in line with the edge's
|
||||||
|
# client_max_body_size.
|
||||||
|
BODY_SIZE_LIMIT: 1G
|
||||||
ORIGIN: ${PUBLIC_WEB_URL}
|
ORIGIN: ${PUBLIC_WEB_URL}
|
||||||
ZITADEL_ISSUER: ${PUBLIC_AUTH_URL}
|
ZITADEL_ISSUER: ${PUBLIC_AUTH_URL}
|
||||||
ZITADEL_CLIENT_ID: ${WEB_CLIENT_ID}
|
ZITADEL_CLIENT_ID: ${WEB_CLIENT_ID}
|
||||||
@@ -495,10 +396,5 @@ services:
|
|||||||
API_BASE_URL: http://api:5111
|
API_BASE_URL: http://api:5111
|
||||||
# Presigned PUTs are signed for the public S3 host; the container sends
|
# Presigned PUTs are signed for the public S3 host; the container sends
|
||||||
# them straight to SeaweedFS instead of going back out through the edge.
|
# them straight to SeaweedFS instead of going back out through the edge.
|
||||||
S3_UPLOAD_ENDPOINT: http://seaweedfs:8333
|
S3_UPLOAD_ENDPOINT: ${S3_INTERNAL_URL}
|
||||||
restart: unless-stopped
|
restart: unless-stopped
|
||||||
|
|
||||||
volumes:
|
|
||||||
postgres_data:
|
|
||||||
rabbitmq_data:
|
|
||||||
seaweedfs_data:
|
|
||||||
|
|||||||
@@ -1,9 +0,0 @@
|
|||||||
# Shared proxy settings for every Musify vhost.
|
|
||||||
proxy_http_version 1.1;
|
|
||||||
proxy_set_header Host $host;
|
|
||||||
proxy_set_header X-Real-IP $remote_addr;
|
|
||||||
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
|
|
||||||
proxy_set_header X-Forwarded-Proto https;
|
|
||||||
proxy_set_header X-Forwarded-Host $host;
|
|
||||||
proxy_set_header Upgrade $http_upgrade;
|
|
||||||
proxy_set_header Connection $connection_upgrade;
|
|
||||||
@@ -1,119 +0,0 @@
|
|||||||
# Musify edge. The nginx entrypoint renders this with envsubst into
|
|
||||||
# /etc/nginx/conf.d/default.conf (replacing the image's stock file) using the
|
|
||||||
# MUSIFY_* variables set in compose.prod.yml.
|
|
||||||
#
|
|
||||||
# Cloudflare (Full strict) terminates the public edge; this box only needs to
|
|
||||||
# serve the Cloudflare Origin Certificate in nginx/certs/. Each vhost proxies to
|
|
||||||
# a compose service by name through the embedded Docker DNS resolver, so nginx
|
|
||||||
# re-resolves it instead of caching the IP of a container that may be recreated.
|
|
||||||
|
|
||||||
resolver 127.0.0.11 valid=10s;
|
|
||||||
|
|
||||||
map $http_upgrade $connection_upgrade {
|
|
||||||
default upgrade;
|
|
||||||
'' close;
|
|
||||||
}
|
|
||||||
|
|
||||||
server {
|
|
||||||
listen 80;
|
|
||||||
listen [::]:80;
|
|
||||||
server_name ${MUSIFY_ROOT_DOMAIN} www.${MUSIFY_ROOT_DOMAIN}
|
|
||||||
api.${MUSIFY_ROOT_DOMAIN} auth.${MUSIFY_ROOT_DOMAIN}
|
|
||||||
stream.${MUSIFY_ROOT_DOMAIN} s3.${MUSIFY_ROOT_DOMAIN};
|
|
||||||
return 301 https://$host$request_uri;
|
|
||||||
}
|
|
||||||
|
|
||||||
# Web player (SvelteKit, adapter-node). Uploads are posted to its own form
|
|
||||||
# actions, so it needs the large body size too.
|
|
||||||
server {
|
|
||||||
listen 443 ssl;
|
|
||||||
http2 on;
|
|
||||||
server_name ${MUSIFY_ROOT_DOMAIN};
|
|
||||||
client_max_body_size ${MUSIFY_MAX_UPLOAD_SIZE};
|
|
||||||
|
|
||||||
ssl_certificate /etc/nginx/certs/origin.pem;
|
|
||||||
ssl_certificate_key /etc/nginx/certs/origin.key;
|
|
||||||
|
|
||||||
location / {
|
|
||||||
set $upstream http://web-player:3000;
|
|
||||||
proxy_pass $upstream;
|
|
||||||
include /etc/nginx/snippets/proxy.conf;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
server {
|
|
||||||
listen 443 ssl;
|
|
||||||
http2 on;
|
|
||||||
server_name www.${MUSIFY_ROOT_DOMAIN};
|
|
||||||
|
|
||||||
ssl_certificate /etc/nginx/certs/origin.pem;
|
|
||||||
ssl_certificate_key /etc/nginx/certs/origin.key;
|
|
||||||
|
|
||||||
return 301 https://${MUSIFY_ROOT_DOMAIN}$request_uri;
|
|
||||||
}
|
|
||||||
|
|
||||||
server {
|
|
||||||
listen 443 ssl;
|
|
||||||
http2 on;
|
|
||||||
server_name api.${MUSIFY_ROOT_DOMAIN};
|
|
||||||
|
|
||||||
ssl_certificate /etc/nginx/certs/origin.pem;
|
|
||||||
ssl_certificate_key /etc/nginx/certs/origin.key;
|
|
||||||
|
|
||||||
location / {
|
|
||||||
set $upstream http://api:5111;
|
|
||||||
proxy_pass $upstream;
|
|
||||||
include /etc/nginx/snippets/proxy.conf;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
# Audio bytes. Buffering off keeps range requests (seeking) responsive.
|
|
||||||
server {
|
|
||||||
listen 443 ssl;
|
|
||||||
http2 on;
|
|
||||||
server_name stream.${MUSIFY_ROOT_DOMAIN};
|
|
||||||
|
|
||||||
ssl_certificate /etc/nginx/certs/origin.pem;
|
|
||||||
ssl_certificate_key /etc/nginx/certs/origin.key;
|
|
||||||
|
|
||||||
location / {
|
|
||||||
set $upstream http://gateway:8081;
|
|
||||||
proxy_pass $upstream;
|
|
||||||
include /etc/nginx/snippets/proxy.conf;
|
|
||||||
proxy_buffering off;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
server {
|
|
||||||
listen 443 ssl;
|
|
||||||
http2 on;
|
|
||||||
server_name auth.${MUSIFY_ROOT_DOMAIN};
|
|
||||||
|
|
||||||
ssl_certificate /etc/nginx/certs/origin.pem;
|
|
||||||
ssl_certificate_key /etc/nginx/certs/origin.key;
|
|
||||||
|
|
||||||
location / {
|
|
||||||
set $upstream http://zitadel:8080;
|
|
||||||
proxy_pass $upstream;
|
|
||||||
include /etc/nginx/snippets/proxy.conf;
|
|
||||||
grpc_set_header Host $host;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
# Presigned S3 PUT/GET straight from the browser.
|
|
||||||
server {
|
|
||||||
listen 443 ssl;
|
|
||||||
http2 on;
|
|
||||||
server_name s3.${MUSIFY_ROOT_DOMAIN};
|
|
||||||
client_max_body_size ${MUSIFY_MAX_UPLOAD_SIZE};
|
|
||||||
|
|
||||||
ssl_certificate /etc/nginx/certs/origin.pem;
|
|
||||||
ssl_certificate_key /etc/nginx/certs/origin.key;
|
|
||||||
|
|
||||||
location / {
|
|
||||||
set $upstream http://seaweedfs:8333;
|
|
||||||
proxy_pass $upstream;
|
|
||||||
include /etc/nginx/snippets/proxy.conf;
|
|
||||||
proxy_request_buffering off;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
+39
-28
@@ -1,47 +1,58 @@
|
|||||||
# Musify — Documentación
|
# Musify documentation
|
||||||
|
|
||||||
Musify es un backend de streaming de música: subir pistas, procesarlas (transcodificar el audio a `.m4a` y generar miniaturas), organizarlas en playlists y reproducirlas en streaming sin que los bytes pasen por la API.
|
Musify is a music streaming backend: upload tracks, process them (transcode
|
||||||
|
the audio to `.m4a`, generate thumbnails), organize them into playlists, and
|
||||||
|
stream them without the bytes ever passing through the API.
|
||||||
|
|
||||||
Esta carpeta explica **qué hace cada pieza, su propósito y por qué está diseñada así**.
|
This folder explains **what each piece does, its purpose, and why it's built
|
||||||
|
this way**.
|
||||||
|
|
||||||
## Índice
|
## Index
|
||||||
|
|
||||||
- [architecture.md](architecture.md) — Visión global: componentes, cómo encajan y los flujos principales.
|
- [architecture.md](architecture.md): overview, components, how they fit together, and the main flows.
|
||||||
- [projects.md](projects.md) — Qué es y para qué sirve cada proyecto/solución.
|
- [projects.md](projects.md): what each project/solution is and what it's for.
|
||||||
- [authentication.md](authentication.md) — Zitadel (OIDC), validación de JWT, provisioning/sync de usuarios.
|
- [authentication.md](authentication.md): Zitadel (OIDC), JWT validation, user provisioning/sync.
|
||||||
- [storage.md](storage.md) — SeaweedFS (S3), estructura de claves, URLs prefirmadas, qué es público y qué privado.
|
- [storage.md](storage.md): SeaweedFS (S3), key layout, presigned URLs, what's public vs. private.
|
||||||
- [streaming.md](streaming.md) — Streaming del audio (`.m4a` por HTTP range) con stream-ticket (RS256) y el reverse proxy (StreamingGateway).
|
- [streaming.md](streaming.md): audio streaming (`.m4a` over HTTP range) with stream tickets (RS256) and the reverse proxy (StreamingGateway).
|
||||||
- [media-processing.md](media-processing.md) — El worker y los workflows de MassTransit (transcode de audio, miniaturas).
|
- [media-processing.md](media-processing.md): the worker and its MassTransit workflows (audio transcoding, thumbnails).
|
||||||
- [development.md](development.md) — Cómo levantar todo en local: contenedores, puertos y comandos.
|
- [development.md](development.md): running everything locally, containers, ports, commands.
|
||||||
- [../deploy/README.md](../deploy/README.md) — El deploy en detalle: compose, scripts y producción.
|
- [../deploy/README.md](../deploy/README.md): deployment in detail, compose, scripts, and production.
|
||||||
|
|
||||||
## Mapa de componentes (alto nivel)
|
## Component map (high level)
|
||||||
|
|
||||||
```
|
```
|
||||||
┌─────────────────────────────────────────────┐
|
┌─────────────────────────────────────────────┐
|
||||||
Cliente ───► │ Musify.Api (:5111) API REST + emite tickets │
|
Client ───► │ Musify.Api (:5111) REST API + issues tickets│
|
||||||
└─────────────────────────────────────────────┘
|
└─────────────────────────────────────────────┘
|
||||||
│ EF Core │ presigned (subidas) │ MassTransit (eventos)
|
│ EF Core │ presigned (uploads) │ MassTransit (events)
|
||||||
▼ ▼ ▼
|
▼ ▼ ▼
|
||||||
PostgreSQL SeaweedFS S3 (:8333) RabbitMQ ──► Worker (procesa)
|
PostgreSQL SeaweedFS S3 (:8333) RabbitMQ ──► Worker (processes)
|
||||||
(metadatos) (objetos) audio→.m4a, imágenes
|
(metadata) (objects) audio→.m4a, images
|
||||||
▲
|
▲
|
||||||
Cliente ───► StreamingGateway (:8081) ┘ sirve audio .m4a (valida ticket, proxy al filer)
|
Client ───► StreamingGateway (:8081) ┘ serves .m4a audio (validates ticket, proxies to the filer)
|
||||||
|
|
||||||
Login: Cliente ◄──► Zitadel (:8080, OIDC) → Musify.Api valida el JWT
|
Login: Client ◄──► Zitadel (:8080, OIDC) → Musify.Api validates the JWT
|
||||||
```
|
```
|
||||||
|
|
||||||
|
PostgreSQL, Zitadel, SeaweedFS and RabbitMQ are not run by this repository.
|
||||||
|
They're shared infrastructure, provided by the separate `Infrastructure`
|
||||||
|
repository, that this stack connects to the same way any other project
|
||||||
|
does.
|
||||||
|
|
||||||
## Stack
|
## Stack
|
||||||
|
|
||||||
- **.NET 10**, C#. Minimal APIs (`Musify.Api`), Worker Service (`Musify.Worker`), ASP.NET + YARP (`Musify.StreamingGateway`).
|
- **.NET 10**, C#. Minimal APIs (`Musify.Api`), Worker Service (`Musify.Worker`), ASP.NET + YARP (`Musify.StreamingGateway`).
|
||||||
- **CQRS** con [Mediator] (source-generator) y resultados con **ErrorOr**.
|
- **CQRS** with [Mediator] (source generator) and results via **ErrorOr**.
|
||||||
- **EF Core 10** + **PostgreSQL** (metadatos).
|
- **EF Core 10** + **PostgreSQL** (metadata).
|
||||||
- **MassTransit 8** + **RabbitMQ** (procesado asíncrono con routing slips/sagas).
|
- **MassTransit 8** + **RabbitMQ** (async processing with routing slips/sagas).
|
||||||
- **SeaweedFS** (almacenamiento de objetos compatible con S3, vía AWSSDK.S3).
|
- **SeaweedFS** (S3-compatible object storage, via AWSSDK.S3).
|
||||||
- **ffmpeg** (transcode de audio a AAC/`.m4a`), **SixLabors.ImageSharp** (miniaturas).
|
- **ffmpeg** (audio transcoding to AAC/`.m4a`), **SixLabors.ImageSharp** (thumbnails).
|
||||||
- **Zitadel** (OIDC/OAuth2) para identidad.
|
- **Zitadel** (OIDC/OAuth2) for identity.
|
||||||
|
|
||||||
## Estructura del repositorio
|
## Repository layout
|
||||||
|
|
||||||
Un solo repo: `backend/` (los seis proyectos .NET), `web-player/` (cliente web
|
A single repo: `backend/` (the six .NET projects), `web-player/` (the
|
||||||
SvelteKit), `deploy/` (stacks de Docker Compose y scripts) y `docs/`.
|
SvelteKit web client), `deploy/` (Docker Compose stacks and scripts), and
|
||||||
|
`docs/`. The shared infrastructure (Postgres, Zitadel, SeaweedFS, RabbitMQ,
|
||||||
|
Jaeger) lives in the separate `Infrastructure` repository, a sibling of this
|
||||||
|
one, and is not part of this layout.
|
||||||
|
|||||||
+41
-41
@@ -1,55 +1,55 @@
|
|||||||
# Arquitectura
|
# Architecture
|
||||||
|
|
||||||
## Principios
|
## Principles
|
||||||
|
|
||||||
- **Clean architecture / capas** en `backend/`: el dominio y la aplicación no dependen de infraestructura; la infraestructura implementa las interfaces (`Application/Contracts`).
|
- **Clean architecture / layers** in `backend/`: the domain and application layers don't depend on infrastructure; infrastructure implements their interfaces (`Application/Contracts`).
|
||||||
- **CQRS**: cada operación es un `Command` o `Query` con su handler (Mediator). Los handlers devuelven `Ardalis.Result` (Ok/NotFound/Unauthorized/Conflict/Invalid…), que la API traduce a códigos HTTP.
|
- **CQRS**: every operation is a `Command` or `Query` with its own handler (Mediator). Handlers return `Ardalis.Result` (Ok/NotFound/Unauthorized/Conflict/Invalid…), which the API translates into HTTP status codes.
|
||||||
- **Los bytes pesados no pasan por la API**: subidas con URLs prefirmadas (cliente → S3) y reproducción vía un reverse proxy dedicado (gateway → S3). La API solo mueve metadatos y emite permisos.
|
- **Heavy bytes never touch the API**: uploads use presigned URLs (client → S3) and playback goes through a dedicated reverse proxy (gateway → S3). The API only moves metadata and issues permissions.
|
||||||
- **Procesado asíncrono**: crear una pista no transcodifica en caliente; publica un evento y un worker hace el trabajo pesado con compensación (sagas/routing slips de MassTransit).
|
- **Async processing**: creating a track doesn't transcode it on the spot. It publishes an event, and a worker does the heavy lifting, with compensation (MassTransit sagas/routing slips).
|
||||||
|
|
||||||
## Componentes
|
## Components
|
||||||
|
|
||||||
| Componente | Responsabilidad | Por qué separado |
|
| Component | Responsibility | Why it's separate |
|
||||||
|---|---|---|
|
|---|---|---|
|
||||||
| **Musify.Api** | API REST (usuarios, pistas, playlists), autoriza, emite presigned URLs y stream-tickets | Punto de entrada fino; no procesa medios ni sirve bytes |
|
| **Musify.Api** | REST API (users, tracks, playlists), authorizes requests, issues presigned URLs and stream tickets | A thin entry point, it doesn't process media or serve bytes |
|
||||||
| **Domain/Application/Infrastructure** | Dominio + casos de uso + infraestructura compartida (EF, S3, MassTransit, ffmpeg) | Lógica reutilizada por la API y el Worker |
|
| **Domain/Application/Infrastructure** | Domain + use cases + shared infrastructure (EF, S3, MassTransit, ffmpeg) | Logic shared between the API and the Worker |
|
||||||
| **StreamingGateway** | Reverse proxy (YARP) que valida el ticket y sirve el audio (`.m4a`) desde SeaweedFS | Escala con el tráfico de bytes, independiente de la API; SeaweedFS queda privado detrás |
|
| **StreamingGateway** | Reverse proxy (YARP) that validates the ticket and serves the audio (`.m4a`) from SeaweedFS | Scales with byte traffic independently of the API, and keeps SeaweedFS private behind it |
|
||||||
| **Worker** | Consume eventos de MassTransit: transcodifica audio a `.m4a` y genera miniaturas | Trabajo CPU/IO intensivo fuera del request HTTP |
|
| **Worker** | Consumes MassTransit events: transcodes audio to `.m4a` and generates thumbnails | Keeps CPU/IO-heavy work off the HTTP request path |
|
||||||
| **PostgreSQL** | Metadatos (usuarios, pistas, playlists, upload intents) | — |
|
| **PostgreSQL** *(shared, runs in the separate `Infrastructure` repo)* | Metadata (users, tracks, playlists, upload intents) | n/a |
|
||||||
| **RabbitMQ** | Cola de eventos + outbox transaccional | Desacopla creación de procesado |
|
| **RabbitMQ** *(shared, runs in Infrastructure)* | Event queue + transactional outbox | Decouples creation from processing |
|
||||||
| **SeaweedFS** | Almacenamiento de objetos (S3 + filer) | Guarda originales y derivados (audio `.m4a`, miniaturas) |
|
| **SeaweedFS** *(shared, runs in Infrastructure)* | Object storage (S3 + filer) | Stores originals and derivatives (`.m4a` audio, thumbnails) |
|
||||||
| **Zitadel** | Identidad (OIDC/OAuth2) | Login y emisión de JWT |
|
| **Zitadel** *(shared, runs in Infrastructure)* | Identity (OIDC/OAuth2) | Login and JWT issuance |
|
||||||
|
|
||||||
## Flujos principales
|
## Main flows
|
||||||
|
|
||||||
### 1. Subida de una pista
|
### 1. Uploading a track
|
||||||
1. Cliente pide `POST /tracks/upload-urls` → la API crea **UploadIntents** (reservas) y devuelve **URLs prefirmadas PUT** (imagen + audio).
|
1. The client requests `POST /tracks/upload-urls`. The API creates **UploadIntents** (reservations) and returns **presigned PUT URLs** (image + audio).
|
||||||
2. Cliente sube los ficheros **directo a SeaweedFS** con esas URLs (no pasan por la API).
|
2. The client uploads the files **directly to SeaweedFS** using those URLs (never through the API).
|
||||||
3. Cliente llama `POST /tracks` con los `intentId`s → la API valida los intents, crea el `Track` (+ `UserHasTrack`) y publica `CreateTrackResourcesEvent`.
|
3. The client calls `POST /tracks` with the `intentId`s. The API validates the intents, creates the `Track` (+ `UserHasTrack`), and publishes `CreateTrackResourcesEvent`.
|
||||||
4. El **Worker** consume el evento: mueve los originales a su sitio, genera miniaturas y transcodifica el audio a `.m4a`, y actualiza el estado del track.
|
4. The **Worker** consumes the event: moves the originals into place, generates thumbnails, transcodes the audio to `.m4a`, and updates the track's status.
|
||||||
|
|
||||||
Ver detalle en [media-processing.md](media-processing.md) y [storage.md](storage.md).
|
See [media-processing.md](media-processing.md) and [storage.md](storage.md) for details.
|
||||||
|
|
||||||
### 2. Reproducción (streaming)
|
### 2. Playback (streaming)
|
||||||
1. Cliente pide `GET /tracks/{id}/stream` (autenticado) → la API autoriza y devuelve `{ manifestUrl, ticket }` (URL al `.m4a` + ticket RS256 acotado a la carpeta de esa pista).
|
1. The client requests `GET /tracks/{id}/stream` (authenticated). The API authorizes it and returns `{ manifestUrl, ticket }` (a URL to the `.m4a` plus an RS256 ticket scoped to that track's folder).
|
||||||
2. El reproductor abre la URL del audio añadiendo `?t=<ticket>` (o cabecera `X-Stream-Ticket`) y descarga por HTTP range.
|
2. The player opens the audio URL with `?t=<ticket>` appended (or an `X-Stream-Ticket` header) and downloads it via HTTP range requests.
|
||||||
3. El **StreamingGateway** valida el ticket y reenvía al filer de SeaweedFS; los bytes van storage → cliente.
|
3. The **StreamingGateway** validates the ticket and forwards the request to the SeaweedFS filer. The bytes flow storage → client.
|
||||||
|
|
||||||
Ver [streaming.md](streaming.md).
|
See [streaming.md](streaming.md).
|
||||||
|
|
||||||
### 3. Autenticación
|
### 3. Authentication
|
||||||
1. Cliente hace login OIDC contra **Zitadel** y obtiene un access token (JWT).
|
1. The client logs in via OIDC against **Zitadel** and gets an access token (JWT).
|
||||||
2. La Musify.Api valida la firma/iss/aud del JWT.
|
2. Musify.Api validates the JWT's signature, issuer and audience.
|
||||||
3. En cada token validado, se **sincroniza** el usuario local (crea/actualiza nombre y foto) — provisioning just-in-time.
|
3. On every validated token, the local user is **synced** (name and picture created or updated). This is just-in-time provisioning.
|
||||||
|
|
||||||
Ver [authentication.md](authentication.md).
|
See [authentication.md](authentication.md).
|
||||||
|
|
||||||
## Modelo de datos (entidades)
|
## Data model (entities)
|
||||||
|
|
||||||
- **User** — `Id` es el `sub` numérico de Zitadel (`long`), más nombre y `ProfilePictureUrl`.
|
- **User**: `Id` is Zitadel's numeric `sub` (`long`), plus a name and `ProfilePictureUrl`.
|
||||||
- **Track** — pista: nombres de originales/derivados, `AudioFolderName` (carpeta del `.m4a` transcodificado), estados de procesado (`ProcessingStatus`), `LifeCycleStatus`.
|
- **Track**: original/derivative file names, `AudioFolderName` (the transcoded `.m4a`'s folder), processing state (`ProcessingStatus`), `LifeCycleStatus`.
|
||||||
- **PlayList** — lista del usuario (`UserId`).
|
- **PlayList**: a user's playlist (`UserId`).
|
||||||
- **PlayListHasTrack** — pistas dentro de una playlist (con `Position`).
|
- **PlayListHasTrack**: the tracks inside a playlist (with `Position`).
|
||||||
- **UserHasTrack** — propiedad: qué usuario posee qué pista.
|
- **UserHasTrack**: ownership, which user owns which track.
|
||||||
- **UploadIntent** — reserva de subida (clave, bucket, tamaño esperado, estado, expiración) para validar y aplicar cuota.
|
- **UploadIntent**: an upload reservation (key, bucket, expected size, status, expiration) used for validation and quota.
|
||||||
- **Upload** — registro de objetos subidos/transferidos al storage.
|
- **Upload**: a record of objects uploaded or transferred to storage.
|
||||||
|
|||||||
+26
-26
@@ -1,40 +1,40 @@
|
|||||||
# Autenticación e identidad
|
# Authentication and identity
|
||||||
|
|
||||||
## Proveedor: Zitadel (OIDC/OAuth2)
|
## Provider: Zitadel (OIDC/OAuth2)
|
||||||
|
|
||||||
El login y la emisión de tokens los hace **Zitadel** (no la API). La API es un **resource server**: solo **valida** el access token (JWT) que le llega en `Authorization: Bearer`.
|
Login and token issuance are handled by **Zitadel**, not the API. The API is a **resource server**: it only **validates** the access token (JWT) it receives in `Authorization: Bearer`.
|
||||||
|
|
||||||
- Config en la sección `Authentication` (`AuthenticationConfiguration`): metadata address, issuer, audience, client id, endpoints de authorize/token, scopes.
|
- Configured under the `Authentication` section (`AuthenticationConfiguration`): metadata address, issuer, audience, client id, authorize/token endpoints, scopes.
|
||||||
- `JwtBearerOptionsSetup` configura la validación (issuer, audience, claves vía metadata OIDC).
|
- `JwtBearerOptionsSetup` sets up validation (issuer, audience, keys via OIDC metadata).
|
||||||
- En dev, `RequireHttpsMetadata = false`.
|
- In dev, `RequireHttpsMetadata = false`.
|
||||||
|
|
||||||
### Requisito clave en la app de Zitadel
|
### Key requirement in the Zitadel app
|
||||||
- **Auth Token Type = JWT** (si no, Zitadel emite un token **opaco** sin puntos y `JwtBearer` no lo puede validar → 401 en todo lo protegido).
|
- **Auth Token Type = JWT** (otherwise Zitadel issues an **opaque** token with no dots, which `JwtBearer` can't validate → 401 on everything protected).
|
||||||
- **User Info inside Token** activado si quieres que el access token traiga claims de perfil (`name`, `email`, `picture`). Por defecto el access token es mínimo (`sub`, `aud`, `iss`, `exp`…), sin perfil.
|
- **User Info inside Token** enabled if you want the access token to carry profile claims (`name`, `email`, `picture`). By default the access token is minimal (`sub`, `aud`, `iss`, `exp`…), with no profile info.
|
||||||
|
|
||||||
## El `sub` es numérico → `User.Id` es `long`
|
## `sub` is numeric → `User.Id` is `long`
|
||||||
|
|
||||||
El `sub` de Zitadel es un entero de 64 bits en string (p. ej. `371953080444977155`), **no un GUID**. Por eso:
|
Zitadel's `sub` is a 64-bit integer encoded as a string (e.g. `371953080444977155`), **not a GUID**. Because of that:
|
||||||
- `User.Id` y todas las FKs de usuario (`PlayList.UserId`, `UserHasTrack.UserId`, `UploadIntent.UserId`) son **`long`**.
|
- `User.Id` and every user-related FK (`PlayList.UserId`, `UserHasTrack.UserId`, `UploadIntent.UserId`) are **`long`**.
|
||||||
- `User.Id` lleva `[DatabaseGenerated(None)]` para que EF **no** lo autogenere: se asigna el `sub`.
|
- `User.Id` has `[DatabaseGenerated(None)]` so EF doesn't autogenerate it: it's assigned from the `sub`.
|
||||||
- `CurrentUser.Id` es `long?` (parsea con `long.TryParse`); `RequiredId` lanza si falta.
|
- `CurrentUser.Id` is `long?` (parsed with `long.TryParse`); `RequiredId` throws if it's missing.
|
||||||
|
|
||||||
**Por qué**: el código antiguo hacía `Guid.Parse(sub)` y reventaba (FormatException) → 401 en cada endpoint protegido. Se decidió guardar el `sub` tal cual como número.
|
**Why**: the original code did `Guid.Parse(sub)`, which threw a `FormatException` → 401 on every protected endpoint. The fix was to store the `sub` as-is, as a number.
|
||||||
|
|
||||||
## Provisioning / sincronización de usuarios
|
## User provisioning / sync
|
||||||
|
|
||||||
En cada token validado, `JwtBearerEventsHandler.TokenValidated`:
|
On every validated token, `JwtBearerEventsHandler.TokenValidated`:
|
||||||
1. Lee `sub` (id), `name`/`preferred_username`/`email` (con fallback) y el claim OIDC `picture`.
|
1. Reads `sub` (id), `name`/`preferred_username`/`email` (with fallback), and the OIDC `picture` claim.
|
||||||
2. Envía un `SyncUserCommand`.
|
2. Sends a `SyncUserCommand`.
|
||||||
|
|
||||||
`SyncUserCommandHandler` hace un **upsert idempotente**:
|
`SyncUserCommandHandler` does an **idempotent upsert**:
|
||||||
- Si el usuario no existe → lo crea.
|
- If the user doesn't exist yet → creates it.
|
||||||
- Si existe → actualiza nombre/apellidos/`ProfilePictureUrl` **solo si cambiaron** (escribe en DB únicamente cuando hay cambios).
|
- If it exists → updates the name and `ProfilePictureUrl` **only if they changed** (a DB write only happens when something is different).
|
||||||
|
|
||||||
**Por qué upsert con "escribe solo si cambió"**: `TokenValidated` se ejecuta en **cada request**; sin ese guardado condicional estaríamos escribiendo en cada petición. Y debe ser resiliente: si el provisioning falla, se loguea pero **no** tumba la autenticación.
|
**Why "write only if changed"**: `TokenValidated` runs on **every request**; without that guard we'd be writing to the DB on every single call. It also needs to be resilient: if provisioning fails, it's logged but authentication **doesn't** fail because of it.
|
||||||
|
|
||||||
> Nota: `CreateUserCommand` (endpoint manual `POST /users`) sigue siendo create-only; el sync del login usa `SyncUserCommand`.
|
> Note: `CreateUserCommand` (the manual `POST /users` endpoint) is still create-only; login sync uses `SyncUserCommand`.
|
||||||
|
|
||||||
## Foto de perfil
|
## Profile picture
|
||||||
|
|
||||||
`User.ProfilePictureUrl` se rellena desde el claim `picture` del token. Si Zitadel no mete la info de perfil en el access token (ver arriba), llega `null` — el código es correcto, pero no hay nada que sincronizar hasta activar esa opción.
|
`User.ProfilePictureUrl` is filled from the token's `picture` claim. If Zitadel doesn't include profile info in the access token (see above), it arrives as `null`. The code is correct, there's just nothing to sync until that option is turned on.
|
||||||
|
|||||||
+92
-88
@@ -1,21 +1,25 @@
|
|||||||
# Entorno de desarrollo
|
# Development environment
|
||||||
|
|
||||||
Cómo levantar todo en local y probarlo. El detalle completo del deploy (y el de
|
How to run everything locally and test it. The full deployment details (dev
|
||||||
producción) está en [deploy/README.md](../deploy/README.md).
|
and prod) live in [deploy/README.md](../deploy/README.md).
|
||||||
|
|
||||||
## Arranque
|
## Startup
|
||||||
|
|
||||||
|
Bring up the separate `Infrastructure` repository first (Postgres, Zitadel,
|
||||||
|
SeaweedFS, RabbitMQ, Jaeger, see its own README for the exact command), then:
|
||||||
|
|
||||||
```sh
|
```sh
|
||||||
docker compose -f deploy/compose.yml -f deploy/compose.dev.yml up -d
|
docker compose -f deploy/compose.yml -f deploy/compose.dev.yml up -d
|
||||||
```
|
```
|
||||||
|
|
||||||
Ese único comando levanta la infraestructura y corre los jobs *one-shot* de
|
That runs Musify's own one-shot bootstrap jobs: it generates the RS256
|
||||||
bootstrap: genera las claves RS256 de stream-ticket en `deploy/keys/`, aplica
|
stream-ticket keys in `deploy/keys/`, applies the EF Core migrations, creates
|
||||||
las migraciones EF Core y provisiona Zitadel (proyecto + apps OIDC), dejando
|
Musify's storage bucket, and provisions Zitadel (project + OIDC apps),
|
||||||
los client ids en `deploy/.env`. Detalle completo, incluido cómo arrancar las
|
writing the client ids into `deploy/.env`. Full details, including how to run
|
||||||
apps en Docker con `--profile apps`, en [deploy/README.md](../deploy/README.md).
|
the apps in Docker with `--profile apps`, are in
|
||||||
|
[deploy/README.md](../deploy/README.md).
|
||||||
|
|
||||||
## Aplicaciones
|
## Applications
|
||||||
|
|
||||||
```powershell
|
```powershell
|
||||||
dotnet run --project backend/Hosts/Musify.Api # :5111
|
dotnet run --project backend/Hosts/Musify.Api # :5111
|
||||||
@@ -24,45 +28,45 @@ dotnet run --project backend/Hosts/Musify.Worker # background
|
|||||||
npm --prefix web-player run dev # :5173
|
npm --prefix web-player run dev # :5173
|
||||||
```
|
```
|
||||||
|
|
||||||
## Mapa de puertos
|
## Port map
|
||||||
|
|
||||||
| Servicio | Puerto |
|
| Service | Port |
|
||||||
|---|---|
|
|---|---|
|
||||||
| Musify.Api | 5111 (`/scalar/v1`, `/openapi/v1.json`) |
|
| Musify.Api | 5111 (`/scalar/v1`, `/openapi/v1.json`) |
|
||||||
| Musify.StreamingGateway | 8081 (`/health`, `/media/...`) |
|
| Musify.StreamingGateway | 8081 (`/health`, `/media/...`) |
|
||||||
| web-player | 5173 (`npm run dev`) o 3000 (contenedor) |
|
| web-player | 5173 (`npm run dev`) or 3000 (container) |
|
||||||
| SeaweedFS S3 / filer / master | 8333 / 8888 / 9333 |
|
|
||||||
| Zitadel | 8080 |
|
|
||||||
| PostgreSQL | 59000 (→ 5432 en el contenedor) |
|
|
||||||
| RabbitMQ AMQP / management | 5672 / 15672 |
|
|
||||||
| Jaeger UI / OTLP | 16686 / 4317 |
|
|
||||||
| pgAdmin (`-Tools`) | 5050 |
|
|
||||||
|
|
||||||
DB: `musify_db`, usuario `postgres`/`postgres`. Bucket S3: `webapi-storage`
|
Postgres, Zitadel, SeaweedFS, RabbitMQ and Jaeger are not run by this repo.
|
||||||
(credenciales `admin_access_key`/`admin_secret_key`). Todos los valores salen de
|
Their ports are documented in the separate `Infrastructure` repository's own
|
||||||
`deploy/.env`.
|
README.
|
||||||
|
|
||||||
## Migraciones
|
DB: `musify_db`, user `postgres`/`postgres` on Infrastructure's shared
|
||||||
|
Postgres. S3 bucket: `webapi-storage` (credentials
|
||||||
|
`admin_access_key`/`admin_secret_key`, also Infrastructure's). All of these
|
||||||
|
come from `deploy/.env`.
|
||||||
|
|
||||||
El servicio `migrate` de `deploy/compose.yml` las aplica en cada `docker
|
## Migrations
|
||||||
compose up -d` (no-op si el esquema ya está al día). A mano, tras cambios de
|
|
||||||
esquema, contra la base local:
|
The `migrate` service in `deploy/compose.yml` applies them on every `docker
|
||||||
|
compose up -d` (a no-op if the schema is already current). To run them by
|
||||||
|
hand after a schema change, against the local database:
|
||||||
|
|
||||||
```powershell
|
```powershell
|
||||||
dotnet ef database update --project backend/Core/Musify.Infrastructure --startup-project backend/Core/Musify.Infrastructure --context Database
|
dotnet ef database update --project backend/Core/Musify.Infrastructure --startup-project backend/Core/Musify.Infrastructure --context Database
|
||||||
```
|
```
|
||||||
|
|
||||||
`Musify.Infrastructure` es a la vez el proyecto de migraciones y el de arranque:
|
`Musify.Infrastructure` is both the migrations project and the startup
|
||||||
tiene el `IDesignTimeDbContextFactory` y el paquete `EFCore.Design`. La cadena
|
project: it has the `IDesignTimeDbContextFactory` and the `EFCore.Design`
|
||||||
de conexión se lee de sus user-secrets, o de la variable de entorno
|
package. The connection string is read from its user-secrets, or from the
|
||||||
`Database__ConnectionString` si no hay user-secrets (así es como la usa el
|
`Database__ConnectionString` environment variable if there are no
|
||||||
contenedor `migrate`, ver `backend/Musify.Migrator.Dockerfile`).
|
user-secrets (which is how the `migrate` container uses it, see
|
||||||
|
`backend/Musify.Migrator.Dockerfile`).
|
||||||
|
|
||||||
## Stream-tickets (claves RS256)
|
## Stream tickets (RS256 keys)
|
||||||
|
|
||||||
La API firma los stream-tickets con la clave privada y el gateway los valida con
|
The API signs stream tickets with the private key, and the gateway validates
|
||||||
la pública. El servicio `keys-init` de `deploy/compose.yml` las genera en
|
them with the public one. The `keys-init` service in `deploy/compose.yml`
|
||||||
`deploy/keys/` si no existen; a mano:
|
generates them in `deploy/keys/` if they don't exist yet; to do it by hand:
|
||||||
|
|
||||||
```powershell
|
```powershell
|
||||||
openssl genpkey -algorithm RSA -pkeyopt rsa_keygen_bits:2048 -out deploy/keys/stream_private.pem
|
openssl genpkey -algorithm RSA -pkeyopt rsa_keygen_bits:2048 -out deploy/keys/stream_private.pem
|
||||||
@@ -78,74 +82,74 @@ openssl rsa -in deploy/keys/stream_private.pem -pubout -out deploy/keys/stream_p
|
|||||||
dotnet test backend/Musify.slnx
|
dotnet test backend/Musify.slnx
|
||||||
```
|
```
|
||||||
|
|
||||||
`backend/Tests/` (carpeta `/Tests/` en el `.slnx`) tiene cinco proyectos xUnit:
|
`backend/Tests/` (the `/Tests/` folder in the `.slnx`) has five xUnit projects:
|
||||||
|
|
||||||
| Proyecto | Cubre | Docker |
|
| Project | Covers | Docker |
|
||||||
|---|---|---|
|
|---|---|---|
|
||||||
| `Musify.Domain.Tests` | Lógica de los value objects (`TrackAudio`/`TrackPictures`: `IsProcessed`, `IsFailed`, `IsInProgress`). Las entidades son en su mayoría anémicas — sin comportamiento propio, nada más que testear ahí. | No |
|
| `Musify.Domain.Tests` | Value object logic (`TrackAudio`/`TrackPictures`: `IsProcessed`, `IsFailed`, `IsInProgress`). Entities are mostly anemic, with no behavior of their own beyond that. | No |
|
||||||
| `Musify.Application.Tests` | Los ~45 handlers y servicios de `Musify.Application` (Albums, Tracks, PlayLists, Users, Mixes). | No |
|
| `Musify.Application.Tests` | The ~45 handlers and services in `Musify.Application` (Albums, Tracks, PlayLists, Users, Mixes). | No |
|
||||||
| `Musify.Infrastructure.Tests` | `Database`/migraciones, `StorageService`, `StreamTicketService`, `AuditableEntityInterceptor`, `PictureService`, `SingleFlightCache`, `PlayListPresetSeeder`. | **Sí** |
|
| `Musify.Infrastructure.Tests` | `Database`/migrations, `StorageService`, `StreamTicketService`, `AuditableEntityInterceptor`, `PictureService`, `SingleFlightCache`, `PlayListPresetSeeder`. | **Yes** |
|
||||||
| `Musify.Api.Tests` | Endpoints HTTP reales (routing, auth, `ValidationFilter`, mapeo `ErrorOr`→HTTP) vía `WebApplicationFactory`, más `ErrorOrHttpExtensions`/`CurrentUser`/validators FluentValidation en aislado. | **Sí** |
|
| `Musify.Api.Tests` | Real HTTP endpoints (routing, auth, `ValidationFilter`, `ErrorOr`→HTTP mapping) via `WebApplicationFactory`, plus `ErrorOrHttpExtensions`/`CurrentUser`/FluentValidation validators in isolation. | **Yes** |
|
||||||
| `Musify.StreamingGateway.Tests` | `TicketValidator` (RS256) y `TicketValidationMiddleware` (traversal, límites de prefijo, extracción de ticket). | No |
|
| `Musify.StreamingGateway.Tests` | `TicketValidator` (RS256) and `TicketValidationMiddleware` (traversal, prefix limits, ticket extraction). | No |
|
||||||
|
|
||||||
`Musify.Worker` no tiene proyecto de test: su `Program.cs` es puro *wiring* de
|
`Musify.Worker` has no test project: its `Program.cs` is pure DI/host wiring
|
||||||
DI/host (Hangfire, MassTransit, jobs) — la lógica real que ejecuta vive en
|
(Hangfire, MassTransit, jobs). The actual logic it runs lives in
|
||||||
`Musify.Infrastructure`, ya cubierta ahí.
|
`Musify.Infrastructure`, already covered there.
|
||||||
|
|
||||||
### Application.Tests: por qué SQLite en memoria y no mocks
|
### Application.Tests: why in-memory SQLite instead of mocks
|
||||||
|
|
||||||
`IDatabase` expone `DbSet<T>` directamente, así que los handlers arman LINQ
|
`IDatabase` exposes `DbSet<T>` directly, so handlers build real LINQ
|
||||||
real (`Where`, `Include`, `Select`...) contra él — no se puede mockear eso con
|
(`Where`, `Include`, `Select`...) against it. That can't be mocked with
|
||||||
NSubstitute, no hay `IQueryable` detrás de un mock. En su lugar,
|
NSubstitute, since there's no `IQueryable` behind a mock. Instead,
|
||||||
`TestSupport/TestDatabase.cs` es un `DbContext` real sobre SQLite en memoria
|
`TestSupport/TestDatabase.cs` is a real `DbContext` on top of in-memory
|
||||||
(una conexión `:memory:` nueva por test, con foreign keys reales — a diferencia
|
SQLite (a fresh `:memory:` connection per test, with real foreign keys,
|
||||||
del proveedor InMemory de EF Core, SQLite además soporta transacciones, que dos
|
unlike EF Core's InMemory provider, SQLite also supports transactions,
|
||||||
handlers usan). `TestSupport/TestEntities.cs` y `TestConfigurations.cs`
|
which two handlers rely on). `TestSupport/TestEntities.cs` and
|
||||||
construyen entidades y configuración con valores por defecto sensatos.
|
`TestConfigurations.cs` build entities and configuration with sensible
|
||||||
`IStorageService`, `IEventBus`, etc. sí se mockean con NSubstitute, al ser
|
defaults. `IStorageService`, `IEventBus`, etc. are mocked with NSubstitute,
|
||||||
interfaces normales.
|
since those are plain interfaces.
|
||||||
|
|
||||||
### Infrastructure.Tests y Api.Tests: Testcontainers
|
### Infrastructure.Tests and Api.Tests: Testcontainers
|
||||||
|
|
||||||
Estos dos sí necesitan Docker corriendo. Levantan un Postgres real
|
These two need Docker running. They spin up a real Postgres
|
||||||
(`Testcontainers.PostgreSql`) y, en `Infrastructure.Tests`, también un
|
(`Testcontainers.PostgreSql`) and, in `Infrastructure.Tests`, a real
|
||||||
SeaweedFS real (mismo `chrislusf/seaweedfs:latest` e imagen/comando que
|
SeaweedFS too (the same `chrislusf/seaweedfs:latest` image/command as
|
||||||
`deploy/compose.yml`, no MinIO) — así `StorageService` se prueba contra el
|
`deploy/compose.yml`, not MinIO), so `StorageService` is tested against the
|
||||||
mismo backend S3-compatible que usa producción, no contra AWS S3. Los
|
same S3-compatible backend production uses, not against AWS S3. Containers
|
||||||
contenedores se levantan una vez por corrida (`ICollectionFixture`), con
|
are started once per run (`ICollectionFixture`), with real EF Core
|
||||||
migraciones EF Core reales aplicadas contra el Postgres efímero.
|
migrations applied against the ephemeral Postgres.
|
||||||
|
|
||||||
`Api.Tests` usa `WebApplicationFactory<Program>` contra ese mismo Postgres —
|
`Api.Tests` uses `WebApplicationFactory<Program>` against that same
|
||||||
routing, auth y validación reales — pero sustituye por fakes los servicios que
|
Postgres, with real routing, auth, and validation, but replaces the services
|
||||||
hablan por red (`IStorageService`, `IEventBus`, `IStreamTicketService`) y el
|
that talk over the network with fakes (`IStorageService`, `IEventBus`,
|
||||||
esquema de autenticación JWT real por un handler
|
`IStreamTicketService`), and swaps the real JWT auth scheme for a test
|
||||||
de prueba (ver `TestSupport/FakeAuthenticationHandler.cs`) que autentica según
|
handler (see `TestSupport/FakeAuthenticationHandler.cs`) that authenticates
|
||||||
un header, sin necesitar un Zitadel real.
|
based on a header, with no real Zitadel needed.
|
||||||
|
|
||||||
Escribir estos tests contra el backend real encontró y arregló dos bugs reales
|
Writing these tests against the real backend found and fixed two actual
|
||||||
(no simulados): `StorageService.RemoveFolderAsync` reventaba con
|
bugs (not staged ones): `StorageService.RemoveFolderAsync` threw a
|
||||||
`NullReferenceException` contra SeaweedFS (su respuesta `DeleteObjects` omite
|
`NullReferenceException` against SeaweedFS (its `DeleteObjects` response
|
||||||
la lista `<Error>` cuando no hay errores, a diferencia de AWS S3), y
|
omits the `<Error>` list when there are no errors, unlike AWS S3), and
|
||||||
`AuditableEntityInterceptor` nunca actualizaba `UpdatedAt` en una actualización
|
`AuditableEntityInterceptor` never updated `UpdatedAt` on a normal update
|
||||||
normal (carga → modifica → guarda) porque corría antes de que EF detectara los
|
(load → modify → save) because it ran before EF had detected the pending
|
||||||
cambios pendientes.
|
changes.
|
||||||
|
|
||||||
## Config (AppSettings)
|
## Config (appsettings)
|
||||||
|
|
||||||
Convención: `AppSettings.json` es la plantilla con valores vacíos o neutros;
|
Convention: `appsettings.json` is the template, with empty or neutral
|
||||||
`AppSettings.Development.json` tiene los valores reales de dev; en Docker todo
|
values; `appsettings.Development.json` has the real dev values; in Docker
|
||||||
se sobreescribe con variables de entorno (`Seccion__Clave`) desde
|
everything is overridden with environment variables (`Section__Key`) from
|
||||||
`deploy/compose.yml`. Para correr `Musify.Api` desde el IDE, los valores de
|
`deploy/compose.yml`. To run `Musify.Api` from the IDE, the `Authentication`
|
||||||
`Authentication` van en user-secrets (no en el repo) — cópialos de
|
values go into user-secrets (not the repo). Copy them from `deploy/.env`
|
||||||
`deploy/.env` una vez que `zitadel-init` los haya generado (ver
|
once `zitadel-init` has generated them (see
|
||||||
[deploy/README.md](../deploy/README.md)).
|
[deploy/README.md](../deploy/README.md)).
|
||||||
|
|
||||||
Secciones clave: `Authentication` (Zitadel), `Database`, `MassTransit`,
|
Key sections: `Authentication` (Zitadel), `Database`, `MassTransit`,
|
||||||
`InfrastructureStorage` (S3), `ApplicationStorage` (bucket), `Track`, `PlayList`,
|
`InfrastructureStorage` (S3), `ApplicationStorage` (bucket), `Track`, `PlayList`,
|
||||||
`UploadIntent`, `AudioTranscoder`, `Workers`, `StreamGateway` / `StreamTicket`.
|
`UploadIntent`, `AudioTranscoder`, `Workers`, `StreamGateway` / `StreamTicket`.
|
||||||
|
|
||||||
## Requisitos del host
|
## Host requirements
|
||||||
|
|
||||||
- .NET 10 SDK y Node 22.
|
- .NET 10 SDK and Node 22.
|
||||||
- **ffmpeg** en el PATH (transcode de audio del Worker).
|
- **ffmpeg** on the PATH (for the Worker's audio transcoding).
|
||||||
- Docker Desktop.
|
- Docker Desktop.
|
||||||
|
|||||||
+25
-25
@@ -1,46 +1,46 @@
|
|||||||
# Procesado de medios (Worker + MassTransit)
|
# Media processing (Worker + MassTransit)
|
||||||
|
|
||||||
El procesado pesado (transcode de audio, miniaturas, transferencias en el bucket) ocurre **fuera del request HTTP**, en el **Worker**, orquestado con **MassTransit** sobre RabbitMQ.
|
Heavy processing (audio transcoding, thumbnails, bucket transfers) happens **outside the HTTP request**, in the **Worker**, orchestrated with **MassTransit** on top of RabbitMQ.
|
||||||
|
|
||||||
## Por qué asíncrono y con routing slips
|
## Why async, and why routing slips
|
||||||
|
|
||||||
- Transcodificar audio y redimensionar imágenes es CPU/IO intensivo: no debe bloquear una llamada a la API.
|
- Transcoding audio and resizing images is CPU/IO-intensive: it shouldn't block an API call.
|
||||||
- Es un proceso **multi-paso con efectos secundarios** (descargar de S3 → transcodificar → subir resultados → actualizar DB). Si un paso falla, hay que **deshacer** los anteriores. Para eso se usan **routing slips** (patrón Saga de MassTransit): cada paso es una *activity* con su **compensación**.
|
- It's a **multi-step process with side effects** (download from S3 → transcode → upload results → update the DB). If a step fails, the previous ones need to be **undone**. That's what **routing slips** are for (MassTransit's Saga pattern): each step is an *activity* with its own **compensation**.
|
||||||
- **Outbox transaccional** (EF + MassTransit): el evento se publica de forma consistente con el cambio en la DB (no se pierde ni se duplica).
|
- **Transactional outbox** (EF + MassTransit): the event is published consistently with the DB change (never lost, never duplicated).
|
||||||
|
|
||||||
## Disparo
|
## Trigger
|
||||||
|
|
||||||
Al crear/actualizar una pista o playlist, el handler publica un evento (p. ej. `CreateTrackResourcesEvent`). Un **consumer** (`CreateTrackConsumer`, `UpdateTrackPictureConsumer`, …) construye un **routing slip** con la secuencia de activities y lo lanza.
|
When a track or playlist is created or updated, the handler publishes an event (e.g. `CreateTrackResourcesEvent`). A **consumer** (`CreateTrackConsumer`, `UpdateTrackPictureConsumer`, …) builds a **routing slip** with the sequence of activities and executes it.
|
||||||
|
|
||||||
## Builders de routing slip
|
## Routing slip builders
|
||||||
|
|
||||||
- `CreateTrackRoutingSlipBuilder` — alta de pista (consume intents, mueve ficheros, dispara procesado).
|
- `CreateTrackRoutingSlipBuilder`: creating a track (consumes intents, moves files, kicks off processing).
|
||||||
- `PictureWorkflowRoutingSlipBuilder` — pipeline de imágenes (descargar → redimensionar → subir → actualizar).
|
- `PictureWorkflowRoutingSlipBuilder`: image pipeline (download → resize → upload → update).
|
||||||
- `AudioWorkflowRoutingSlipBuilder` — pipeline de audio (descargar → transcodificar a `.m4a` → subir carpeta → actualizar).
|
- `AudioWorkflowRoutingSlipBuilder`: audio pipeline (download → transcode to `.m4a` → upload folder → update).
|
||||||
- `DeleteTrackRoutingSlipBuilder`, `DeletePlayListRoutingSlipBuilder`, `PlayListPictureSourceRoutingSlipBuilder`.
|
- `DeleteTrackRoutingSlipBuilder`, `DeletePlayListRoutingSlipBuilder`, `PlayListPictureSourceRoutingSlipBuilder`.
|
||||||
|
|
||||||
## Activities (pasos)
|
## Activities (steps)
|
||||||
|
|
||||||
Agrupadas por área:
|
Grouped by area:
|
||||||
- **Files**: `DownloadFileFromBucketActivity`, `UploadFileToBucketActivity`, `TransferFilesToBucketActivity`, `CopyFileInBucketActivity`, `RemoveFileFromBucketActivity`.
|
- **Files**: `DownloadFileFromBucketActivity`, `UploadFileToBucketActivity`, `TransferFilesToBucketActivity`, `CopyFileInBucketActivity`, `RemoveFileFromBucketActivity`.
|
||||||
- **Audio**: `GenerateAudioWorkflowPathsActivity`, `TranscodeAudioActivity` (ffmpeg → `.m4a`), `UpdateTrackAudioActivity` (guarda `AudioFolderName` y marca `Completed`).
|
- **Audio**: `GenerateAudioWorkflowPathsActivity`, `TranscodeAudioActivity` (ffmpeg → `.m4a`), `UpdateTrackAudioActivity` (saves `AudioFolderName` and marks it `Completed`).
|
||||||
- **Pictures**: `GeneratePictureWorkflowPathsActivity`, `ResizePictureActivity` (ImageSharp), `UpdateTrackPictureActivity`, `UpdatePlayListPictureActivity`.
|
- **Pictures**: `GeneratePictureWorkflowPathsActivity`, `ResizePictureActivity` (ImageSharp), `UpdateTrackPictureActivity`, `UpdatePlayListPictureActivity`.
|
||||||
- **Tracks/PlayLists**: `MarkTrackAsRemovingActivity`, `DeleteTrackFromDbActivity`, `PublishTrackProcessingEventsActivity`, y equivalentes de playlist.
|
- **Tracks/Playlists**: `MarkTrackAsRemovingActivity`, `DeleteTrackFromDbActivity`, `PublishTrackProcessingEventsActivity`, and their playlist equivalents.
|
||||||
- **UploadIntents**: `ConsumeUploadIntentsActivity`.
|
- **UploadIntents**: `ConsumeUploadIntentsActivity`.
|
||||||
|
|
||||||
`RoutingSlipCleanUpConsumer` reacciona a `Completed`/`Faulted` para limpiar (p. ej. borrar el directorio temporal de trabajo).
|
`RoutingSlipCleanUpConsumer` reacts to `Completed`/`Faulted` to clean up (e.g. deleting the temporary working directory).
|
||||||
|
|
||||||
## Transcode de audio
|
## Audio transcoding
|
||||||
|
|
||||||
`AudioTranscoderService` invoca **ffmpeg** leyendo el original por stdin y produciendo en un directorio de trabajo un único `audio.m4a`: AAC con codec/bitrate/sample-rate/profile configurables y `-movflags +faststart` (moov al inicio → seek por range). Luego `TransferFilesToBucketActivity` sube esa carpeta a `Tracks/ProcessedAudios/{folder}/` y `UpdateTrackAudioActivity` guarda el `AudioFolderName` (la carpeta **de destino en S3**, no la local) y pone el estado en `Completed`.
|
`AudioTranscoderService` calls **ffmpeg**, reading the original from stdin and producing a single `audio.m4a` in a working directory: AAC with a configurable codec/bitrate/sample-rate/profile, and `-movflags +faststart` (moov at the start, so range-based seeking works). `TransferFilesToBucketActivity` then uploads that folder to `Tracks/ProcessedAudios/{folder}/`, and `UpdateTrackAudioActivity` saves the `AudioFolderName` (the **destination folder in S3**, not the local one) and sets the status to `Completed`.
|
||||||
|
|
||||||
## Configuración que necesita el Worker
|
## What the Worker needs configured
|
||||||
|
|
||||||
El Worker registra: storage (S3), transcoder (ffmpeg), pictures, DB, los consumers de MassTransit y los jobs de upload intents. Su `AppSettings` debe tener las secciones correctas: `Bucket` (= `webapi-storage`, el mismo que la API), `PlayList`, `Track`, `MassTransit`, `AudioTranscoder`, `UploadIntent`, `Workers` (directorio temporal), `InfrastructureStorage`, `Database`.
|
The Worker registers: storage (S3), transcoder (ffmpeg), pictures, the DB, the MassTransit consumers, and the upload-intent jobs. Its `appsettings` needs the right sections: `Bucket` (= `webapi-storage`, same as the API), `PlayList`, `Track`, `MassTransit`, `AudioTranscoder`, `UploadIntent`, `Workers` (temp directory), `InfrastructureStorage`, `Database`.
|
||||||
|
|
||||||
## Jobs de fondo
|
## Background jobs
|
||||||
|
|
||||||
- `UploadIntentExpirationJob` — marca intents caducados como expirados (libera cuota) cada `ExpirationJobIntervalSeconds`.
|
- `UploadIntentExpirationJob`: marks expired intents as such (freeing quota) every `ExpirationJobIntervalSeconds`.
|
||||||
- `TemporalUploadsCleanUpJob` — limpia subidas temporales antiguas.
|
- `TemporalUploadsCleanUpJob`: cleans up old temporary uploads.
|
||||||
|
|
||||||
Corren en el host del Worker (`AddUploadIntentJobs`). Si no se registran en ningún proceso, los intents colgados no se limpian solos.
|
Both run on the Worker host (`AddUploadIntentJobs`). If they aren't registered anywhere, stale intents never get cleaned up on their own.
|
||||||
|
|||||||
Some files were not shown because too many files have changed in this diff Show More
Reference in New Issue
Block a user