Skip to content

Conversation

@madhuriupadhye
Copy link
Contributor

No description provided.

Signed-off-by: Madhuri Upadhye <mupadhye@redhat.com>
@madhuriupadhye madhuriupadhye marked this pull request as draft January 12, 2026 13:28
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request modifies the project's dependencies to use a personal fork of sssd-test-framework. This introduces a critical security and maintainability risk by sourcing code from an unofficial repository. My review strongly advises against this change and recommends reverting to the official dependency to maintain the project's integrity.

Comment on lines +8 to +9
#git+https://github.com/SSSD/sssd-test-framework
git+https://github.com/madhuriupadhye/sssd-test-framework@dns_zone

Choose a reason for hiding this comment

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

security-critical critical

This change replaces the official sssd-test-framework dependency with a version from a personal fork. Using dependencies from personal forks is a major security risk, as they can contain unvetted or malicious code, and are not guaranteed to be maintained. This also creates a maintenance burden, as the fork may diverge from the official project. Please revert to using the official dependency from the SSSD organization.

git+https://github.com/SSSD/sssd-test-framework

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.

1 participant