Files
gamer147 bf783639c1 fix(rank-battle): inherit BaseRequest so auth fields survive translation roundtrip
The translation middleware decrypts + msgpack-decodes the request body
into the action's first-parameter type, then re-serializes that DTO to
JSON for the auth handler to read. Phase 3's DoMatchingRequestDto and
RankBattleFinishRequestDto didn't inherit BaseRequest, so viewer_id /
steam_id / steam_session_ticket were dropped during the msgpack → DTO
→ JSON pivot — the auth handler then saw a body with no auth fields
and 401'd every request.

Fixed by making both DTOs extend BaseRequest, mirroring the Phase 2 TK2
DoMatchingRequest pattern.

Also added [FromBody] BaseRequest parameters to the previously body-less
actions (AiStart × 2, ForceFinish, AddClientLog, GetLatestMasterPoint).
The translation middleware explicitly requires at least one parameter
to bind the decrypted msgpack body (see L130-136 of the middleware);
without it the request would throw InvalidOperationException at runtime.

Tests updated to post viewer_id / steam_id / steam_session_ticket
placeholder values in the request body, matching the existing TK2 test
pattern.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-06-02 09:29:48 -04:00

44 lines
1.4 KiB
C#

using System.Text.Json.Serialization;
using MessagePack;
using SVSim.EmulatedEntrypoint.Models.Dtos.Requests;
namespace SVSim.EmulatedEntrypoint.Models.Dtos.RankBattle;
/// <summary>
/// Standard DoMatchingParam shape for rank battle (rotation/unlimited). Inherits viewer_id /
/// steam_id / steam_session_ticket from <see cref="BaseRequest"/> so the auth fields survive
/// the translation-middleware msgpack → DTO → JSON round-trip (otherwise the
/// SteamSessionAuthenticationHandler sees a body without auth fields and 401s).
/// </summary>
[MessagePackObject]
public sealed class DoMatchingRequestDto : BaseRequest
{
[JsonPropertyName("deck_no")]
[Key("deck_no")]
public int DeckNo { get; set; }
[JsonPropertyName("need_init")]
[Key("need_init")]
public int NeedInit { get; set; }
[JsonPropertyName("card_master_hash")]
[Key("card_master_hash")]
public string? CardMasterHash { get; set; }
[JsonPropertyName("log")]
[Key("log")]
public int Log { get; set; }
[JsonPropertyName("use_stage_select")]
[Key("use_stage_select")]
public int UseStageSelect { get; set; }
[JsonPropertyName("excluded_field_id_list")]
[Key("excluded_field_id_list")]
public List<long> ExcludedFieldIdList { get; set; } = new();
[JsonPropertyName("is_default_skin")]
[Key("is_default_skin")]
public int IsDefaultSkin { get; set; }
}