maskit commented on PR #12938:
URL: https://github.com/apache/trafficserver/pull/12938#issuecomment-4040594747

   > Just please keep it consistent with ja3_fingerprint as well.
   
   Ah, the design came from the ja3 version.... Although the difference from 
ja3 is noted on README, logging is disabled by default on ja3. The behavior is 
already inconsistent.
   
   And the ja3 version does not have ways to specify filename and header names. 
There isn't a way to disable adding the "via" header either.
   
   It doesn't seem like the design is good to me. Should we stick to it? Or 
should the ja3 version be updated (with incompatible changes) as well? This 
makes me want to add another ja4 plugin for people who don't use ja3.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to