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

[DX] Collect Telemetry data on Web Server. #5423

Closed
quicksketch opened this issue Dec 31, 2021 · 4 comments · Fixed by backdrop/backdrop#3882
Closed

[DX] Collect Telemetry data on Web Server. #5423

quicksketch opened this issue Dec 31, 2021 · 4 comments · Fixed by backdrop/backdrop#3882

Comments

@quicksketch
Copy link
Member

quicksketch commented Dec 31, 2021

Description of the need

In addition to PHP version, MySQL version, and Server OS, we should collect web server: Apache or nginx (supported platforms) as well as other options (either consolidated into "Other" or separated).

Proposed solution

Add Web Server into system_telemetry_data().

Part of meta issue #5231.


translation update:

New strings:

  • Web server
  • The web server that delivers HTTP pages to the web browser, such as Apache or nginx.
@quicksketch quicksketch changed the title [DX] Telemetry: Add Web Server [DX] Telemetry: Add Web Server (Apache/nginx/etc) Dec 31, 2021
@quicksketch
Copy link
Member Author

PR filed at backdrop/backdrop#3882 that segments into the following:

  • Apache
  • nginx
  • Other

I'm not sure if we should try and identify other options individually or not. Backdrop officially supports Apache and nginx. Other webservers that could run Backdrop and are above 1% web usage are Litespeed and IIS according to https://w3techs.com/technologies/overview/web_server

image

Options like "Cloudflare Server" and "Google Server" I believe are not application servers but merely reverse proxy caches that site in front of the application server.

@klonos
Copy link
Member

klonos commented Dec 31, 2021

This metric would help in issues like #5047 #4743 #4335, but if the "other" metric is consistently considerably below 20%, then I don't see how it would matter to dig further into which these "other" web servers are 🤷🏼

@klonos
Copy link
Member

klonos commented Dec 31, 2021

Thanks @quicksketch 🙏🏼 ...this is RTBC, but I've left a comment in the PR re using title case for Web Server.

@laryn
Copy link
Contributor

laryn commented Jan 12, 2022

Thanks @quicksketch for filing and fixing, and @klonos for the review! Merged into 1.x and 1.20.x.

@jenlampton jenlampton changed the title [DX] Telemetry: Add Web Server (Apache/nginx/etc) [DX] Collect Telemetry data on Web Server. Jan 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants