Skip to content

[b/549538131] - Enhancement for ONFIDO parser to support JSON formatted logs - #1188

Open
dongari-uday wants to merge 1 commit into
mainfrom
onfido_new_changes
Open

[b/549538131] - Enhancement for ONFIDO parser to support JSON formatted logs#1188
dongari-uday wants to merge 1 commit into
mainfrom
onfido_new_changes

Conversation

@dongari-uday

@dongari-uday dongari-uday commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Description

This PR enhances the ONFIDO parser to support and parse JSON formatted logs


Checklist:

Please ensure you have completed the following items before submitting your PR.
This helps us review your contribution faster and more efficiently.

General Checks:

  • I have read and followed the project's contributing.md guide.
  • My code follows the project's coding style guidelines.
  • I have performed a self-review of my own code.
  • My changes do not introduce any new warnings.
  • My changes pass all existing tests.
  • I have added new tests where appropriate to cover my changes. (If applicable)
  • I have updated the documentation where necessary (e.g., README, API docs). (If applicable)

Open-Source Specific Checks:

  • My changes do not introduce any Personally Identifiable Information (PII) or sensitive customer data.
  • My changes do not expose any internal-only code examples, configurations, or URLs.
  • All code examples, comments, and messages are generic and suitable for a public repository.
  • I understand that any internal context or sensitive details related to this work are handled separately in internal systems (Buganizer for Google team members).

For Google Team Members and Reviewers Only:

  • I have included the Buganizer ID in the PR title or description (e.g., "Internal Buganizer ID: 123456789" or "Related Buganizer: go/buganizer/123456789").
  • I have ensured that all internal discussions and PII related to this work remain in Buganizer.
  • I have tagged the PR with one or more labels that reflect the pull request purpose.

Screenshots (If Applicable)

If your changes involve UI or visual elements, please include screenshots or GIFs here.
Ensure any sensitive data is redacted or generalized.


Further Comments / Questions

Any additional comments, questions, or areas where you'd like specific feedback.
SKIP_GOOGLE_PARSER_CHECK="Not Applicable for now"

@dongari-uday
dongari-uday requested a review from a team as a code owner August 28, 2026 11:49
@dongari-uday dongari-uday added Parser Parser Validations parser-migration To be used for all piper logtype migrations to github labels Aug 28, 2026
@dongari-uday dongari-uday removed Parser Parser Validations parser-migration To be used for all piper logtype migrations to github labels Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants