airflow
airflow copied to clipboard
Add Support for GitHub App Installation Authentication in `GithubHook`
Description
Enhance the GithubHook to support GitHub App installation authentication.
While the hook currently supports only token-based authentication, adding support for GitHub App installation authentication will provide a more secure option for Airflow users interacting with GitHub.
Use case/motivation
GitHub App installation authentication offers a more secure and granular permission model compared to personal access tokens. It allows for better audit trails and security practices, which are crucial for organizations managing complex workflows on GitHub.
By integrating this authentication method into GithubHook, Airflow users can leverage the benefits of GitHub Apps, such as fine-grained permissions, enhanced security, and improved automation capabilities within their data pipelines.
Related issues
No response
Are you willing to submit a PR?
- [X] Yes I am willing to submit a PR!
Code of Conduct
- [X] I agree to follow this project's Code of Conduct
Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.
@kesugano will you open a PR for this feature?
@eladkal I am submitting my ICLA to the Apache Software Foundation. I will open a PR once it is accepted.
@kesugano Is there any update regarding this?
Can I work on this issue? Thank you!
Hi @anandkaranubc , is there any update? I’m interested in contributing and would be happy to help with this feature. Please let me know if it's okay for me to work on it.
Thanks!
Assigned both of you know - I guess if no response is heard soon, feel free to work on it @jeff3071 -> in the end you can merge the solutions and review each other's code and learn from each other if it happens you work on parallel on it.
@jeff3071 any progress with the PR? Would be great to get Github App support
@liam60 Sorry for late response. I am working on it now. Will update PR soon.