Skip to content

Commit f7a938b

Browse files
tstirrat15tstirrat15
authored
Merge pull request #19 from authzed/update-api-and-add-api-update-logic
Update api and add api update logic
2 parents c519177 + fba6ca7 commit f7a938b

21 files changed

Lines changed: 2621 additions & 658 deletions

‎.github/workflows/automatic-api-update.yaml‎

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,41 @@
1+
name: "Called update for API change"
2+
on:
3+
repository_dispatch:
4+
types: ["api_update"]
5+
jobs:
6+
test:
7+
name: "Create PR for API update"
8+
timeout-minutes: 10
9+
runs-on: ubuntu-latest
10+
steps:
11+
- uses: actions/checkout@v4
12+
- uses: actions/setup-node@v4
13+
- name: "Update Buf Script"
14+
id: buf-update
15+
uses: authzed/actions/buf-api-update@main
16+
with:
17+
api-commit: "${{ github.event.client_payload.BUFTAG }}"
18+
spec-path: buf.gen.yaml
19+
file-format: "buf-gen-yaml"
20+
- name: "Output update status"
21+
env:
22+
UPDATED_STATUS: ${{ steps.buf-update.outputs.updated }}
23+
run: |
24+
echo "Update status: $UPDATED_STATUS"
25+
- name: "Install buf"
26+
uses: "bufbuild/buf-setup-action@v1.43.0"
27+
with:
28+
github_token: ${{ secrets.GITHUB_TOKEN }}
29+
if: steps.buf-update.outputs.updated == 'true'
30+
- name: "Run buf generate"
31+
if: steps.buf-update.outputs.updated == 'true'
32+
run: "buf generate"
33+
- name: Create Pull Request
34+
uses: peter-evans/create-pull-request@v7.0.5
35+
if: steps.buf-update.outputs.updated == 'true'
36+
with:
37+
delete-branch: "true"
38+
title: "Update API to ${{ github.event.client_payload.BUFTAG }}"
39+
branch: "api-change/${{ github.event.client_payload.BUFTAG }}"
40+
base: "main"
41+
token: ${{ secrets.GITHUB_TOKEN }}

‎.github/workflows/manual-api-update.yaml‎

Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
---
2+
name: Update for API change
3+
on:
4+
workflow_dispatch:
5+
inputs:
6+
buftag:
7+
description: Tag or commit from https://buf.build/authzed/api/tags/main
8+
required: true
9+
type: string
10+
jobs:
11+
test:
12+
name: "Create PR for API update"
13+
timeout-minutes: 10
14+
runs-on: ubuntu-latest
15+
steps:
16+
- uses: actions/checkout@v4
17+
- uses: actions/setup-node@v4
18+
- name: "Update Buf Script"
19+
id: buf-update
20+
uses: authzed/actions/buf-api-update@main
21+
with:
22+
api-commit: ${{ inputs.buftag }}
23+
spec-path: buf.gen.yaml
24+
file-format: "buf-gen-yaml"
25+
- name: "Output update status"
26+
env:
27+
UPDATED_STATUS: ${{ steps.buf-update.outputs.updated }}
28+
run: |
29+
echo "Update status: $UPDATED_STATUS"
30+
- name: "Install buf"
31+
uses: "bufbuild/buf-setup-action@v1.43.0"
32+
with:
33+
github_token: ${{ secrets.GITHUB_TOKEN }}
34+
if: steps.buf-update.outputs.updated == 'true'
35+
- name: "Run buf generate"
36+
if: steps.buf-update.outputs.updated == 'true'
37+
run: "buf generate"
38+
- name: Create Pull Request
39+
uses: peter-evans/create-pull-request@v7.0.5
40+
if: steps.buf-update.outputs.updated == 'true'
41+
with:
42+
delete-branch: "true"
43+
title: Update API to ${{ inputs.buftag }}
44+
branch: api-change/${{ inputs.buftag }}
45+
base: "main"
46+
token: ${{ secrets.GITHUB_TOKEN }}

‎buf.gen.yaml‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,18 +2,14 @@
22
# yamllint disable rule:line-length
33
version: "v2"
44
plugins:
5-
- protoc_builtin: "csharp"
5+
- remote: "buf.build/protocolbuffers/csharp:v28.3"
66
out: "src/Authzed.Net"
77
opt: "base_namespace="
88
include_imports: true
9-
# TODO: this is arch-specific and may be host-specific as well (in terms of where .nuget is)
10-
# NOTE: part of the reason this is a pain is that the csharp ecosystem expects you to use
11-
# the dotnet toolchain from top-to-bottom to compile protoc, so feeding things through buf
12-
# and using a local plugin isn't in their use case.
13-
- local: "/home/tstirrat/.nuget/packages/grpc.tools/2.65.0/tools/linux_x64/grpc_csharp_plugin"
9+
- remote: "buf.build/grpc/csharp:v1.67.1"
1410
out: "src/Authzed.Net"
1511
opt:
1612
- "base_namespace="
1713
- "no_server=on"
1814
inputs:
19-
- module: buf.build/authzed/api:v1.37.0
15+
- module: buf.build/authzed/api:v1.38.0

‎src/Authzed.Net/Authzed/Api/Materialize/V0/Watchpermissions.cs‎

Lines changed: 44 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ static WatchpermissionsReflection() {
6969

7070
}
7171
#region Messages
72+
[global::System.Diagnostics.DebuggerDisplayAttribute("{ToString(),nq}")]
7273
public sealed partial class WatchPermissionsRequest : pb::IMessage<WatchPermissionsRequest>
7374
#if !GOOGLE_PROTOBUF_REFSTRUCT_COMPATIBILITY_MODE
7475
, pb::IBufferMessage
@@ -253,7 +254,11 @@ public void MergeFrom(pb::CodedInputStream input) {
253254
#else
254255
uint tag;
255256
while ((tag = input.ReadTag()) != 0) {
256-
switch(tag) {
257+
if ((tag & 7) == 4) {
258+
// Abort on any end group tag.
259+
return;
260+
}
261+
switch(tag) {
257262
default:
258263
_unknownFields = pb::UnknownFieldSet.MergeFieldFrom(_unknownFields, input);
259264
break;
@@ -279,7 +284,11 @@ public void MergeFrom(pb::CodedInputStream input) {
279284
void pb::IBufferMessage.InternalMergeFrom(ref pb::ParseContext input) {
280285
uint tag;
281286
while ((tag = input.ReadTag()) != 0) {
282-
switch(tag) {
287+
if ((tag & 7) == 4) {
288+
// Abort on any end group tag.
289+
return;
290+
}
291+
switch(tag) {
283292
default:
284293
_unknownFields = pb::UnknownFieldSet.MergeFieldFrom(_unknownFields, ref input);
285294
break;
@@ -301,6 +310,7 @@ public void MergeFrom(pb::CodedInputStream input) {
301310

302311
}
303312

313+
[global::System.Diagnostics.DebuggerDisplayAttribute("{ToString(),nq}")]
304314
public sealed partial class WatchedPermission : pb::IMessage<WatchedPermission>
305315
#if !GOOGLE_PROTOBUF_REFSTRUCT_COMPATIBILITY_MODE
306316
, pb::IBufferMessage
@@ -555,7 +565,11 @@ public void MergeFrom(pb::CodedInputStream input) {
555565
#else
556566
uint tag;
557567
while ((tag = input.ReadTag()) != 0) {
558-
switch(tag) {
568+
if ((tag & 7) == 4) {
569+
// Abort on any end group tag.
570+
return;
571+
}
572+
switch(tag) {
559573
default:
560574
_unknownFields = pb::UnknownFieldSet.MergeFieldFrom(_unknownFields, input);
561575
break;
@@ -586,7 +600,11 @@ public void MergeFrom(pb::CodedInputStream input) {
586600
void pb::IBufferMessage.InternalMergeFrom(ref pb::ParseContext input) {
587601
uint tag;
588602
while ((tag = input.ReadTag()) != 0) {
589-
switch(tag) {
603+
if ((tag & 7) == 4) {
604+
// Abort on any end group tag.
605+
return;
606+
}
607+
switch(tag) {
590608
default:
591609
_unknownFields = pb::UnknownFieldSet.MergeFieldFrom(_unknownFields, ref input);
592610
break;
@@ -613,6 +631,7 @@ public void MergeFrom(pb::CodedInputStream input) {
613631

614632
}
615633

634+
[global::System.Diagnostics.DebuggerDisplayAttribute("{ToString(),nq}")]
616635
public sealed partial class WatchPermissionsResponse : pb::IMessage<WatchPermissionsResponse>
617636
#if !GOOGLE_PROTOBUF_REFSTRUCT_COMPATIBILITY_MODE
618637
, pb::IBufferMessage
@@ -850,7 +869,11 @@ public void MergeFrom(pb::CodedInputStream input) {
850869
#else
851870
uint tag;
852871
while ((tag = input.ReadTag()) != 0) {
853-
switch(tag) {
872+
if ((tag & 7) == 4) {
873+
// Abort on any end group tag.
874+
return;
875+
}
876+
switch(tag) {
854877
default:
855878
_unknownFields = pb::UnknownFieldSet.MergeFieldFrom(_unknownFields, input);
856879
break;
@@ -883,7 +906,11 @@ public void MergeFrom(pb::CodedInputStream input) {
883906
void pb::IBufferMessage.InternalMergeFrom(ref pb::ParseContext input) {
884907
uint tag;
885908
while ((tag = input.ReadTag()) != 0) {
886-
switch(tag) {
909+
if ((tag & 7) == 4) {
910+
// Abort on any end group tag.
911+
return;
912+
}
913+
switch(tag) {
887914
default:
888915
_unknownFields = pb::UnknownFieldSet.MergeFieldFrom(_unknownFields, ref input);
889916
break;
@@ -912,6 +939,7 @@ public void MergeFrom(pb::CodedInputStream input) {
912939

913940
}
914941

942+
[global::System.Diagnostics.DebuggerDisplayAttribute("{ToString(),nq}")]
915943
public sealed partial class PermissionChange : pb::IMessage<PermissionChange>
916944
#if !GOOGLE_PROTOBUF_REFSTRUCT_COMPATIBILITY_MODE
917945
, pb::IBufferMessage
@@ -1207,7 +1235,11 @@ public void MergeFrom(pb::CodedInputStream input) {
12071235
#else
12081236
uint tag;
12091237
while ((tag = input.ReadTag()) != 0) {
1210-
switch(tag) {
1238+
if ((tag & 7) == 4) {
1239+
// Abort on any end group tag.
1240+
return;
1241+
}
1242+
switch(tag) {
12111243
default:
12121244
_unknownFields = pb::UnknownFieldSet.MergeFieldFrom(_unknownFields, input);
12131245
break;
@@ -1251,7 +1283,11 @@ public void MergeFrom(pb::CodedInputStream input) {
12511283
void pb::IBufferMessage.InternalMergeFrom(ref pb::ParseContext input) {
12521284
uint tag;
12531285
while ((tag = input.ReadTag()) != 0) {
1254-
switch(tag) {
1286+
if ((tag & 7) == 4) {
1287+
// Abort on any end group tag.
1288+
return;
1289+
}
1290+
switch(tag) {
12551291
default:
12561292
_unknownFields = pb::UnknownFieldSet.MergeFieldFrom(_unknownFields, ref input);
12571293
break;

0 commit comments

Comments
 (0)