Skip to content

fix(redfish): fixed failures on redfish protocol validator#823

Draft
DevipriyaS17 wants to merge 1 commit intoredfishfrom
redfish_protocol_fix
Draft

fix(redfish): fixed failures on redfish protocol validator#823
DevipriyaS17 wants to merge 1 commit intoredfishfrom
redfish_protocol_fix

Conversation

@DevipriyaS17
Copy link
Contributor

@DevipriyaS17 DevipriyaS17 commented Mar 5, 2026

  • Added /redfish version endpoint
  • Fixed OData service context and header validation
  • Enhanced response headers (Link, Allow, WWW-Authenticate)
  • Removed single-session-per-user restriction
  • Added POST to /SessionService/Sessions/Members

Fixed 17 of 20 DMTF Protocol Validator failures.

Results
FAIL: 20 → 3

@DevipriyaS17 DevipriyaS17 linked an issue Mar 5, 2026 that may be closed by this pull request
@DevipriyaS17 DevipriyaS17 force-pushed the redfish_protocol_fix branch 2 times, most recently from 1600a7d to 7270d18 Compare March 5, 2026 08:52
@codecov
Copy link

codecov bot commented Mar 5, 2026

Codecov Report

❌ Patch coverage is 62.50000% with 18 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (redfish@5ca1b5d). Learn more about missing BASE report.

Files with missing lines Patch % Lines
redfish/component.go 0.00% 9 Missing ⚠️
...fish/internal/controller/http/v1/handler/errors.go 84.37% 5 Missing ⚠️
...nternal/controller/http/v1/handler/service_root.go 20.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             redfish     #823   +/-   ##
==========================================
  Coverage           ?   34.31%           
==========================================
  Files              ?      135           
  Lines              ?    14163           
  Branches           ?        0           
==========================================
  Hits               ?     4860           
  Misses             ?     8868           
  Partials           ?      435           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

[Redfish] Resolve issues in the Redfish protocol validator tests

1 participant