Skip to content

bugfix/add: замена камеры на штативную#8557

Open
NuSanya wants to merge 6 commits intoss220-space:master220from
NuSanya:tripod-stuff
Open

bugfix/add: замена камеры на штативную#8557
NuSanya wants to merge 6 commits intoss220-space:master220from
NuSanya:tripod-stuff

Conversation

@NuSanya
Copy link
Contributor

@NuSanya NuSanya commented Mar 3, 2026

Что этот ПР делает

Заменяет старую video camera на новую broadcast camera, при этом исправляя существующие с старой камерой баги.
Вернул закоменченные перчатки что бьют током в набор дарклорда, ибо попалось под руку и лень отдельный пр под это делать. Как понимаю, это для будущего боевого искусства было закомменчено, но сомневаюсь, что оно будет.

Демонстрация изменений

Демонстрации изменений

2026-03-03.104338.1.mp4

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request successfully replaces the legacy video camera system with a new, more functional broadcast camera, improving organization by moving code to a separate file. However, a significant security regression has been introduced: the sanitization of chat messages is explicitly disabled in the hear_talk procedure, making it vulnerable to HTML injection and XSS attacks. Furthermore, there are several areas for improvement concerning performance and stability, including a critical risk of hard deletion in Destroy(), inefficient global searches in update_feeds(), and overly complex logic in init_camera().

@NuSanya
Copy link
Contributor Author

NuSanya commented Mar 3, 2026

merge it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant