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.

Sign up for GitHub

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

Jump to bottom

Add protocol version check for outgoing DATA messages #97

Merged
merged 52 commits into from
Aug 24, 2023
Merged

Add protocol version check for outgoing DATA messages #97

merged 52 commits into from
Aug 24, 2023

Conversation

sfod
Copy link
Contributor

@sfod sfod commented Aug 15, 2023

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

sbSteveK
sbSteveK reviewed Aug 15, 2023
View reviewed changes
source/secure_tunneling.c Outdated Show resolved Hide resolved
@sfod sfod force-pushed the protocol-mismatch-handling branch from e986d97 to 8f07b40 Compare August 15, 2023 22:53
@sfod sfod marked this pull request as draft August 17, 2023 00:05
sbSteveK
sbSteveK reviewed Aug 21, 2023
View reviewed changes
source/secure_tunneling.c Outdated Show resolved Hide resolved
sbSteveK
sbSteveK reviewed Aug 21, 2023
View reviewed changes
source/secure_tunneling.c Outdated Show resolved Hide resolved
sbSteveK
sbSteveK reviewed Aug 21, 2023
View reviewed changes
source/secure_tunneling.c Outdated Show resolved Hide resolved
sbSteveK
sbSteveK reviewed Aug 21, 2023
View reviewed changes
source/secure_tunneling.c Outdated Show resolved Hide resolved
sbSteveK
sbSteveK reviewed Aug 21, 2023
View reviewed changes
source/secure_tunneling.c Outdated Show resolved Hide resolved
sfodagain added 25 commits August 22, 2023 17:17
@sfodagain
Remove todo
e6e79f8
@sfodagain
Fix logging format
1ed9483
@sfodagain
Rename goto label
388ff13
@sfodagain
Remove unused include
04c463a
@sfodagain
Remove stream start test
59b863f
@sfodagain
Fix tests naming
662e32e
@sfodagain
Receive messages from mocked server in the event loop
fb2c9f5
@sfodagain
Add missing static
7757a8e
@sfodagain
Fix naming
fc70e82
@sfodagain
Remove unnecessary unused-casts
3cdf140
@sfodagain
Remove unused function
dbbdbf2
@sfodagain
Use vtable for fixing connection id in outbound message
5eea789
@sfodagain
Add doxygen comments
1c1d97e
@sfodagain
Move STREAM_START validation to event loop
5a92cd9
@sfodagain
Add test for STREM_START in SOURCE mode
1e7af76
@sfodagain
Revert "Move STREAM_START validation to event loop"
3b017c0 
This reverts commit 5a92cd9.
@sfodagain
Fix stream_start test
9a77ade
@sfodagain
Add CONNECTION_START test for source mode
05878fa
@sfodagain
Revert "Revert "Move STREAM_START validation to event loop""
2cbe3e4 
This reverts commit 3b017c0.
@sfodagain
Revert "Move STREAM_START validation to event loop"
85da045 
This reverts commit 5a92cd9.
@sfodagain
Add test for outbound STREAM_START message causing reset
0ab2997
@sfodagain
Revert "Revert "Move STREAM_START validation to event loop""
5d1d183 
This reverts commit 85da045.
@sfodagain
Move validation of CONNECTION_START to event loop
5521fd1
@sfodagain
Disable failing test
254293a
@sfodagain
Fix stream_start test with reset
78bc7fa
@sfod sfod marked this pull request as ready for review August 24, 2023 20:17
@sfod sfod merged commit 6acf306 into main Aug 24, 2023
@sfod sfod deleted the protocol-mismatch-handling branch August 24, 2023 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sbSteveK sbSteveK sbSteveK approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sfod @sbSteveK @sfodagain