-
Notifications
You must be signed in to change notification settings - Fork 51
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
Default flow results in error for missing installationId #578
Comments
👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labled with |
It's const { slug } = await appOctokit.request("GET /app");
console.log("authenticated as %s", slug); |
Right, but the README says to use The implication is that an app could make requests like, "list users" and things like that without having an installation. However, this is impossible. |
Cheers! I'll get that fixed |
🎉 This issue has been resolved in version 6.0.4 🎉 The release is available on: Your semantic-release bot 📦🚀 |
What happened?
The README.md gives the following code example for authenticating as an app.
If one tries to use this code (with valid values for the private key, client secret, etc), the following client-side error occurs:
installationId option is required for installation authentication
This means the library is unuseable without an app installation. If this is desired and correct behavior, then please adjust the README examples.
Versions
Octokit 3.1.2
Node v18.17.0
Relevant log output
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: