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

feat: Time format in logs should display seconds instead of just hours and minutes #7335

Closed
manoj-ayyappan opened this issue Apr 11, 2024 · 2 comments · Fixed by #7336
Closed
Labels
developers This issue targets an issue that is for developers/collaborators/module writers/technical users Feature Enhancement Changing a current feature for better usage. Security
Milestone

Comments

@manoj-ayyappan
Copy link
Contributor

Describe the bug

ASVS 7.1.4 (Verify that each log event includes necessary information that would allow for a detailed investigation of the timeline when an event happens)
The OpenEMR logs events into the system which are sorted in order by date and time. The time is displayed in the format Hours:Minutes which is insufficient for tracking the timeline in which events happen. It should be changed to Hours:Minutes:Seconds

To Reproduce

Steps to reproduce the behavior:

  1. Login to OpenEMR using correct credentials
  2. Navigate to Admin --> System --> Logs
  3. Click on Submit button

image

Expected behavior

The Logs are displayed in the format Hours:Minutes

Client configuration

  • Browser: Arc Version 1.37.0 (48361) Chromium Engine Version 123.0.6312.106
  • OpenEMR version: v7.0.2
  • Operating system: MacOS
@stephenwaite
Copy link
Sponsor Member

nice @manoj-ayyappan !

can you edit the issue title to be bug: Inadequate time format in logs please?

@manoj-ayyappan manoj-ayyappan changed the title Inadequate time format in logs bug: Inadequate time format in logs Apr 11, 2024
@manoj-ayyappan
Copy link
Contributor Author

The issue and fix titles are now changed as suggested.

@adunsulag adunsulag added this to the 7.0.2.1 milestone Apr 23, 2024
@adunsulag adunsulag added developers This issue targets an issue that is for developers/collaborators/module writers/technical users Security Feature Enhancement Changing a current feature for better usage. labels May 20, 2024
@adunsulag adunsulag changed the title bug: Inadequate time format in logs feat: Time format in logs should display seconds instead of just hours and minutes May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developers This issue targets an issue that is for developers/collaborators/module writers/technical users Feature Enhancement Changing a current feature for better usage. Security
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants