bug: v1 api endpoint /get_replay appears to be incomplete
skrungly opened this issue · 3 comments
Describe the bug
replays downloaded from v1/get_replay cannot be processed by the client, because the replay headers are missing regardless of the include_headers parameter. presumably, include_headers=True should return a functioning .osr file, but the server code seems to process it the other way around.
even with this in mind, specifying include_headers=False still provides the same incomplete replay file because the replay_data variable which contains the headers is never actually affixed to the raw replay file before it is sent off.
To Reproduce
- download the .osr file from https://api.skrungly.com/v1/get_replay?id=1447&include_headers=True (also ensure you have the map)
- import into your game
- stonks!
Expected behavior
when specifying include_headers=True, the .osr file should include headers.
bancho.py Version
latest
Python Version
3.9.X (Default)
Relevant log output
No response
Additional context
i understand that the v1 api is soon to be deprecated, but to my knowledge it is currently the only way to receive certain data (such as whole replay files) from the server, and so it should at least be functional in the meantime.
can you check to see if #459 fixes this for you? i run the changes in that PR on my main production server (example replay from the server and the map you'll need)
yep! i looked but somehow didn't spot your PR in the works. literally just implemented a nearly-identical fix for my own server hehe.
banger of a map btw! :D
i had just made it ~5 minutes after you made this issue, so i can see how you missed it lol
glad you found a fix on your own 👍