Skip to content

Zalo Chat and Users Parser#1511

Merged
JamesHabben merged 3 commits into
abrignoni:mainfrom
prosch88:main
Jun 2, 2026
Merged

Zalo Chat and Users Parser#1511
JamesHabben merged 3 commits into
abrignoni:mainfrom
prosch88:main

Conversation

@prosch88
Copy link
Copy Markdown
Contributor

@prosch88 prosch88 commented Jun 1, 2026

This adds a parser for the app Zalo (most commonly used in Vietnam or by people with ties to Vietnam).

parses:

  • Chats
  • Groupchats
  • known Users
image

Some newer versions of the app use JPEG XL. To view them, you'll need a modern browser (currently mostly still in beta) or the latest version of LAVA.

@prosch88
Copy link
Copy Markdown
Contributor Author

prosch88 commented Jun 1, 2026

Thanks to @BrunoFischerGermany for the test data sets!

@JamesHabben
Copy link
Copy Markdown
Collaborator

2 small comments

  1. if you use the newer context processor preamble in the functions you dont have to underscore the unused parameters. then use context.get_files_found() to replace the files_found parameter of the older processor function preamble.
  2. I see a mobile column header. is that possible a phonenumber type of data?

@prosch88
Copy link
Copy Markdown
Contributor Author

prosch88 commented Jun 1, 2026

You're right. It is indeed a phone number. I didn't notice the switch to "context." It's definitely easier to read.

@JamesHabben
Copy link
Copy Markdown
Collaborator

The context parameter also makes the modules more resilient to core code updates since changing the preamble of the processor function requires updates to every module. We can update context with new features now and not trigger a massive module update.

@prosch88
Copy link
Copy Markdown
Contributor Author

prosch88 commented Jun 2, 2026

I had changed it. It still works with my test data. So it should be fine, right?

@JamesHabben JamesHabben merged commit 796ff4b into abrignoni:main Jun 2, 2026
1 check passed
@JamesHabben
Copy link
Copy Markdown
Collaborator

yes. take another look through and it looks good to me so merged it in base on you saying your tests still work. thanks for contributing!

@JamesHabben JamesHabben added the Artifact Module Update related to an artifact module label Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Artifact Module Update related to an artifact module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants