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.

Sign up for GitHub

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

Jump to bottom

Date & Time is shown for the "Modified by" field instead of "User Name" under the Trusted Application. #102308

Closed
ghost opened this issue Jun 16, 2021 · 9 comments · Fixed by #102377
Closed

Date & Time is shown for the "Modified by" field instead of "User Name" under the Trusted Application. #102308

ghost opened this issue Jun 16, 2021 · 9 comments · Fixed by #102377
Assignees
@academo
Labels
bug Fixes for quality problems that affect the customer experience impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. OLM Sprint QA:Validated Issue has been validated by QA Team:Defend Workflows “EDR Workflows” sub-team of Security Solution Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v7.14.0

Comments

@ghost
Copy link

ghost commented Jun 16, 2021 •
edited by manishgupta-qasource
Loading

Describe the bug
Date & Time is shown for the "Modified by" field instead of "User Name" under the Trusted Application.

Build Details

Version: 7.14.0-SNAPSHOT
Commit: 9838db3
Build number: 41559
Artifact: https://artifacts-api.elastic.co/v1/search/7.14.0-SNAPSHOT

Preconditions
Elastic 7.14.0-SNAPSHOT environment should be deployed.

Steps to Reproduce

  1. Navigate to the administration tab under security.
  2. Go to trusted application tab and add an trusted application.
  3. Observe that Date & Time is shown for the "Modified by" field instead of "User Name" under the Trusted Application.

Test data
N/A

Impacted Test case(s)
N/A

Actual Result
Date & Time is shown for the "Modified by" field instead of "User Name" under the Trusted Application.

Expected Result
User Name should be displayed in "Modified by" field .
or
Modified by field name should be changed "Modified On"

What's Working
N/A

What's Not Working
N/A

Screenshot:
modifiedbycolumn (2)

Logs:
N/A
@ghost ghost added bug Fixes for quality problems that affect the customer experience Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. labels Jun 16, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

@ghost
Copy link
Author

ghost commented Jun 16, 2021

@manishgupta-qasource please review.

@muskangulati-qasource muskangulati-qasource added impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. Team:Defend Workflows “EDR Workflows” sub-team of Security Solution v7.14.0 labels Jun 16, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-onboarding-and-lifecycle-mgt (Team:Onboarding and Lifecycle Mgt)

@manishgupta-qasource manishgupta-qasource changed the title Date & Time is shown for the modified by column under the Trusted Application. Date & Time is shown for the "Modified by" column instead of "User Name" under the Trusted Application. Jun 16, 2021
@manishgupta-qasource manishgupta-qasource changed the title Date & Time is shown for the "Modified by" column instead of "User Name" under the Trusted Application. Date & Time is shown for the "Modified by" field instead of "User Name" under the Trusted Application. Jun 16, 2021
@manishgupta-qasource
Copy link

Reviewed & assigned to @kevinlog

@manishgupta-qasource manishgupta-qasource added impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. and removed impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. labels Jun 16, 2021
@kevinlog kevinlog assigned academo and unassigned kevinlog Jun 16, 2021
@kevinlog
Copy link
Contributor

@academo can you take a look?

@academo
Copy link
Contributor

academo commented Jun 16, 2021

@samratbhadra-qasource

I did a test with a fresh elastic search instance on the exact same commit reported in the branch 7.x and I could not reproduce this bug.

Attached a video of my test

Peek 2021-06-16 17-48

@academo
Copy link
Contributor

academo commented Jun 16, 2021

@samratbhadra-qasource

even though I could not reproduce the issue, after checking further in the code I can see a case where this error can happen. I will be pushing a fix soon.

@academo academo mentioned this issue Jun 16, 2021
Merged
1 task
academo added a commit that referenced this issue Jun 17, 2021
@academo
Fix trusted apps modified by field displayed as a date field (#102377)
a83d0e7 
Fixes #102308
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Jun 17, 2021
@academo @kibanamachine
Fix trusted apps modified by field displayed as a date field (elastic…
d520a10 
…#102377)

Fixes elastic#102308
@academo academo reopened this Jun 17, 2021
@academo academo added the QA:Ready for Testing Code is merged and ready for QA to validate label Jun 17, 2021
kibanamachine added a commit that referenced this issue Jun 17, 2021
@kibanamachine @academo
Fix trusted apps modified by field displayed as a date field (#102377) (
d8acb7b 
#102458)

Fixes #102308

Co-authored-by: Esteban Beltran <[email protected]>
@ghost
Copy link
Author

ghost commented Jun 21, 2021

Hi @kevinlog ,

We tested this ticket on the latest 7.14.0-SNAPSHOT build & found that issue is now Fixed. Please find below the exact testing details:

Build Details:

VERSION: 7.14.0-SNAPSHOT
BUILD: 41846
COMMIT: 3cfbe712030c898819006195e6c9ddae6cbd0a3a
ARTIFACT: https://artifacts-api.elastic.co/v1/search/7.14.0-SNAPSHOT

Screenshots
User Name is displaying in "Modified by" field .
Screenshot (71)

Hence, closing this issue and marking it as "Validated".

Thanks

@ghost ghost closed this as completed Jun 21, 2021
@ghost ghost added QA:Validated Issue has been validated by QA and removed QA:Ready for Testing Code is merged and ready for QA to validate labels Jun 21, 2021
@muskangulati-qasource
Copy link

Exist ## Bug Conversion - Test-Case not required as this particular checkpoint is already covered in the following test case:     - https://elastic.testrail.io/index.php?/cases/view/28363 Thanks!

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@academo academo

Labels
bug Fixes for quality problems that affect the customer experience impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. OLM Sprint QA:Validated Issue has been validated by QA Team:Defend Workflows “EDR Workflows” sub-team of Security Solution Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v7.14.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix trusted apps modified by field displayed as a date field academo/kibana
5 participants
@academo @elasticmachine @kevinlog @manishgupta-qasource @muskangulati-qasource