Skip to content

Test fixes #127

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
Merged
jboddey merged 2 commits into from
Sep 27, 2023
Merged

Test fixes #127

jboddey merged 2 commits into from
Sep 27, 2023

Conversation

@jhughesoti
Copy link
Contributor

@jhughesoti jhughesoti commented Sep 26, 2023

Fixes the following issues:

TLS Module:

  • TLS client details reported results from the monitor when only startup had valid packets

Connection Module:

  • Different devices reported DHCP Request options in different orders causing inconsistent results.

@jhughesoti
Fix tls details for client test
8f7d902
@jhughesoti
Fix single ip test for different DHCP option order
fc40ac1 
Add non-debug logging to allow for easier troubleshooting if single ip test fails
jboddey
jboddey reviewed Sep 26, 2023
View reviewed changes
@jboddey jboddey merged commit fe4275d into dev Sep 27, 2023
@jboddey jboddey deleted the test_fixes branch October 5, 2023 17:03
hitnik pushed a commit that referenced this pull request Jul 22, 2024
@jhughesoti
Test fixes (#127)
403db46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@noursaidi noursaidi Awaiting requested review from noursaidi

1 more reviewer

@jboddey jboddey jboddey left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jhughesoti @jboddey