Problems accessing the admin/user page

pb66 · 2018-09-21T12:27:11.487Z

There are a few threads about issues logging in to new installs and there is also a discussion about the admin session timeout, since updating a old test instance to the very latest everything, I have this odd quirk that when the admin session times out and I click log out and then log back in again, this is what I see

image2023×1134 120 KB

Why is there a “Admin re-authentication required” banner there when I have just logged in as an admin user?

Also none of the buttons in the navbar on the right work (except log out which just circles back here once I log in again), the apps menu works but not the dashboard link (on the left)

Below the footer there is what appears to be a unpopulated user list, which is my chosen landing page

And even more bizarrely, what’s going on when I narrow the page

image1197×1044 57.1 KB

I now have a page within a page!!!

My only way out of this page(s) is to open the apps page at which point my other navbar button become live and I can navigate to the page of my choice

here’s the database update page accessed via the admin page

image1147×289 16.4 KB

But! If I go to admin/users I get that funky page again

image1166×1005 42.7 KB

At this point I’m running all master branches, all fully updated and have cleared browser cache and updated database, there are no errors reported in the log or in browser console, I have done a “git diff” and I have no local changes.

When i started writing this I thought it was a log-in or session auth issue, I now think it might just be an issue with the admin/user list page perhaps?

pb66 · 2018-09-21T14:37:58.646Z

I have just tried defining a different start page (now feed/list, was admin/users) and it logs in just fine to the feeds page, but I still get a funky admin/users page when I navigate to it via the admin page, which, that route means I am definitely an admin level user at that time, so this is looking more like just a issue with the admin/users page so I will change the thread title from “Bizarre login behavior”.

TrystanLea · 2018-09-21T15:22:17.057Z

Thats strange! sanity checking here looks fine… can anyone else replicate this?

pb66 · 2018-09-21T20:23:15.399Z

More info on this, I get a warning in the browser console that is not present on any other page

DevTools failed to parse SourceMap: https://myserver.co.uk/admin/hammer.min.js.map

the link https://myserver.co.uk/admin/hammer.min.js.map lands me this page

image1457×650 30 KB

and looking at the source think it is looking for https://myserver.co.uk/Lib/hammer.min.js.map which is valid

image1908×1019 61.7 KB

There has been some work done in the basic theme recently (ref added detection of swipe · emoncms/emoncms@a3fc8cf · GitHub) involving hammer.min.js.map, could that be the cause?

I was unable to get it working by commenting out that code so I tried some-backdating to see where it stopped working and I have to go back to 9.8.29 to get it working fully

Screenshot_61191×468 18.8 KB

But both 9.8.30 and 9.8.31 both result in a blank page, no user table or error banner

image1187×478 16 KB

and the current master (or indexedinputs)

image1191×996 42.3 KB

pb66 · 2018-09-26T10:10:26.996Z

Any thoughts on how to proceed with this ? @TrystanLea or @emrys

There have been significant changes overall since 9.8.30

But the changes between 9.8.29 and 9.8.30 where I lost the admin page are

Comparing 9.8.29...9.8.30 · emoncms/emoncms · GitHub

and the changes between 9.8.31 and the current master where it changed behavior are

Comparing 9.8.31...master · emoncms/emoncms · GitHub

pb66 · 2018-09-26T10:37:19.368Z

Ok, a bit more info…

I just tried commenting out some of the more recent changes and it has started working ok.

I have commented out this whole block in admin_controller.php

github.com

emoncms/emoncms/blob/a8975097ea5fd88abcf46fbb7b75efa4a4f62ec6/Modules/admin/admin_controller.php#L23-L40

    
      
          if ($updatelogin===true) {
              $route->format = 'html';
              if ($route->action == 'db')
              {
                  $applychanges = false;
                  if (isset($_GET['apply']) && $_GET['apply']==true) $applychanges = true;
          
          
        require_once "Lib/dbschemasetup.php";
                  $updates = array(array(
                      'title'=>"Database schema", 'description'=>"",
                      'operations'=>db_schema_setup($mysqli,load_db_schema(),$applychanges)
                  ));
          
          
        return array('content'=>view("Modules/admin/update_view.php", array('applychanges'=>$applychanges, 'updates'=>$updates)));
              }
          }
          
          
if ($session['admin']) {

and replaced it with

    if ($updatelogin || $session['admin']) {

essentially undoing the changes made to this file in this commit

github.com/emoncms/emoncms

include db update login bypass setting in settings.php

committed 05:57PM - 03 May 18 UTC

+26 -1

following on from that I have found I must of previously forgotten to undo the $updatelogin = true: setting following a problem with logging in.

    // Set to true to run database update without logging in
    // URL Example: http://localhost/emoncms/admin/db
    $updatelogin = true;

setting this to false and reverting the file to master has worked and I have the page back.

Shouldn’t the if ($updatelogin) block come after the if ($session['admin']) block rather than before so that only if it’s not an active admin session will the $updatelogin setting be checked?

TrystanLea · 2018-09-26T21:27:07.564Z

aha, yes of course, thankyou for spotting that, I’ve moved it to an ‘else if’ at the bottom:

github.com/emoncms/emoncms

fix update login

committed 09:26PM - 26 Sep 18 UTC

+16 -16