Page MenuHomePhorge

Review branch dev/mollekopf2 Avoid unnecessary explicit cast
ClosedPublic

Authored by mollekopf on Apr 8 2020, 4:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 16, 4:46 AM
Unknown Object (File)
Tue, Apr 9, 5:10 AM
Unknown Object (File)
Mar 2 2024, 4:28 PM
Unknown Object (File)
Feb 15 2024, 11:19 PM
Unknown Object (File)
Feb 13 2024, 11:57 AM
Unknown Object (File)
Feb 10 2024, 7:33 AM
Unknown Object (File)
Feb 7 2024, 11:43 AM
Unknown Object (File)
Jan 19 2024, 8:42 PM
Subscribers
None

Details

Reviewers
None
Group Reviewers
Restricted Project
Commits
rGa3c5d59f8fc4: Removed the binary:matches(PreprocessData, <<"\r\n">>) workaround
Summary

The extra case is not necessary to deal with empty lines

Moved record to separate file so it can be reused from tests

kolab_guam_session_SUITE

Multi-line packet processing is still failing.

Split out rule application into a seprate function.

Perhaps not the right split and certainly not the right name, but
provides a bit more overview.

Process client-data line by line to deal multiple commands arriving in the same packet.

E.g:

y1 ID (\"name\" \"Trojita\" \"os\" \"Qt/5.13.2; xcb; Linux; \" \"version\" \"0.7-git\")\r\ny2 ENABLE QRESYNC

Check current_command_split in tests

Full protocol dumps in debug output

test cleanup

Test more package splits

Honor reset_on_next_client_command

We used to rely on getting something from the server inbetween (which is
not a valid assumption).

Test various package splits

Removed the binary:matches(PreprocessData, <<"\r\n">>) workaround

This in turn also makes the applyRules function unnecessary.

The workaround was handling a special case when \r\n was prepended in
the packet (that was the only way to hit the codepath at all). Now that
we process line by line we can't run into this situation anymore.

Diff Detail

Repository
rG guam
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mollekopf created this revision.
This revision was not accepted when it landed; it landed in state Needs Review.Apr 15 2020, 9:20 AM
This revision was automatically updated to reflect the committed changes.