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

Version 2.2 Fork / Login Form #43

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

Conversation

mkkeck
Copy link

@mkkeck mkkeck commented Mar 15, 2017

  • Login form
  • Added session class for login form
  • Display different navigations if user is logged in / logged out

mkkeck added 5 commits March 15, 2017 21:18
* Login form
* Added session class for login form
* Display different navigations if user is logged in / logged out
Better description for login details in config_example.php
* Readme section restructured
* Config / Install description updated for admin login
* Linking to Issues
* Linking to original project from Christian Beer
* Readme section restructured
* Config / Install description updated for admin login
* Linking to Issues
* Linking to original project from Christian Beer
* Readme section restructured
* Config / Install description updated for admin login
* Linking to Issues
* Linking to original project from Christian Beer
@ChristianBeer
Copy link
Owner

Hi, do you want to create your own fork or is this meant to be merged and released as version 2.3? I just glanced at the changes which look good in general. I will have more time to review this later this week. I want to take a closer look at the session handling.

* Added security for Apache Webserver:
  - `configs/`  not accessible via browser
  - `includes/` not accessible via browser
* Moved `tables*.sql` to `install/tables*.sql`
* Added `install/config-examples` for Linux Distributions
* Added `install/config-examples/debian` as example and info how to setup on Debian Jessie
@mkkeck
Copy link
Author

mkkeck commented Mar 16, 2017

I've created my own fork. But it would be also ok to be merged and released as new version.
The session handling may be secure enough, it is based on the ideas from other great admin tools like phpMyAdmin, Roundcube etc. But such admin tools should be accessed only via https. Perhabs it would be needed to document and forced via the tool itself.

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