Skip to content

[_] polish docker#59

Merged
TamaraFinogina merged 2 commits into
masterfrom
upgrade_docker
Mar 20, 2026
Merged

[_] polish docker#59
TamaraFinogina merged 2 commits into
masterfrom
upgrade_docker

Conversation

@TamaraFinogina
Copy link
Copy Markdown
Contributor

Fix issues found by the auto-scanner (user not set, privilege escalation is not prohibited) + warning about version being deprecated

@TamaraFinogina TamaraFinogina requested a review from sg-gs March 11, 2026 15:20
@TamaraFinogina TamaraFinogina self-assigned this Mar 11, 2026
Comment thread Dockerfile
Copy link
Copy Markdown
Member

@sg-gs sg-gs left a comment

Choose a reason for hiding this comment

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

Has this been tested through execution? The build can work but that does not mean the execution will start successfully.

@TamaraFinogina
Copy link
Copy Markdown
Contributor Author

TamaraFinogina commented Mar 11, 2026

@sg-gs I tested it locally - calls were working (create call, join call as invited user, finish the call)

Comment thread development.Dockerfile
Copy link
Copy Markdown
Member

@sg-gs sg-gs left a comment

Choose a reason for hiding this comment

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

On your side @jzunigax2

Comment thread Dockerfile
Comment thread Dockerfile
Comment thread Dockerfile
@sonarqubecloud
Copy link
Copy Markdown

@TamaraFinogina TamaraFinogina merged commit 016d285 into master Mar 20, 2026
7 checks passed
@TamaraFinogina TamaraFinogina deleted the upgrade_docker branch March 20, 2026 13:55
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.

3 participants