airflow icon indicating copy to clipboard operation
airflow copied to clipboard

Add Support for GitHub App Installation Authentication in `GithubHook`

Open nizimar opened this issue 1 year ago • 5 comments

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

nizimar avatar May 07 '24 09:05 nizimar

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.

boring-cyborg[bot] avatar May 07 '24 09:05 boring-cyborg[bot]

@kesugano will you open a PR for this feature?

eladkal avatar Jun 17 '24 07:06 eladkal

@eladkal I am submitting my ICLA to the Apache Software Foundation. I will open a PR once it is accepted.

nizimar avatar Jun 25 '24 05:06 nizimar

@kesugano Is there any update regarding this?

Ethelom avatar Oct 08 '24 14:10 Ethelom

Can I work on this issue? Thank you!

anandkaranubc avatar Jun 16 '25 19:06 anandkaranubc

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!

jeff3071 avatar Aug 10 '25 05:08 jeff3071

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.

potiuk avatar Aug 10 '25 12:08 potiuk

@jeff3071 any progress with the PR? Would be great to get Github App support

liam60 avatar Nov 19 '25 05:11 liam60

@liam60 Sorry for late response. I am working on it now. Will update PR soon.

jeff3071 avatar Nov 19 '25 12:11 jeff3071