Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When checking if a view is allowed, having no session can result in errors #3781

Closed
bmfmancini opened this issue Aug 31, 2020 · 1 comment
Closed
Labels
bug Undesired behaviour resolved A fixed issue
Milestone

Comments

@bmfmancini
Copy link
Member

bmfmancini commented Aug 31, 2020

Hey Guys

I found the following errors when logging into a new 1.2.14 install
This was a clone of the 1.2.x branch this morning

2020/08/31  15:01:06 - CMDPHP PHP ERROR NOTICE Backtrace:   (/index.php[25]:include(), /include/auth.php[171]:require_once(),  /auth_login.php[669]:html_common_header(),  /lib/html.php[2381]:is_view_allowed(),  /lib/auth.php[617]:CactiErrorHandler())

2020/08/31 15:01:06 - ERROR PHP NOTICE: Undefined index:  sess_user_id in file: /var/www/html/cacti/lib/auth.php  on line: 617
2020/08/31 15:01:06 - CMDPHP PHP ERROR NOTICE Backtrace:   (/index.php[25]:include(), /include/auth.php[171]:require_once(),  /auth_login.php[669]:html_common_header(),  /lib/html.php[2381]:is_view_allowed(),  /lib/auth.php[603]:CactiErrorHandler())
2020/08/31 15:01:06 - ERROR PHP NOTICE: Undefined index:  sess_user_id in file: /var/www/html/cacti/lib/auth.php  on line: 603
root@cact1:/var/www/html/cacti# php -v
PHP 7.0.33-0ubuntu0.16.04.15 (cli) ( NTS )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.0.0, Copyright (c) 1998-2017 Zend Technologies
    with Zend OPcache v7.0.33-0ubuntu0.16.04.15, Copyright (c) 1999-2017, by Zend Technologies
@bmfmancini bmfmancini added bug Undesired behaviour unverified Some days we don't have a clue labels Aug 31, 2020
TheWitness added a commit that referenced this issue Sep 6, 2020
Cacti V1.2.14 php backtraces on Auth.php errors upon login
@TheWitness TheWitness added resolved A fixed issue and removed unverified Some days we don't have a clue labels Sep 6, 2020
@TheWitness TheWitness added this to the 1.2.15 milestone Sep 6, 2020
@TheWitness
Copy link
Member

Yea, I can see how this would happen. Fixed.

@netniV netniV changed the title Cacti V1.2.14 php backtraces on Auth.php errors upon login When checking if a view is allowed, having no session can result in errors Nov 2, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Feb 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Undesired behaviour resolved A fixed issue
Projects
None yet
Development

No branches or pull requests

2 participants