Skip to content
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

Missing documentation for ProviderAppAuth input #348

Closed
Hanskun opened this issue May 4, 2023 · 4 comments
Closed

Missing documentation for ProviderAppAuth input #348

Hanskun opened this issue May 4, 2023 · 4 comments
Assignees
Labels
area/docs Improvements or additions to documentation kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed

Comments

@Hanskun
Copy link

Hanskun commented May 4, 2023

File: themes/default/content/registry/packages/github/api-docs/provider/_index.md

I have a attempted to use the GitHub provider along wiht a GitHub App I have created, but there is no documentation to the input fields to configure this. For the pemFile argument I am trying to provide a path to the pemFile, but it is giving me the error error: no decodeable PEM data found

So I think that I need to provide the PEM data directly in the input field, but there is no information regarding whether this should be base64 encoded or just as a plaintext string. Would appreciate if there was some additional documentation here to avoid the trial and error approach to have this work.

@susanev susanev transferred this issue from pulumi/registry May 5, 2023
@susanev susanev added area/docs Improvements or additions to documentation needs-triage Needs attention from the triage team labels May 5, 2023
@guineveresaenger
Copy link
Contributor

guineveresaenger commented May 8, 2023

Hi @Hanskun - thank you for filing this issue! This is an instance of pulumi/pulumi-terraform-bridge#1045 - we'll take a look as soon as we can.

@guineveresaenger guineveresaenger removed the needs-triage Needs attention from the triage team label May 8, 2023
@mnlumi mnlumi added the kind/bug Some behavior is incorrect or out of spec label Jul 21, 2023
@mnlumi
Copy link

mnlumi commented Jul 21, 2023

The documentation is still an issue. The current docs show blank spaces here: Screenshot 2023-07-20 at 11 29 36 PM

@nikaro
Copy link

nikaro commented Sep 13, 2023

Is there a way to pass the app_auth credentials into config without overriding the default provider?

I tried this without success:

config:
  github:app_auth:
    id:
      secure: ****
    installation_id:
      secure: ****
    pem_file:
      secure: ****

Edit: it looks like it works with this (camelCase instead of snake_case):

config:
  github:appAuth:
    id:
      secure: ****
    installationId:
      secure: ****
    pemFile:
      secure: ****

@guineveresaenger guineveresaenger self-assigned this Jan 31, 2024
@guineveresaenger guineveresaenger added the resolution/fixed This issue was fixed label Jan 31, 2024
@guineveresaenger
Copy link
Contributor

This should be fixed via pulumi/pulumi-terraform-bridge#1650, and you should be seeing docs updates on the next release of this provider.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs Improvements or additions to documentation kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed
Projects
None yet
Development

No branches or pull requests

5 participants