Breaking changes between Giraffe 0.1.0-alpha-025 and 0.1.0-beta001

[giraffe-fsharp]

Breaking changes between Giraffe 0.1.0-alpha-025 and 0.1.0-beta001

[gerardtoconnor]

Both this continuation update and task were done for performance reasons, you can read blog post I did to explain performance benefits here if your interested.

https://medium.com/@gerardtoconnor/carry-on-continuation-over-binding-pipelines-for-functional-web-58bd7e6ea009

It's a slightly more complex way of doing the handlers but there is a focus on performance and the consensus was it was worth breaking now for the long run.

If you have anymore issues let me know, happy to help.

[dustinmoris]

Apologies for this issue. Even though I have updated the latest giraffe-template, the previous versions break now when you pull the beta package from an alpha template. I should have not referenced giraffe with a wildcard in the .fsproj. I will set it to a specific version going forward with the new template releases, but meanwhile if someone runs into this issue you can either pin the latest giraffe alpha package, or preferably update the error handler to match the new beta. You can simplify the error handler even further:

let errorHandler (ex : Exception) (logger : ILogger) =
    logger.LogError(EventId(), ex, "An unhandled exception has occurred while executing the request.")
    clearResponse >=> setStatusCode 500 >=> text ex.Message

Also please update the giraffe-template by running:

dotnet new -i giraffe-template::0.1.7

[gerardtoconnor]

The error handler needs to be updated to use new HttpHandler format such that it additionally accepts a 'HttpFunc' functions as well as 'HttpContext' like 'fun next ctx ->. ..'

In the last line where handlers are composed, instead of 'ctx |> (handler)' you need to apply both parameters to functions like '(handler) next ctx'

[ryukinix]

Yes, I understand. Your tip fixed the problem. I can know why this change was made? Is related to the new Task support #53?

If anyone struggle with it too, the new errorHandler which works is it:

// ---------------------------------
// Error handler
// ---------------------------------

let errorHandler (ex : Exception) (logger : ILogger) (next: HttpFunc) (ctx : HttpContext) =
    logger.LogError(EventId(0), ex, "An unhandled exception has occurred while executing the request.")
    (clearResponse >=> setStatusCode 500 >=> text ex.Message) next ctx