Skip to content

Allow invalid certs on BMCs. libredfish default changed to deny them by default#824

Draft
wminckler wants to merge 1 commit intoNVIDIA:mainfrom
wminckler:bmc-cert
Draft

Allow invalid certs on BMCs. libredfish default changed to deny them by default#824
wminckler wants to merge 1 commit intoNVIDIA:mainfrom
wminckler:bmc-cert

Conversation

@wminckler
Copy link
Copy Markdown
Contributor

Description

libredfish changed to deny invalid certs by default. this re-enables accepting invalid certs for BMCs

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Related Issues (Optional)

nvbug 6025251

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

Requres merge of NVIDIA/libredfish#56

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Apr 6, 2026

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

[workspace.dependencies]
clap = { version = "4", features = ["derive", "env"] }
libredfish = { git = "https://github.com/NVIDIA/libredfish.git", tag = "v0.43.5" }
libredfish = { path = "../libredfish" }
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Local path.

Copy link
Copy Markdown
Contributor Author

@wminckler wminckler Apr 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes. I can't push the libredfish change until security reviews it, but they want to see changes

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.

2 participants