Lanmikeman/openvk
Lanmikeman/openvk
1
1
Fork 0
mirror of https://github.com/openvk/openvk synced 2024-09-21 08:36:17 +03:00
Code Issues Projects Releases Packages Wiki Activity

Global: Fix CSRF vulnerabilities in /logout and language change

Browse source 
Closes #281
This commit is contained in:
Maxim Leshchenko 2021-12-05 00:48:25 +02:00
parent 735e75fa21
commit 98f8819c17
No known key found for this signature in database
GPG key ID: BB9C44A8733FBEEE
4 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
Web/Presenters/AboutPresenter.php
View file

@ -63,6 +63,7 @@ final class AboutPresenter extends OpenVKPresenter
$this->template->languages = getLanguages();
if(!is_null($_GET['lg'])){
$this->assertNoCSRF();
setLanguage($_GET['lg']);
}
}

1
Web/Presenters/AuthPresenter.php
View file

@ -182,6 +182,7 @@ final class AuthPresenter extends OpenVKPresenter
function renderLogout(): void
{
$this->assertUserLoggedIn();
$this->assertNoCSRF();
$this->authenticator->logout();
Session::i()->set("_su", NULL);

2
Web/Presenters/templates/@layout.xml
View file

@ -108,7 +108,7 @@
</a>
</div>
<div class="link">
<a href="/logout">{_"header_log_out"}</a>
<a href="/logout?hash={urlencode($csrfToken)}">{_"header_log_out"}</a>
</div>
<div class="link">
<form action="/search" method="get">

2
Web/Presenters/templates/About/Language.xml
View file

@ -8,7 +8,7 @@
{block content}
<div class="navigation">
{foreach $languages as $language}
<a href="language?lg={$language['code']}" class="link"><img src="/assets/packages/static/openvk/img/flags/{$language['flag']}.gif"> {$language['native_name']}</a>
<a href="language?lg={$language['code']}&hash={urlencode($csrfToken)}" class="link"><img src="/assets/packages/static/openvk/img/flags/{$language['flag']}.gif"> {$language['native_name']}</a>
{/foreach}
</div>
{/block}