Skip to content

Explanation of fork in README #179

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

Closed
petterroea opened this issue May 9, 2024 · 3 comments
Closed

Explanation of fork in README #179

petterroea opened this issue May 9, 2024 · 3 comments

Comments

@petterroea
Copy link

It would be nice to have a proper description of why this fork exists in README so stakeholders and reviewers can better understand why their code switches dependency to kafka-python-ng.

This makes code reviews of the switch easier, and also helps with the worries some have due to the increase in supply chain attacks in recent years.

@wbarnha
Copy link
Collaborator

wbarnha commented May 14, 2024

I agree... there have been a lot of supply chain attacks and I've been the target of many phishing attempts, hence why I hid my personal details from my GitHub page (I assume that a web scraper is collecting emails targeting maintainers of projects with more than a hundred or so stars).

Ever since I hid my email, the emails have stopped for the most part.

@bfontaine
Copy link

bfontaine commented Jul 16, 2024

Gentle ping on this :)

@wbarnha
Copy link
Collaborator

wbarnha commented Jul 17, 2024

Sorry about that. My schedule for contributing has been out of sorts. I was appointed by Dana to have Push permissions to kafka-python, and as described in https://github.com/dpkp/kafka-python/issues/2431#issuecomment-2195874401, Dana has been away attending to personal affairs. Since I don't have PyPi credentials for that project, the most rational decision appeared to be fork and just redirect users here in the meantime.

This project is a pure Python distribution in wheels, so there isn't any malicious code hidden in compiled wheels because they aren't compiled to begin with and I also use trusted publishing to PyPi to demonstrate that these files came directly from GitHub. I'll add this note to the readme.

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

No branches or pull requests

3 participants