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

imapd: track characters used in IMAP tags #5055

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

elliefm
Copy link
Contributor

@elliefm elliefm commented Sep 25, 2024

This extends the existing IMAP client behaviour tracking to record categories of characters used in tags

We don't have existing tests for the client behaviour logging, except for a couple of cursory checks in Simple and ID. I might feel more comfortable with some tests that verify the character class mappings detect as expected, but it'd be a whole new test suite, is it worth the effort? I dunno.

rsto
rsto previously approved these changes Sep 25, 2024
ksmurchison
ksmurchison previously approved these changes Sep 25, 2024
@elliefm
Copy link
Contributor Author

elliefm commented Sep 30, 2024

We think we don't want to actually merge this, just include it in our builds until we've got the data we're interested in, then drop it.

@rsto
Copy link
Member

rsto commented Nov 3, 2024

@elliefm I removed the include-in-fastmail tag from this PR because it conflicts with the main branch.

@elliefm elliefm force-pushed the v311/client-imaptags-behaviour branch from 5e9e139 to c7e3929 Compare November 3, 2024 23:29
@elliefm
Copy link
Contributor Author

elliefm commented Nov 3, 2024

@rsto I've fixed the conflicts, so we could re-tag it now. But actually, I think I'll take the opportunity to change the data it collects based on what we saw from the first run. So let's not re-tag it just yet, wait till I've got it ready for the next lot of stats.

@elliefm elliefm dismissed stale reviews from ksmurchison and rsto November 7, 2024 22:06

stale

@elliefm elliefm force-pushed the v311/client-imaptags-behaviour branch from 5c9fab8 to 2ba2745 Compare November 12, 2024 22:21
@elliefm elliefm requested review from ksmurchison and rsto November 12, 2024 22:21
@elliefm
Copy link
Contributor Author

elliefm commented Nov 12, 2024

@ksmurchison @rsto I've updated this to log unusual tags at LOG_INFO rather than LOG_DEBUG, and improve the tracking and reporting based on shortcomings I noticed in the original stats. The last 3 commits are the new ones.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants