Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
65 changes: 28 additions & 37 deletions src/Servers/GenHttpServer/Program.cs
Original file line number Diff line number Diff line change
@@ -1,44 +1,35 @@
using GenHTTP.Api.Content;
using GenHTTP.Api.Protocol;

using GenHTTP.Engine.Internal;

using GenHTTP.Modules.Functional;
using GenHTTP.Modules.Layouting;
using GenHTTP.Modules.Practices;

var port = args.Length > 0 && int.TryParse(args[0], out var p) ? p : 8080;
var port = (args.Length > 0 && ushort.TryParse(args[0], out var p)) ? p : (ushort)8080;

var echoHandler = Inline.Create()
.Any(async (IRequest request) =>
{
await ValueTask.CompletedTask;
var sb = new System.Text.StringBuilder();
foreach (var h in request.Headers)
sb.AppendLine($"{h.Key}: {h.Value}");
return sb.ToString();
});

var rootHandler = Inline.Create()
.Get(async (IRequest request) =>
{
await ValueTask.CompletedTask;
return "OK";
})
.Post(async (IRequest request) =>
var app = Inline.Create()
.Get("/echo", (IRequest request) => Echo(request))
Copy link
Owner

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ah - POST with no body is rather unintuitive 🤔

Copy link
Owner

Choose a reason for hiding this comment

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

Hmm indeed, I think initial idea was to send a body with it because malformed headers can cause some header data to leak to the body

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Interestingly GET scored better because it returned a 405 then :D

image

Copy link
Owner

Choose a reason for hiding this comment

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

Oh snap, need to adjust those tests pass/fail criteria.. not simple because there is no explicit RFC for it

.Post((Stream body) => RequestContent(body))
.Any(() => StringContent());

return await Host.Create()
.Handler(app)
.Defaults()
.Port(port)
.RunAsync();

static string Echo(IRequest request)
{
var headers = new System.Text.StringBuilder();

foreach (var h in request.Headers)
{
if (request.Content is not null)
{
using var reader = new StreamReader(request.Content);
return await reader.ReadToEndAsync();
}
return "";
});

var layout = Layout.Create()
.Add("echo", echoHandler)
.Add(rootHandler);

await Host.Create()
.Handler(layout)
.Defaults()
.Port((ushort)port)
.RunAsync();
headers.AppendLine($"{h.Key}: {h.Value}");
}

return headers.ToString();
}

static string StringContent() => "OK";

static Stream RequestContent(Stream body) => body;
Loading