Compare commits

..

No commits in common. "cd9bf618f193bc96829c6edc90886ca9bb084fe0" and "6a9ff3de24892dae464898a71326b31cf4d140f2" have entirely different histories.

4 changed files with 129 additions and 118 deletions

View File

@ -1,29 +0,0 @@
name: Generic zig build
on:
workflow_dispatch:
push:
branches:
- '*'
- '!zig-develop*'
jobs:
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: elerch/setup-zig@v3
with:
version: 0.12.0
- uses: elerch/zig-action-cache@v1.1.6
- name: Build project
run: zig build --summary all
- name: Run tests
run: zig build test --summary all
- name: Notify
uses: elerch/action-notify-ntfy@v2.github
if: always() && env.GITEA_ACTIONS == 'true'
with:
host: ${{ secrets.NTFY_HOST }}
topic: ${{ secrets.NTFY_TOPIC }}
status: ${{ job.status }}
user: ${{ secrets.NTFY_USER }}
password: ${{ secrets.NTFY_PASSWORD }}

View File

@ -45,7 +45,7 @@ pub fn build(b: *std.Build) !void {
const test_step = b.step("test", "Run library tests"); const test_step = b.step("test", "Run library tests");
test_step.dependOn(&run_main_tests.step); test_step.dependOn(&run_main_tests.step);
const exe = b.addExecutable(.{ var exe = b.addExecutable(.{
.name = "custom", .name = "custom",
.root_source_file = .{ .path = "src/sample-main.zig" }, .root_source_file = .{ .path = "src/sample-main.zig" },
.target = target, .target = target,
@ -72,6 +72,6 @@ pub fn build(b: *std.Build) !void {
/// deploy depends on iam and package /// deploy depends on iam and package
/// ///
/// iam and package do not have any dependencies /// iam and package do not have any dependencies
pub fn lambdaBuildOptions(b: *std.Build, exe: *std.Build.Step.Compile) !void { pub fn lambdaBuildOptions(b: *std.build.Builder, exe: *std.Build.Step.Compile) !void {
try @import("lambdabuild.zig").lambdaBuildOptions(b, exe); try @import("lambdabuild.zig").lambdaBuildOptions(b, exe);
} }

View File

@ -30,7 +30,7 @@ fn addArgs(allocator: std.mem.Allocator, original: []const u8, args: [][]const u
/// deploy depends on iam and package /// deploy depends on iam and package
/// ///
/// iam and package do not have any dependencies /// iam and package do not have any dependencies
pub fn lambdaBuildOptions(b: *std.Build, exe: *std.Build.Step.Compile) !void { pub fn lambdaBuildOptions(b: *std.build.Builder, exe: *std.Build.Step.Compile) !void {
// The rest of this function is currently reliant on the use of Linux // The rest of this function is currently reliant on the use of Linux
// system being used to build the lambda function // system being used to build the lambda function
// //
@ -143,7 +143,7 @@ pub fn lambdaBuildOptions(b: *std.Build, exe: *std.Build.Step.Compile) !void {
} }
const cmd = try std.fmt.allocPrint(b.allocator, ifstatement, .{ const cmd = try std.fmt.allocPrint(b.allocator, ifstatement, .{
function_name_file, function_name_file,
std.fs.path.dirname(exe.root_module.root_source_file.?.path).?, std.fs.path.dirname(exe.root_src.?.path).?,
function_name_file, function_name_file,
function_name, function_name,
not_found_fmt, not_found_fmt,

View File

@ -5,6 +5,7 @@ const HandlerFn = *const fn (std.mem.Allocator, []const u8) anyerror![]const u8;
const log = std.log.scoped(.lambda); const log = std.log.scoped(.lambda);
var empty_headers: std.http.Headers = undefined;
var client: ?std.http.Client = null; var client: ?std.http.Client = null;
const prefix = "http://"; const prefix = "http://";
@ -19,7 +20,7 @@ pub fn deinit() void {
/// This function is intended to loop infinitely. If not used in this manner, /// This function is intended to loop infinitely. If not used in this manner,
/// make sure to call the deinit() function /// make sure to call the deinit() function
pub fn run(allocator: ?std.mem.Allocator, event_handler: HandlerFn) !void { // TODO: remove inferred error set? pub fn run(allocator: ?std.mem.Allocator, event_handler: HandlerFn) !void { // TODO: remove inferred error set?
const lambda_runtime_uri = std.posix.getenv("AWS_LAMBDA_RUNTIME_API") orelse test_lambda_runtime_uri.?; const lambda_runtime_uri = std.os.getenv("AWS_LAMBDA_RUNTIME_API") orelse test_lambda_runtime_uri.?;
// TODO: If this is null, go into single use command line mode // TODO: If this is null, go into single use command line mode
var gpa = std.heap.GeneralPurposeAllocator(.{}){}; var gpa = std.heap.GeneralPurposeAllocator(.{}){};
@ -36,6 +37,8 @@ pub fn run(allocator: ?std.mem.Allocator, event_handler: HandlerFn) !void { // T
// so we'll do this instead // so we'll do this instead
if (client != null) return error.MustDeInitBeforeCallingRunAgain; if (client != null) return error.MustDeInitBeforeCallingRunAgain;
client = .{ .allocator = alloc }; client = .{ .allocator = alloc };
empty_headers = std.http.Headers.init(alloc);
defer empty_headers.deinit();
log.info("tid {d} (lambda): Bootstrap initializing with event url: {s}", .{ std.Thread.getCurrentId(), url }); log.info("tid {d} (lambda): Bootstrap initializing with event url: {s}", .{ std.Thread.getCurrentId(), url });
while (lambda_remaining_requests == null or lambda_remaining_requests.? > 0) { while (lambda_remaining_requests == null or lambda_remaining_requests.? > 0) {
@ -55,7 +58,7 @@ pub fn run(allocator: ?std.mem.Allocator, event_handler: HandlerFn) !void { // T
var ev = getEvent(req_allocator, uri) catch |err| { var ev = getEvent(req_allocator, uri) catch |err| {
// Well, at this point all we can do is shout at the void // Well, at this point all we can do is shout at the void
log.err("Error fetching event details: {}", .{err}); log.err("Error fetching event details: {}", .{err});
std.posix.exit(1); std.os.exit(1);
// continue; // continue;
}; };
if (ev == null) continue; // this gets logged in getEvent, and without if (ev == null) continue; // this gets logged in getEvent, and without
@ -76,12 +79,12 @@ pub fn run(allocator: ?std.mem.Allocator, event_handler: HandlerFn) !void { // T
const Event = struct { const Event = struct {
allocator: std.mem.Allocator, allocator: std.mem.Allocator,
event_data: []const u8, event_data: []u8,
request_id: []const u8, request_id: []u8,
const Self = @This(); const Self = @This();
pub fn init(allocator: std.mem.Allocator, event_data: []const u8, request_id: []const u8) Self { pub fn init(allocator: std.mem.Allocator, event_data: []u8, request_id: []u8) Self {
return .{ return .{
.allocator = allocator, .allocator = allocator,
.event_data = event_data, .event_data = event_data,
@ -124,36 +127,45 @@ const Event = struct {
defer self.allocator.free(content_fmt); defer self.allocator.free(content_fmt);
log.err("Posting to {s}: Data {s}", .{ err_url, content_fmt }); log.err("Posting to {s}: Data {s}", .{ err_url, content_fmt });
var err_headers = std.http.Headers.init(self.allocator);
defer err_headers.deinit();
err_headers.append(
"Lambda-Runtime-Function-Error-Type",
"HandlerReturned",
) catch |append_err| {
log.err("Error appending error header to post response for request id {s}: {}", .{ self.request_id, append_err });
std.os.exit(1);
};
// TODO: There is something up with using a shared client in this way // TODO: There is something up with using a shared client in this way
// so we're taking a perf hit in favor of stability. In a practical // so we're taking a perf hit in favor of stability. In a practical
// sense, without making HTTPS connections (lambda environment is // sense, without making HTTPS connections (lambda environment is
// non-ssl), this shouldn't be a big issue // non-ssl), this shouldn't be a big issue
var cl = std.http.Client{ .allocator = self.allocator }; var cl = std.http.Client{ .allocator = self.allocator };
defer cl.deinit(); defer cl.deinit();
const res = cl.fetch(.{ var req = try cl.request(.POST, err_uri, empty_headers, .{});
.method = .POST, // var req = try client.?.request(.POST, err_uri, empty_headers, .{});
.payload = content_fmt, // defer req.deinit();
.location = .{ .uri = err_uri }, req.transfer_encoding = .{ .content_length = content_fmt.len };
.extra_headers = &.{ req.start() catch |post_err| { // Well, at this point all we can do is shout at the void
.{
.name = "Lambda-Runtime-Function-Error-Type",
.value = "HandlerReturned",
},
},
}) catch |post_err| { // Well, at this point all we can do is shout at the void
log.err("Error posting response (start) for request id {s}: {}", .{ self.request_id, post_err }); log.err("Error posting response (start) for request id {s}: {}", .{ self.request_id, post_err });
std.posix.exit(1); std.os.exit(1);
};
try req.writeAll(content_fmt);
try req.finish();
req.wait() catch |post_err| { // Well, at this point all we can do is shout at the void
log.err("Error posting response (wait) for request id {s}: {}", .{ self.request_id, post_err });
std.os.exit(1);
}; };
// TODO: Determine why this post is not returning // TODO: Determine why this post is not returning
if (res.status != .ok) { if (req.response.status != .ok) {
// Documentation says something about "exit immediately". The // Documentation says something about "exit immediately". The
// Lambda infrastrucutre restarts, so it's unclear if that's necessary. // Lambda infrastrucutre restarts, so it's unclear if that's necessary.
// It seems as though a continue should be fine, and slightly faster // It seems as though a continue should be fine, and slightly faster
log.err("Post fail: {} {s}", .{ log.err("Get fail: {} {s}", .{
@intFromEnum(res.status), @intFromEnum(req.response.status),
res.status.phrase() orelse "", req.response.status.phrase() orelse "",
}); });
std.posix.exit(1); std.os.exit(1);
} }
log.err("Error reporting post complete", .{}); log.err("Error reporting post complete", .{});
} }
@ -165,20 +177,24 @@ const Event = struct {
.{ prefix, lambda_runtime_uri, postfix, self.request_id }, .{ prefix, lambda_runtime_uri, postfix, self.request_id },
); );
defer self.allocator.free(response_url); defer self.allocator.free(response_url);
const response_uri = try std.Uri.parse(response_url);
var cl = std.http.Client{ .allocator = self.allocator };
defer cl.deinit();
var req = try cl.request(.POST, response_uri, empty_headers, .{});
// var req = try client.?.request(.POST, response_uri, empty_headers, .{});
defer req.deinit();
const response_content = try std.fmt.allocPrint( const response_content = try std.fmt.allocPrint(
self.allocator, self.allocator,
"{{ \"content\": \"{s}\" }}", "{{ \"content\": \"{s}\" }}",
.{event_response}, .{event_response},
); );
defer self.allocator.free(response_content); defer self.allocator.free(response_content);
var cl = std.http.Client{ .allocator = self.allocator };
defer cl.deinit(); req.transfer_encoding = .{ .content_length = response_content.len };
const res = try cl.fetch(.{ try req.start();
.method = .POST, try req.writeAll(response_content);
.payload = response_content, try req.finish();
.location = .{ .url = response_url }, try req.wait();
});
if (res.status != .ok) return error.UnexpectedStatusFromPostResponse;
} }
}; };
@ -189,33 +205,37 @@ fn getEvent(allocator: std.mem.Allocator, event_data_uri: std.Uri) !?Event {
// non-ssl), this shouldn't be a big issue // non-ssl), this shouldn't be a big issue
var cl = std.http.Client{ .allocator = allocator }; var cl = std.http.Client{ .allocator = allocator };
defer cl.deinit(); defer cl.deinit();
var response_bytes = std.ArrayList(u8).init(allocator); var req = try cl.request(.GET, event_data_uri, empty_headers, .{});
defer response_bytes.deinit(); // var req = try client.?.request(.GET, event_data_uri, empty_headers, .{});
var server_header_buffer: [16 * 1024]u8 = undefined; // defer req.deinit();
try req.start();
try req.finish();
// Lambda freezes the process at this line of code. During warm start, // Lambda freezes the process at this line of code. During warm start,
// the process will unfreeze and data will be sent in response to client.get // the process will unfreeze and data will be sent in response to client.get
var res = try cl.fetch(.{ try req.wait();
.server_header_buffer = &server_header_buffer, if (req.response.status != .ok) {
.location = .{ .uri = event_data_uri },
.response_storage = .{ .dynamic = &response_bytes },
});
if (res.status != .ok) {
// Documentation says something about "exit immediately". The // Documentation says something about "exit immediately". The
// Lambda infrastrucutre restarts, so it's unclear if that's necessary. // Lambda infrastrucutre restarts, so it's unclear if that's necessary.
// It seems as though a continue should be fine, and slightly faster // It seems as though a continue should be fine, and slightly faster
// std.os.exit(1); // std.os.exit(1);
log.err("Lambda server event response returned bad error code: {} {s}", .{ log.err("Lambda server event response returned bad error code: {} {s}", .{
@intFromEnum(res.status), @intFromEnum(req.response.status),
res.status.phrase() orelse "", req.response.status.phrase() orelse "",
}); });
return error.EventResponseNotOkResponse; return error.EventResponseNotOkResponse;
} }
var request_id: ?[]const u8 = null; var request_id: ?[]const u8 = null;
var header_it = std.http.HeaderIterator.init(server_header_buffer[0..]); var content_length: ?usize = null;
while (header_it.next()) |h| { for (req.response.headers.list.items) |h| {
if (std.ascii.eqlIgnoreCase(h.name, "Lambda-Runtime-Aws-Request-Id")) if (std.ascii.eqlIgnoreCase(h.name, "Lambda-Runtime-Aws-Request-Id"))
request_id = h.value; request_id = h.value;
if (std.ascii.eqlIgnoreCase(h.name, "Content-Length")) {
content_length = std.fmt.parseUnsigned(usize, h.value, 10) catch null;
if (content_length == null)
log.warn("Error parsing content length value: '{s}'", .{h.value});
}
// TODO: XRay uses an environment variable to do its magic. It's our // TODO: XRay uses an environment variable to do its magic. It's our
// responsibility to set this, but no zig-native setenv(3)/putenv(3) // responsibility to set this, but no zig-native setenv(3)/putenv(3)
// exists. I would kind of rather not link in libc for this, // exists. I would kind of rather not link in libc for this,
@ -231,12 +251,26 @@ fn getEvent(allocator: std.mem.Allocator, event_data_uri: std.Uri) !?Event {
log.err("Could not find request id: skipping request", .{}); log.err("Could not find request id: skipping request", .{});
return null; return null;
} }
if (content_length == null) {
// We can't report back an issue because the runtime error reporting endpoint
// uses request id in its path. So the best we can do is log the error and move
// on here.
log.err("No content length provided for event data", .{});
return null;
}
const req_id = request_id.?; const req_id = request_id.?;
log.debug("got lambda request with id {s}", .{req_id}); log.debug("got lambda request with id {s}", .{req_id});
var resp_payload = try std.ArrayList(u8).initCapacity(allocator, content_length.?);
defer resp_payload.deinit();
try resp_payload.resize(content_length.?);
var response_data = try resp_payload.toOwnedSlice();
errdefer allocator.free(response_data);
_ = try req.readAll(response_data);
return Event.init( return Event.init(
allocator, allocator,
try response_bytes.toOwnedSlice(), response_data,
try allocator.dupe(u8, req_id), try allocator.dupe(u8, req_id),
); );
} }
@ -265,18 +299,22 @@ fn startServer(allocator: std.mem.Allocator) !std.Thread {
} }
fn threadMain(allocator: std.mem.Allocator) !void { fn threadMain(allocator: std.mem.Allocator) !void {
var server = std.http.Server.init(allocator, .{ .reuse_address = true });
// defer server.deinit();
const address = try std.net.Address.parseIp("127.0.0.1", 0); const address = try std.net.Address.parseIp("127.0.0.1", 0);
var http_server = try address.listen(.{ .reuse_address = true }); try server.listen(address);
server_port = http_server.listen_address.in.getPort(); server_port = server.socket.listen_address.in.getPort();
test_lambda_runtime_uri = try std.fmt.allocPrint(allocator, "127.0.0.1:{d}", .{server_port.?}); test_lambda_runtime_uri = try std.fmt.allocPrint(allocator, "127.0.0.1:{d}", .{server_port.?});
log.debug("server listening at {s}", .{test_lambda_runtime_uri.?}); log.debug("server listening at {s}", .{test_lambda_runtime_uri.?});
defer server.deinit();
defer test_lambda_runtime_uri = null; defer test_lambda_runtime_uri = null;
defer server_port = null; defer server_port = null;
log.info("starting server thread, tid {d}", .{std.Thread.getCurrentId()}); log.info("starting server thread, tid {d}", .{std.Thread.getCurrentId()});
var arena = std.heap.ArenaAllocator.init(allocator); var arena = std.heap.ArenaAllocator.init(allocator);
defer arena.deinit(); defer arena.deinit();
const aa = arena.allocator(); var aa = arena.allocator();
// We're in control of all requests/responses, so this flag will tell us // We're in control of all requests/responses, so this flag will tell us
// when it's time to shut down // when it's time to shut down
while (server_remaining_requests > 0) { while (server_remaining_requests > 0) {
@ -291,7 +329,7 @@ fn threadMain(allocator: std.mem.Allocator) !void {
// } // }
// } // }
processRequest(aa, &http_server) catch |e| { processRequest(aa, &server) catch |e| {
log.err("Unexpected error processing request: {any}", .{e}); log.err("Unexpected error processing request: {any}", .{e});
if (@errorReturnTrace()) |trace| { if (@errorReturnTrace()) |trace| {
std.debug.dumpStackTrace(trace.*); std.debug.dumpStackTrace(trace.*);
@ -300,33 +338,35 @@ fn threadMain(allocator: std.mem.Allocator) !void {
} }
} }
fn processRequest(allocator: std.mem.Allocator, server: *std.net.Server) !void { fn processRequest(allocator: std.mem.Allocator, server: *std.http.Server) !void {
server_ready = true; server_ready = true;
errdefer server_ready = false; errdefer server_ready = false;
log.debug( log.debug(
"tid {d} (server): server waiting to accept. requests remaining: {d}", "tid {d} (server): server waiting to accept. requests remaining: {d}",
.{ std.Thread.getCurrentId(), server_remaining_requests + 1 }, .{ std.Thread.getCurrentId(), server_remaining_requests + 1 },
); );
var connection = try server.accept(); var res = try server.accept(.{ .allocator = allocator });
defer connection.stream.close();
server_ready = false; server_ready = false;
defer res.deinit();
defer _ = res.reset();
try res.wait(); // wait for client to send a complete request head
var read_buffer: [1024 * 16]u8 = undefined; const errstr = "Internal Server Error\n";
var http_server = std.http.Server.init(connection, &read_buffer); var errbuf: [errstr.len]u8 = undefined;
@memcpy(&errbuf, errstr);
var response_bytes: []const u8 = errbuf[0..];
if (http_server.state == .ready) { if (res.request.content_length) |l|
var request = http_server.receiveHead() catch |err| switch (err) { server_request_aka_lambda_response = try res.reader().readAllAlloc(allocator, @as(usize, l));
error.HttpConnectionClosing => return,
else => { log.debug(
std.log.err("closing http connection: {s}", .{@errorName(err)}); "tid {d} (server): {d} bytes read from request",
std.log.debug("Error occurred from this request: \n{s}", .{read_buffer[0..http_server.read_buffer_len]}); .{ std.Thread.getCurrentId(), server_request_aka_lambda_response.len },
return; );
},
}; // try response.headers.append("content-type", "text/plain");
server_request_aka_lambda_response = try (try request.reader()).readAllAlloc(allocator, std.math.maxInt(usize)); response_bytes = serve(allocator, &res) catch |e| brk: {
var respond_options = std.http.Server.Request.RespondOptions{}; res.status = .internal_server_error;
const response_bytes = serve(allocator, request, &respond_options) catch |e| brk: {
respond_options.status = .internal_server_error;
// TODO: more about this particular request // TODO: more about this particular request
log.err("Unexpected error from executor processing request: {any}", .{e}); log.err("Unexpected error from executor processing request: {any}", .{e});
if (@errorReturnTrace()) |trace| { if (@errorReturnTrace()) |trace| {
@ -334,20 +374,20 @@ fn processRequest(allocator: std.mem.Allocator, server: *std.net.Server) !void {
} }
break :brk "Unexpected error generating request to lambda"; break :brk "Unexpected error generating request to lambda";
}; };
try request.respond(response_bytes, respond_options); res.transfer_encoding = .{ .content_length = response_bytes.len };
try res.do();
_ = try res.writer().writeAll(response_bytes);
try res.finish();
log.debug( log.debug(
"tid {d} (server): sent response: {s}", "tid {d} (server): sent response",
.{ std.Thread.getCurrentId(), response_bytes }, .{std.Thread.getCurrentId()},
); );
}
} }
fn serve(allocator: std.mem.Allocator, request: std.http.Server.Request, respond_options: *std.http.Server.Request.RespondOptions) ![]const u8 { fn serve(allocator: std.mem.Allocator, res: *std.http.Server.Response) ![]const u8 {
_ = allocator; _ = allocator;
_ = request; // try res.headers.append("content-length", try std.fmt.allocPrint(allocator, "{d}", .{server_response.len}));
respond_options.extra_headers = &.{ try res.headers.append("Lambda-Runtime-Aws-Request-Id", "69");
.{ .name = "Lambda-Runtime-Aws-Request-Id", .value = "69" },
};
return server_response; return server_response;
} }
@ -366,7 +406,7 @@ fn test_run(allocator: std.mem.Allocator, event_handler: HandlerFn) !std.Thread
fn lambda_request(allocator: std.mem.Allocator, request: []const u8, request_count: usize) ![]u8 { fn lambda_request(allocator: std.mem.Allocator, request: []const u8, request_count: usize) ![]u8 {
var arena = std.heap.ArenaAllocator.init(allocator); var arena = std.heap.ArenaAllocator.init(allocator);
defer arena.deinit(); defer arena.deinit();
const aa = arena.allocator(); var aa = arena.allocator();
// Setup our server to run, and set the response for the server to the // Setup our server to run, and set the response for the server to the
// request. There is a cognitive disconnect here between mental model and // request. There is a cognitive disconnect here between mental model and
// physical model. // physical model.