| Age | Commit message (Collapse) | Author |
|---|
|
The used `func(http.ResponseWriter, *http.Request)` return values made
the HTTP handler factory functions quite unreadable. Thus it is switched
to the http.Handler type.
|
|
The reason for the introduction of this dependency was that it was easy
to setup routes with HTTP method restrictions.
Since Go 1.22 this feature is part of the standard library. Method
restrictions are part of the patterns used to register routes [1].
[1]: https://pkg.go.dev/net/http#hdr-Patterns-ServeMux
|
|
It is not required to create a struct definition with only one member.
The new solution is more elegant.
|
|
Reducing global variables makes it easier to understand functions
independently of the rest of the code.
Adding the new model.DB type as a custom variant of the sql.DB type
makes it possible to write methods for the database which makes the code
way more readable.
|
|
To avoid not clickable recipes on the /recipes page a default name used
to be inserted on recipe creation.
This was not a proper fix for the problem and also was annoying that the
user first had to remove the default recipe name.
This commit removes this default name.
|
|
This variable will be the only used representation for version
information. It can trivially be used for an equality check. Further
data as semantic versioning must be parsed from this string.
|
|
This separates the main control flow and HTTP-related high-level code.
Furthermore the new main.Server type makes the related methods and
function more like the functions from the standard library with a
NewServer() function and methods with only one word as name.
|
|
Restarting from scratch seems to be the fastest approach to switch to
sqlite and get rid of some other structural mistakes from the past.
|
|
This removes a lot of repetative code.
|
|
This is now just not needed.
|
|
|
|
Recipe images are currently complicated to implement and have little
use. Probably they will not be covered in the next release.
|
|
This auto-applies the recommended Go codestyle.
|
|
Logging during a request is at the moment not really needed. Printing
the config to the log was a stupid idea too.
|
|
Multiplexer functions used to return closures which are able to handle
HTTP requests for historc reasons. Now the multiplexers can be regular
HTTP handlers which is simpler.
|
|
Passing the database pointer around is a lot of text and has no benefit.
|
|
|
|
This fits better because this file is the central entry point of the
Ceres webserver.
|
