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

Added missing new line in M-SEARCH request #143

Merged
merged 2 commits into from
Jul 27, 2023
Merged

Added missing new line in M-SEARCH request #143

merged 2 commits into from
Jul 27, 2023

Conversation

mraineri
Copy link
Contributor

Fix #142

@PeterKchen2 would you be able to test this on your system for me? I don't have access to an AMI system and other systems are able to handle the missing newline in the request.

mraineri added 2 commits July 26, 2023 11:48
Signed-off-by: Mike Raineri <michael.raineri@dell.com>
Signed-off-by: Mike Raineri <michael.raineri@dell.com>
@mraineri
Copy link
Contributor Author

Also needed another fix with the regex when inspecting the USN header.

@PeterKchen2
Copy link
Contributor

OK

@PeterKchen2
Copy link
Contributor

Dear mraineri

it work

Redfish services:
66ecf5de-bfde-11d3-01ea-819556480d00: https://192.168.199.7/redfish/v1/ (AMI Redfish Server)

@mraineri
Copy link
Contributor Author

Awesome, thanks for testing.

Merging out of cycle to push out a bug fix.

@mraineri mraineri merged commit 9b56398 into main Jul 27, 2023
@mraineri mraineri deleted the Fix142-SSDP-Fix branch July 27, 2023 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rf_discover.py can not found AMI redfish server service
2 participants