Skip to content

chore: use client-id for create-github-app-token - #175

Merged
marandaneto merged 1 commit into
mainfrom
chore/use-client-id-for-github-app-token
Apr 16, 2026
Merged

chore: use client-id for create-github-app-token#175
marandaneto merged 1 commit into
mainfrom
chore/use-client-id-for-github-app-token

Conversation

@marandaneto

Copy link
Copy Markdown
Member

💡 Motivation and Context

GitHub Actions now warns that app-id is deprecated in actions/create-github-app-token.
This updates the release workflow to use client-id and bumps the action to v3 where needed.

💚 How did you test it?

  • ran git diff --check

📝 Checklist

  • I reviewed the submitted code.
  • I added tests to verify the changes.
  • I updated the docs if needed.
  • No breaking change or entry added to the changelog.

If releasing new changes

  • Added the release label to the PR
  • Added exactly one version bump label: bump-patch, bump-minor, or bump-major

@marandaneto
marandaneto requested review from a team and haacked as code owners April 16, 2026 15:31
@github-actions

Copy link
Copy Markdown
Contributor

posthog-dotnet Compliance Report

Date: 2026-04-16 15:32:20 UTC
Duration: 266ms

⚠️ Some Tests Failed

0/1 tests passed, 1 failed


Feature_Flags Tests

⚠️ 0/1 tests passed, 1 failed

View Details
Test Status Duration
Request Payload.Request With Person Properties Device Id 44ms

Failures

request_payload.request_with_person_properties_device_id

404, message='Not Found', url='http://sdk-adapter:8080/get_feature_flag'

@marandaneto
marandaneto enabled auto-merge (squash) April 16, 2026 15:32
@greptile-apps

greptile-apps Bot commented Apr 16, 2026

Copy link
Copy Markdown
Contributor
Prompt To Fix All With AI
This is a comment left during a code review.
Path: .github/workflows/release.yml
Line: 84

Comment:
**Secret value may need to be updated alongside the parameter rename**

A GitHub App has two distinct identifiers: the numeric **App ID** (e.g. `12345`) and the OAuth **Client ID** (e.g. `Iv1.xxxxxxxxxxxxxxxx`). The secret `GH_APP_POSTHOG_DOTNET_RELEASER_APP_ID` is named for the App ID, suggesting it stores the numeric value. The `client-id` input now expects the Client ID string — a different value found in the GitHub App's settings page under "Client ID."

If the secret still holds the numeric App ID, the action may reject it or produce a hard-to-diagnose auth failure on the next release run. Please confirm the secret value is the Client ID (the `Iv1.…` string), and update the secret if it currently stores the numeric App ID.

How can I resolve this? If you propose a fix, please make it concise.

Reviews (1): Last reviewed commit: "chore: use client-id for create-github-a..." | Re-trigger Greptile

uses: actions/create-github-app-token@v3
with:
app-id: ${{ secrets.GH_APP_POSTHOG_DOTNET_RELEASER_APP_ID }}
client-id: ${{ secrets.GH_APP_POSTHOG_DOTNET_RELEASER_APP_ID }}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 Secret value may need to be updated alongside the parameter rename

A GitHub App has two distinct identifiers: the numeric App ID (e.g. 12345) and the OAuth Client ID (e.g. Iv1.xxxxxxxxxxxxxxxx). The secret GH_APP_POSTHOG_DOTNET_RELEASER_APP_ID is named for the App ID, suggesting it stores the numeric value. The client-id input now expects the Client ID string — a different value found in the GitHub App's settings page under "Client ID."

If the secret still holds the numeric App ID, the action may reject it or produce a hard-to-diagnose auth failure on the next release run. Please confirm the secret value is the Client ID (the Iv1.… string), and update the secret if it currently stores the numeric App ID.

Prompt To Fix With AI
This is a comment left during a code review.
Path: .github/workflows/release.yml
Line: 84

Comment:
**Secret value may need to be updated alongside the parameter rename**

A GitHub App has two distinct identifiers: the numeric **App ID** (e.g. `12345`) and the OAuth **Client ID** (e.g. `Iv1.xxxxxxxxxxxxxxxx`). The secret `GH_APP_POSTHOG_DOTNET_RELEASER_APP_ID` is named for the App ID, suggesting it stores the numeric value. The `client-id` input now expects the Client ID string — a different value found in the GitHub App's settings page under "Client ID."

If the secret still holds the numeric App ID, the action may reject it or produce a hard-to-diagnose auth failure on the next release run. Please confirm the secret value is the Client ID (the `Iv1.…` string), and update the secret if it currently stores the numeric App ID.

How can I resolve this? If you propose a fix, please make it concise.

@marandaneto
marandaneto merged commit f587ddc into main Apr 16, 2026
15 checks passed
@marandaneto
marandaneto deleted the chore/use-client-id-for-github-app-token branch April 16, 2026 15:35
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.

2 participants