Skip to content

Commit

Permalink
Fix github auth app issues (#49)
Browse files Browse the repository at this point in the history
  • Loading branch information
AndrewKahr authored Jun 5, 2024
1 parent bd3002f commit df93ec5
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 4 deletions.
8 changes: 7 additions & 1 deletion .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,12 @@ jobs:
- name: Deploy to Firebase
uses: w9jds/firebase-action@v13.10.2
with:
args: deploy --except functions:testFunction
args: deploy
env:
GCP_SA_KEY: '${{ secrets.FIREBASE_SERVICE_ACCOUNT_UNITY_CI_VERSIONS }}'
- name: Cleanup Firebase Test
uses: w9jds/firebase-action@v13.10.2
with:
args: functions:delete testFunction --force
env:
GCP_SA_KEY: '${{ secrets.FIREBASE_SERVICE_ACCOUNT_UNITY_CI_VERSIONS }}'
3 changes: 1 addition & 2 deletions functions/src/config/settings.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,7 @@ export const settings = {
},
github: {
auth: {
id: 84327,
installationId: 12321333,
appId: 84327,
clientId: 'Iv1.fa93dce6a47c9357',
},
},
Expand Down
2 changes: 1 addition & 1 deletion functions/src/service/github.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import { settings } from '../config/settings';
import { logger } from 'firebase-functions/v2';

export class GitHub {
// https://octokit.github.io/rest.js/v18
// https://octokit.github.io/rest.js/v20
static async init(privateKey: string, clientSecret: string): Promise<Octokit> {
const appOctokit = new Octokit({
authStrategy: createAppAuth,
Expand Down

0 comments on commit df93ec5

Please sign in to comment.