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

Add OData-Version header to requests #8288

Merged
merged 1 commit into from
Nov 6, 2020

Conversation

smizach
Copy link
Contributor

@smizach smizach commented Oct 19, 2020

When using the redfish plugin against an HPE iLO4 host it fails parsing the responses. This is due to a backwards compatibility in iLO4 to before the standardization of field naming in embedded links. Setting the optional OData-Version on request headers signals the server to reply in a redfish 1.0 conformant format (https://hewlettpackard.github.io/ilo-rest-api-docs/ilo4/#redfish-1-0-conformance). This will not change the responses from already compliant hosts (tested against iDRAC)

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

Fixes #8093

@timhallinflux timhallinflux requested a review from ssoroka November 6, 2020 02:15
@ssoroka ssoroka merged commit d369003 into influxdata:master Nov 6, 2020
ssoroka pushed a commit that referenced this pull request Nov 13, 2020
@smizach smizach deleted the redfish-header-fix branch November 24, 2020 22:18
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
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 plugin error parsing input: invalid character ‘<’
2 participants