? ? Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
6 Dec 2021

Summary of Changes

Browsers do support a server timing header field which adds information what happens on the server side. This basically gives the developer insight what happens in the "Time to First Byte".
This pr adds the missing header in the debug plugin, as this header should not be added when in production mode because of performance reasons.

Would be nice to get some feedback about spicing up the values of the header as currently I'm just logging them as they are.

The view looks like in the following picture:
image

Testing Instructions

  • When testing in Firefox, make sure you open the page with HTTPS
  • Put joomla in debug mode
  • In the debug plugin with make sure that the Profile collector is enabled (is by default)
  • Open any page
  • Open the dev tools and check the Network tab
  • Select the first entry and then click on the timings tab of the specific request

Actual result BEFORE applying this Pull Request

Only the request timing information is shown.

Expected result AFTER applying this Pull Request

Beside the request timing is the server timing information shown.

avatar laoneo laoneo - open - 6 Dec 2021
avatar laoneo laoneo - change - 6 Dec 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Dec 2021
Category Libraries Front End Plugins
avatar laoneo laoneo - change - 6 Dec 2021
The description was changed
avatar laoneo laoneo - edited - 6 Dec 2021
avatar laoneo laoneo - change - 6 Dec 2021
The description was changed
avatar laoneo laoneo - edited - 6 Dec 2021
52f2f7c 6 Dec 2021 avatar laoneo cs
avatar laoneo laoneo - change - 6 Dec 2021
Labels Added: ?
avatar dgrammatiko
dgrammatiko - comment - 14 Jan 2022

I have tested this item successfully on cf42d70

@bembelimen this would be great Dev DX if could land in 4.1


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36231.

avatar dgrammatiko dgrammatiko - test_item - 14 Jan 2022 - Tested successfully
avatar Quy
Quy - comment - 20 Jan 2022

Maybe add a colon after each description for consistency.

avatar laoneo
laoneo - comment - 20 Jan 2022

@Quy that's how it comes from the profiler. How would you do it then, can you make a suggestion?

avatar Quy
Quy - comment - 20 Jan 2022

I have tested this item successfully on 27300cf


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36231.

avatar Quy Quy - test_item - 20 Jan 2022 - Tested successfully
avatar Quy Quy - alter_testresult - 20 Jan 2022 - dgrammatiko: Tested successfully
avatar Quy Quy - change - 20 Jan 2022
Status Pending Ready to Commit
avatar Quy
Quy - comment - 20 Jan 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36231.

avatar laoneo laoneo - change - 22 Jan 2022
Labels Added: ?
avatar bembelimen bembelimen - change - 25 Jan 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-01-25 03:46:01
Closed_By bembelimen
avatar bembelimen bembelimen - close - 25 Jan 2022
avatar bembelimen bembelimen - merge - 25 Jan 2022
avatar bembelimen
bembelimen - comment - 25 Jan 2022

Thx

Add a Comment

Login with GitHub to post a comment