Skip to content

Make authenticated requests to the Github API using a Github App#892

Open
magopian wants to merge 2 commits into
mainfrom
417-authenticated-github-api-requests-with-github-app
Open

Make authenticated requests to the Github API using a Github App#892
magopian wants to merge 2 commits into
mainfrom
417-authenticated-github-api-requests-with-github-app

Conversation

@magopian
Copy link
Copy Markdown
Collaborator

@magopian magopian commented May 20, 2026

Fixes #417 , duplicate of #883

Cette PR fais les choses bien en utilisant une "github app" (déjà créée par Julien Dauphant), mais qui ne sera utilisable qu'une fois installée sur notre repository, or je n'en ai pas les droits.

Une fois que ce sera fait et qu'on aura pu tester que ça fonctionne comme prévu, on pourra utiliser cette PR en lieu et place de la PR #883.

/!\ attention ne pas merger avant qu'on ai pu tester et valider que la github App fonctionnait correctement.

@magopian magopian force-pushed the 417-authenticated-github-api-requests-with-github-app branch from 854a8e1 to 3b15ae7 Compare May 20, 2026 12:53
@magopian
Copy link
Copy Markdown
Collaborator Author

magopian commented May 20, 2026

Avant le merge (hypothétique) de cette PR, penser à recopier les variables d'env suivantes sur l'environnement de staging :

  • GITHUB_APP_ID
  • GITHUB_APP_PRIVATE_KEY

Si la PR #883 a été mergée auparavant, supprimer la variable d'env GITHUB_PERSONAL_ACCESS_TOKEN_REVIEW_APPS qui ne sera plus utile.

Copy link
Copy Markdown
Collaborator

@cdesquilbet-dila cdesquilbet-dila left a comment

Choose a reason for hiding this comment

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

Ok pour moi, merci !
De ce que je comprends, la PR883 n'est pas à revoir donc je ne prévois pas de le faire, mais n'hésite pas à me demander de la revoir si j'ai mal compris !

Comment thread ami/utils/tests/test_all.py Outdated
@magopian
Copy link
Copy Markdown
Collaborator Author

Ok pour moi, merci ! De ce que je comprends, la PR883 n'est pas à revoir donc je ne prévois pas de le faire, mais n'hésite pas à me demander de la revoir si j'ai mal compris !

Aujourd'hui, on a pas les droits pour utiliser la github App, cette PR donc.

Si tu peux/veux revoir l'autre, on peut la merger d'ores et déjà, et avoir déjà la possibilité de faire des requêtes authentifiées, qui vont simplifier la vie des testeureuses (et des recetteureuses ;), et quand on aura les droits pour mettre cette PR en place, on pourra basculer.

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.

Rate limit on the github REST API with unauthenticated user

2 participants