Some works I am doing on the cockpit side to integrate the apache features inside the new server-manager, the new things I needed to learn is to make a table from a key type/database, well a lot of fun.
2 posts were split to a new topic: Install wordpress inside a virtualhost
the dashboard is coming, I detect all php scl and mysql version
I have the best to do, make this
to a nice graph
well some fun to come
Well done Steph!
- I’d change some labels, the most important is the app title: why not keeping the same Nethgui title: “Web server”?
- About the graph: it needs a caption/label/title to understand its meaning. What about “Server workload”? And a caption for each colour.
The “Lamp status” is a nice idea, we can develop further. I understand its purpose is to inform the admin of the available software stack, to install some PHP web application in a vhost home.
If possible I’d group together some elements, instead of giving the bare RPM list. Furtheremore, I’d not limit the list to the “LAMP” concept: also PostgreSQL is citizen here
For instance, the groups could be
- PHP version(s) and which one is using FPM
- SQL Databases
- Apache itself – we can never forget it
All this information can be folded, like you can see in other Cockpit dashboards, because it’s required only during (hopefully) rare occasions, like a new web app software installation.
nice ideas, I wanted yesterday to detect portsgre(official and scl), but I stepped back because of the time to detect all services, but it is probably just a matter of customisations/adaptations, so yes I will go further
agreed but web server could be also nginx
The works on the graph is not terminated, so definitively yes, captation and title, but due to css, it is hard, I hope I will have more chance soon
Agree, I would leave the original name that hides the software we’re using.
It’s almost time to test it
READY to testing -> https://github.com/NethServer/dev/issues/5778#issuecomment-503465646
please go to verify this new nethserver-package
the goals are :
- apache virtualhosts
Thank to @dev_team for their supports
Hi Steph! I’m testing your RPM and I hope other people will join this conversation and test it too!
First of all, great work! Let’s discuss what can be polished now.
[root@vm5 ~]# rpm -qa nethserver-httpd* nethserver-httpd-virtualhosts-3.2.7-1.126.g4b93876.ns7.noarch nethserver-httpd-admin-2.3.4-1.ns7.noarch nethserver-httpd-3.2.7-1.126.g4b93876.ns7.noarch nethserver-httpd-proxypass-3.2.7-1.126.g4b93876.ns7.noarch
Ok! but I want to test it on a real server
- There’s still a missing space in the title label =>
Create a virtual host
- Inline notification text:
The server alias name of each virtual hosts will be automatically created. The plural
virtual hostsshould be
virtual host. I’d reword it as
Virtual host names will be added as host name aliases under System > Dashboard > Hostname.
Allow access from trusted networks onlyenabled by default. Now that it is folded under “Advanced settings” I’d change the default state to unchecked.
I’m afraid the enabled default is too restrictive and might produce an unexpected result. Once created the access to the web site is forbidden from a public IP. It might be right for a testing web site, but misleading.
What do you think?
Createbutton is pushed with no value in
Virtual host names (FQDN)field the validator error message is
This FQDN is already used in another virtualhost: I propose to mark the HTML field mandatory to prevent server validation or to fix the validator message and reflect the actual error condition.
- The validator label
This FQDN is already used in another virtualhostshould be
The virtual host name is already used
- Take note of
Enable FTP accesslabel text and compare it in the next section.
FTP accesslabel from previous screenshot becomes
FTP authin the List view. I’d go with
- The items in the List view seem to have no sorting rule applied. I’d expect
Defaultalways in the first position, then the other items in alphabetical sort.
- The dialog title should display the (first) virtual host name, e.g.
Delete virtual host www.third.net
- I’d remove the warning message completely
- The inline notification text could be reworded, according to the creation form:
Existing host name aliases under System > Dashboard > Hostname are preserved.
First-time empty state:
- The tile should be
No FTP users found. Subtitle:
Create the first FTP user account to access the FTP server.
Once an user is created I can see:
…I don’t like the “FTP server is disabled” empty state. I think we can display the normal layout instead, that one I can see when both FTP server is enabled and at least one user exists:
To recap the changes I propose here are
- “Configuration” section always visible: it is necessary to enable/disable the FTP server for Virtual hosts. Remove the “FTP server is disabled” empty state and display the normal layout.
- Change the first time empty state labels:
No FTP users found-
Create the first FTP user account to access the FTP server
- If possible, I’d list the virtual hosts FTP users here: they cannot just be enabled/disabled.
- the event notification messages say “Account xyz updated”: I’d say “FTP user” instead.
- when FTP is enabled there’s an untranslated label in the event notification message
- Change title label
Delete user <username>
- Subtitle label:
A reverse proxy forwards HTTP client requests to another web server
- Fix the Info text as proposed in the
Virtual hosts > Creation formsection above
- Title label without
Delete reverse proxy <fpname>
- Remove the warning completely
- Fix the Info text as proposed in the
List view > Delete formsection above
Here and There
Event notification messages say “X has been well updated”: I’d remove “well”!
I agree on this.
We already used the same behavior on other pages.
I disagree See above.
I’ve already evaluated it, IMHO it can be misleading but I’m not sure.
For all other label changes, I have no preferences, so feel free to open a PR.
Well, it is a new way now to use the virtualhost, proxypass, FTP, cockpit is cool