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

mloginfo --clients: more robust parsing of client metadata #765

Closed
stennie opened this issue Dec 31, 2019 · 0 comments
Closed

mloginfo --clients: more robust parsing of client metadata #765

stennie opened this issue Dec 31, 2019 · 0 comments
Assignees
@stennie
Labels
improvement mloginfo
Milestone
1.6.1

Comments

@stennie
Copy link
Collaborator

stennie commented Dec 31, 2019

mloginfo --clients currently massages the client metadata input before parsing as JSON. This fails to parse some valid client metadata if the driver info isn't listed first.

The parsing of client metadata should be moved to logevent and make additional metadata available (such as os and application details).
@stennie stennie added this to the 1.6.1 milestone Dec 31, 2019
@stennie stennie self-assigned this Dec 31, 2019
stennie added a commit to stennie/mtools that referenced this issue Dec 31, 2019
@stennie
Fix rueckstiess#765: mloginfo --clients: more robust parsing of clien…
ab4d5dc 
…t metadata
@stennie stennie closed this as completed in 6ce1b6d Jan 3, 2020
bugre added a commit to bugre/mtools that referenced this issue Jan 5, 2020
@bugre
Merge branch 'develop' of github.com:bugre/mtools into fix/bugre-#764…
9abc89d 
…_JSONSerialize_crash_w_bytes

* 'develop' of github.com:bugre/mtools:
  Add VSCode config to .gitignore
  Bump dev version
  Update for 1.6.1 release
  Fix Flake8 complaints
  Fix rueckstiess#765: mloginfo --clients: more robust parsing of client metadata (rueckstiess#766)
  Fix rueckstiess#761: mtools should use python3 in shebangs
  Fix rueckstiess#698: Add rounding option for mloginfo --queries (rueckstiess#758)
  More specific match for checkpoint duration log line
  Fix rueckstiess#258: Add timezone to mloginfo summary
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stennie stennie

Labels
improvement mloginfo
Projects
None yet
Milestone
1.6.1
Development

No branches or pull requests
1 participant
@stennie