Jump to content
er314

PW version displayed on the backend login screen

Recommended Posts

this one is trivial :-)

feature request : PW should not disclose its version number to unauthenticated users on the backend login screen.

ok, this is about security through mere obscurity, so nothing vital here, but experience tells that attackers really love to easily know the exact version of their target.

  • Like 4

Share this post


Link to post
Share on other sites

If there were (known) vulnerabilities, this would be more important, but +1 from me for changing this behaviour anyway. In my opinion there's absolutely no need to disclose the specific version number :)

Share this post


Link to post
Share on other sites

Yes, here it's more about general good practice.

Imagine that tomorrow a security flaw is discovered in current PW (2.7.2 and older). So, the day after tomorrow :-) , PW 2.7.3 is released, fixing the flaw. So everyone upgrades to 2.7.3. But if some site is not upgraded, then it is at risk. And, as PW discloses its version number, the attackers will be happy :-)

Share this post


Link to post
Share on other sites

It's basically that in both admin theme's default.php files:

--- /Users/Benni/Projekte/ProcessWire2/www/html/wire/modules/AdminTheme/AdminThemeReno/default.php
+++ /Users/Benni/Projekte/ProcessWire2/www/html/wire/modules/AdminTheme/AdminThemeReno/default.php (Unsaved)
@@ -129,7 +129,7 @@
 				<a class="action" href="<?php echo $config->urls->admin; ?>login/logout/"><i class="fa <?php echo $adminTheme->signout;?>"></i> <?php echo $helpers->_('Logout'); ?></a>
 			</span>
 		<?php endif; ?>
-		ProcessWire <?php echo $config->versionName . ' <!--v' . $config->systemVersion; ?>--> © <?php echo date("Y"); ?> 
+		ProcessWire <?php if($user->isLoggedIn())echo $config->versionName . ' <!--v' . $config->systemVersion . '-->'); ?> © <?php echo date("Y"); ?> 
 	</p>

 	<?php
  • Like 2

Share this post


Link to post
Share on other sites

Makes sense to me, I'll move that <?php endif; ?> down one line, so that the name/version only appears when logged in. 

  • Like 4

Share this post


Link to post
Share on other sites

Makes sense to me, I'll move that <?php endif; ?> down one line, so that the name/version only appears when logged in. 

Once an update was made, the version number gets visible again due the notification message ;)

  • Like 1

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


  • Recently Browsing   0 members

    No registered users viewing this page.

×
×
  • Create New...