.gitea/workflows/build.yaml

@@ -18,17 +18,11 @@ jobs:
       - name: Check out repository code
         uses: actions/checkout@v4
       - name: Setup Zig
-        uses: https://github.com/mlugg/setup-zig@v1.2.1
+        uses: mlugg/setup-zig@v1.2.1
         with:
           version: 0.14.0
-      - name: Restore Zig caches
-        uses: https://github.com/Hanaasagi/zig-action-cache@3954aae427f8b05914e08dfd79f15e1f2e435929
-      - name: Run smoke test
-        run: zig build smoke-test --verbose
       - name: Run tests
-        run: zig build test --verbose --summary all
-      - name: Run tests (release mode)
-        run: zig build test -Doptimize=ReleaseSafe --verbose
+        run: zig build test --verbose
       # Zig build scripts don't have the ability to import depenedencies directly
       # (https://github.com/ziglang/zig/issues/18164). We can allow downstream
       # build scripts to import aws with a few tweaks, but we can't @import("src/aws.zig")
@@ -73,7 +67,7 @@ jobs:
       # - run: echo "URL of foo (1) is ${{ steps.sign.outputs.URL_1 }}"
       - name: Publish source code with generated models
         run: |
-          curl -s --user ${{ github.actor }}:${{ secrets.PACKAGE_PUSH }} \
+          curl --user ${{ github.actor }}:${{ secrets.PACKAGE_PUSH }} \
             --upload-file ${{ runner.temp }}/${{ github.sha }}-with-models.tar.gz \
             https://git.lerch.org/api/packages/lobo/generic/aws-sdk-with-models/${{ github.sha }}/${{ github.sha }}-with-models.tar.gz
       - name: Build example

.gitea/workflows/zig-mach.yaml

@@ -26,19 +26,15 @@ jobs:
         with:
           ref: zig-mach
       - name: Setup Zig
-        uses: https://github.com/mlugg/setup-zig@v1.2.1
+        uses: mlugg/setup-zig@v1.2.1
         with:
           version: mach-latest
-      - name: Restore Zig caches
-        uses: https://github.com/Hanaasagi/zig-action-cache@3954aae427f8b05914e08dfd79f15e1f2e435929
       - name: Run gen
         run: zig build gen --verbose
       - name: Run smoke test
         run: zig build smoke-test --verbose
       - name: Run full tests
-        run: zig build test --verbose --summary all
-      - name: Run tests (release mode)
-        run: zig build test -Doptimize=ReleaseSafe --verbose
+        run: zig build test --verbose
       # Zig package manager expects everything to be inside a directory in the archive,
       # which it then strips out on download. So we need to shove everything inside a directory
       # the way GitHub/Gitea does for repo archives
@@ -71,7 +67,7 @@ jobs:
       # - run: echo "URL of foo (1) is ${{ steps.sign.outputs.URL_1 }}"
       - name: Publish source code with generated models
         run: |
-          curl -s --user ${{ github.actor }}:${{ secrets.PACKAGE_PUSH }} \
+          curl --user ${{ github.actor }}:${{ secrets.PACKAGE_PUSH }} \
             --upload-file ${{ runner.temp }}/${{ github.sha }}${{ env.PKG_PREFIX }}-with-models.tar.gz \
             https://git.lerch.org/api/packages/lobo/generic/aws-sdk-with-models/${{ github.sha }}/${{ github.sha }}${{ env.PKG_PREFIX }}-with-models.tar.gz
       - name: Build example

.gitea/workflows/zig-nightly.yaml

@@ -26,17 +26,11 @@ jobs:
         with:
           ref: zig-develop
       - name: Setup Zig
-        uses: https://github.com/mlugg/setup-zig@v1.2.1
+        uses: mlugg/setup-zig@v1.2.1
         with:
           version: master
-      - name: Restore Zig caches
-        uses: https://github.com/Hanaasagi/zig-action-cache@3954aae427f8b05914e08dfd79f15e1f2e435929
-      - name: Run smoke test
-        run: zig build smoke-test --verbose
-      - name: Run full tests
-        run: zig build test --verbose --summary all
-      - name: Run tests (release mode)
-        run: zig build test -Doptimize=ReleaseSafe --verbose
+      - name: Run tests
+        run: zig build test --verbose
       # Zig package manager expects everything to be inside a directory in the archive,
       # which it then strips out on download. So we need to shove everything inside a directory
       # the way GitHub/Gitea does for repo archives
@@ -69,7 +63,7 @@ jobs:
       # - run: echo "URL of foo (1) is ${{ steps.sign.outputs.URL_1 }}"
       - name: Publish source code with generated models
         run: |
-          curl -s --user ${{ github.actor }}:${{ secrets.PACKAGE_PUSH }} \
+          curl --user ${{ github.actor }}:${{ secrets.PACKAGE_PUSH }} \
             --upload-file ${{ runner.temp }}/${{ github.sha }}${{ env.PKG_PREFIX }}-with-models.tar.gz \
             https://git.lerch.org/api/packages/lobo/generic/aws-sdk-with-models/${{ github.sha }}/${{ github.sha }}${{ env.PKG_PREFIX }}-with-models.tar.gz
       - name: Build example

.gitea/workflows/zig-previous.yaml

@@ -20,18 +20,11 @@ jobs:
         with:
           ref: zig-0.13
       - name: Setup Zig
-        uses: https://github.com/mlugg/setup-zig@v1.2.1
+        uses: mlugg/setup-zig@v1.2.1
         with:
           version: 0.13.0
-      - name: Restore Zig caches
-        uses: https://github.com/Hanaasagi/zig-action-cache@3954aae427f8b05914e08dfd79f15e1f2e435929
-      - name: Run smoke test
-        run: zig build smoke-test --verbose
-      - name: Run full tests
-        run: zig build test --verbose --summary all
-      # Release mode fix not backported to 0.13.0 code
-      #- name: Run tests (release mode)
-      #  run: zig build test -Doptimize=ReleaseSafe --verbose
+      - name: Run tests
+        run: zig build test --verbose
       # Zig build scripts don't have the ability to import depenedencies directly
       # (https://github.com/ziglang/zig/issues/18164). We can allow downstream
       # build scripts to import aws with a few tweaks, but we can't @import("src/aws.zig")
@@ -76,7 +69,7 @@ jobs:
       # - run: echo "URL of foo (1) is ${{ steps.sign.outputs.URL_1 }}"
       - name: Publish source code with generated models
         run: |
-          curl -s --user ${{ github.actor }}:${{ secrets.PACKAGE_PUSH }} \
+          curl --user ${{ github.actor }}:${{ secrets.PACKAGE_PUSH }} \
             --upload-file ${{ runner.temp }}/${{ github.sha }}-with-models.tar.gz \
             https://git.lerch.org/api/packages/lobo/generic/aws-sdk-with-models/${{ github.sha }}/${{ github.sha }}-with-models.tar.gz
       - name: Build example

.github/workflows/build.yaml

@@ -14,11 +14,7 @@ jobs:
         uses: mlugg/setup-zig@v1.2.1
         with:
           version: 0.14.0
-      - name: Restore Zig caches
-        uses: Hanaasagi/zig-action-cache@3954aae427f8b05914e08dfd79f15e1f2e435929
       - name: Run tests
         run: zig build test -Dbroken-windows --verbose # Github runners try to run the windows tests despite disabling foreign checks
-      - name: Run tests (release mode)
-        run: zig build test -Doptimize=ReleaseSafe -Dbroken-windows --verbose # Github runners try to run the windows tests despite disabling foreign checks
       - name: Build example
         run: ( cd example && zig build ) # Make sure example builds

.github/workflows/zig-mach.yaml

@@ -14,11 +14,7 @@ jobs:
         uses: mlugg/setup-zig@v1.2.1
         with:
           version: mach-latest
-      - name: Restore Zig caches
-        uses: Hanaasagi/zig-action-cache@3954aae427f8b05914e08dfd79f15e1f2e435929
       - name: Run tests
         run: zig build test -Dbroken-windows --verbose
-      - name: Run tests (release mode)
-        run: zig build test -Doptimize=ReleaseSafe -Dbroken-windows --verbose # Github runners try to run the windows tests despite disabling foreign checks
       - name: Build example
         run: ( cd example && zig build ) # Make sure example builds

.github/workflows/zig-nightly.yaml

@@ -14,11 +14,7 @@ jobs:
         uses: mlugg/setup-zig@v1.2.1
         with:
           version: master
-      - name: Restore Zig caches
-        uses: Hanaasagi/zig-action-cache@3954aae427f8b05914e08dfd79f15e1f2e435929
       - name: Run tests
         run: zig build test -Dbroken-windows --verbose
-      - name: Run tests (release mode)
-        run: zig build test -Doptimize=ReleaseSafe -Dbroken-windows --verbose # Github runners try to run the windows tests despite disabling foreign checks
       - name: Build example
         run: ( cd example && zig build ) # Make sure example builds

.github/workflows/zig-previous.yaml

@@ -14,14 +14,7 @@ jobs:
         uses: mlugg/setup-zig@v1.2.1
         with:
           version: 0.13.0
-      - name: Restore Zig caches
-        uses: Hanaasagi/zig-action-cache@3954aae427f8b05914e08dfd79f15e1f2e435929
       - name: Run tests
-      - name: Run tests (release mode)
-        run: zig build test -Doptimize=ReleaseSafe -Dbroken-windows --verbose # Github runners try to run the windows tests despite disabling foreign checks
         run: zig build test -Dbroken-windows --verbose # Github runners try to run the windows tests despite disabling foreign checks
-      # This is broken without intension of backporting the patch at this time
-      # - name: Run tests (release mode)
-      #   run: zig build test -Doptimize=ReleaseSafe -Dbroken-windows --verbose # Github runners try to run the windows tests despite disabling foreign checks
       - name: Build example
         run: ( cd example && zig build ) # Make sure example builds

src/aws.zig

@@ -688,6 +688,7 @@ pub fn Request(comptime request_action: anytype) type {
 
         fn ParsedJsonData(comptime T: type) type {
             return struct {
+                raw_response_parsed: bool,
                 parsed_response_ptr: *T,
                 allocator: std.mem.Allocator,
 
@@ -696,6 +697,7 @@ pub fn Request(comptime request_action: anytype) type {
                 pub fn deinit(self: MySelf) void {
                     // This feels like it should result in a use after free, but it
                     // seems to be working?
+                    if (self.raw_response_parsed)
                         self.allocator.destroy(self.parsed_response_ptr);
                 }
             };
@@ -711,13 +713,11 @@ pub fn Request(comptime request_action: anytype) type {
                 std.mem.eql(u8, key, action.action_name ++ "Response") or
                 std.mem.eql(u8, key, action.action_name ++ "Result") or
                 isOtherNormalResponse(response_types.NormalResponse, key);
+            var raw_response_parsed = false;
             var stream = json.TokenStream.init(data);
             const parsed_response_ptr = blk: {
-                const ptr = try options.client.allocator.create(response_types.NormalResponse);
-                errdefer options.client.allocator.destroy(ptr);
-
-                if (!response_types.isRawPossible or found_normal_json_response) {
-                    ptr.* = (json.parse(response_types.NormalResponse, &stream, parser_options) catch |e| {
+                if (!response_types.isRawPossible or found_normal_json_response)
+                    break :blk &(json.parse(response_types.NormalResponse, &stream, parser_options) catch |e| {
                         log.err(
                             \\Call successful, but unexpected response from service.
                             \\This could be the result of a bug or a stale set of code generated
@@ -733,10 +733,10 @@ pub fn Request(comptime request_action: anytype) type {
                         return e;
                     });
 
-                    break :blk ptr;
-                }
-
                 log.debug("Appears server has provided a raw response", .{});
+                raw_response_parsed = true;
+                const ptr = try options.client.allocator.create(response_types.NormalResponse);
+                errdefer options.client.allocator.destroy(ptr);
                 @field(ptr.*, std.meta.fields(action.Response)[0].name) =
                     json.parse(response_types.RawResponse, &stream, parser_options) catch |e| {
                         log.err(
@@ -756,7 +756,8 @@ pub fn Request(comptime request_action: anytype) type {
                 break :blk ptr;
             };
             return ParsedJsonData(response_types.NormalResponse){
-                .parsed_response_ptr = parsed_response_ptr,
+                .raw_response_parsed = raw_response_parsed,
+                .parsed_response_ptr = @constCast(parsed_response_ptr), //TODO: why doesn't changing const->var above fix this?
                 .allocator = options.client.allocator,
             };
         }
@@ -1701,7 +1702,7 @@ const TestSetup = struct {
     request_options: TestOptions,
     server_thread: std.Thread = undefined,
     creds: aws_auth.Credentials = undefined,
-    client: Client = undefined,
+    client: *Client = undefined,
     started: bool = false,
 
     const Self = @This();
@@ -1737,8 +1738,8 @@ const TestSetup = struct {
             null,
         );
         aws_creds.static_credentials = self.creds;
-        const client = Client.init(self.allocator, .{});
-        self.client = client;
+        var client = Client.init(self.allocator, .{});
+        self.client = &client;
         return .{
             .region = "us-west-2",
             .client = client,