Page MenuHomeIn-Portal Phabricator

INP-1926 - Display "Hostname" and "Server Hostname" in the Session Log
ClosedPublic

Authored by alex on Wed, Sep 3, 11:51 AM.

Details

Test Plan
NOTE: Perform these tests after clean install and after upgrade.

Preparations

Apply the patch from the D539 if it's not already committed.

Test Plan

  1. login to the Admin Console
  2. go to the ConfigurationWebsiteAdvanced section
  3. enable the Track database changes to change log system setting and save changes
  4. re-login to the Admin Console
  5. go to the Logs & ReportsSession Log section
  6. confirm, that these extra columns are present and aren't empty:
    • Hostname
    • Server Hostname

Diff Detail

Repository
rINP In-Portal
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

alex created this revision.Wed, Sep 3, 11:51 AM
alex requested review of this revision.Wed, Sep 3, 11:51 AM
erik requested changes to this revision.Wed, Sep 3, 12:25 PM

All by test plan is ok, but missing phrase for "Server Hostname" field (Session log grid column title).

This revision now requires changes to proceed.Wed, Sep 3, 12:25 PM
alex edited the test plan for this revision. (Show Details)Thu, Sep 4, 2:18 AM
alex requested review of this revision.Thu, Sep 4, 2:24 AM
In D542#11101, @erik wrote:

All by test plan is ok, but missing phrase for "Server Hostname" field (Session log grid column title).

The missing phrase was added as part of D539. Since it's committed already, then you:

  • either started testing before I've made a commit (before Wed, Sep 3, 19:46)
  • or haven't updated a project before starting the testing

Either way please update the project now and repeat the test.

erik requested changes to this revision.Thu, Sep 4, 5:11 AM

In the D539 added another phrase. Phrase la_fld_ServerHostname is detected as missing during tests.

This revision now requires changes to proceed.Thu, Sep 4, 5:11 AM
alex updated this revision to Diff 1405.Thu, Sep 4, 5:57 AM

Use "la_fld_LogServerHostname" phrases (existing) instead of "la_fld_ServerHostname" (non-existing) in the Session Log grid.

erik accepted this revision.Thu, Sep 4, 6:28 AM
This revision is now accepted and ready to land.Thu, Sep 4, 6:28 AM
This revision was landed with ongoing or failed builds.Thu, Sep 4, 7:28 AM
This revision was automatically updated to reflect the committed changes.