mirror of
https://github.com/fnproject/fn.git
synced 2022-10-28 21:29:17 +03:00
* push down app listeners to a datastore fnext.NewDatastore returns a datastore that wraps the appropriate methods for AppListener in a Datastore implementation. this is more future proof than needing to wrap every call of GetApp/UpdateApp/etc with the listeners, there are a few places where this can happen and it seems like the AppListener behavior is supposed to wrap the datastore, not just the front end methods surrounding CRUD ops on an app. the hairy case that came up was when fiddling with the create/update route business. this changes the FireBeforeApp* ops to be an AppListener implementation itself rather than having the Server itself expose certain methods to fire off the app listeners, now they're on the datastore itself, which the server can return the instance of. small change to BeforeAppDelete/AfterAppDelete -- we were passing in a half baked struct with only the name filled in and not filling in the fields anywhere. this is mostly just misleading, we could fill in the app, but we weren't and don't really want to, it's more to notify of an app deletion event so that an extension can behave accordingly instead of letting a user inspect the app. i know of 3 extensions and the changes required to update are very small. cleans up all the front end implementations FireBefore/FireAfter. this seems potentially less flexible than previous version if we do want to allow users some way to call the database methods without using the extensions, but that's exactly the trade off, as far as the AppListener's are described it seems heavily implied that this should be the case. mostly a feeler, for the above reasons, but this was kind of odorous so just went for it. we do need to lock in the extension api stuff. * hand em an app that's been smokin the reefer
150 lines
4.0 KiB
Go
150 lines
4.0 KiB
Go
package server
|
|
|
|
import (
|
|
"context"
|
|
"net/http"
|
|
"path"
|
|
"strings"
|
|
|
|
"github.com/fnproject/fn/api"
|
|
"github.com/fnproject/fn/api/models"
|
|
"github.com/gin-gonic/gin"
|
|
)
|
|
|
|
/* handleRouteCreateOrUpdate is used to handle POST PUT and PATCH for routes.
|
|
Post will only create route if its not there and create app if its not.
|
|
create only
|
|
Post does not skip validation of zero values
|
|
Put will create app if its not there and if route is there update if not it will create new route.
|
|
update if exists or create if not exists
|
|
Put does not skip validation of zero values
|
|
Patch will not create app if it does not exist since the route needs to exist as well...
|
|
update only
|
|
Patch accepts partial updates / skips validation of zero values.
|
|
*/
|
|
func (s *Server) handleRoutesPostPutPatch(c *gin.Context) {
|
|
ctx := c.Request.Context()
|
|
method := strings.ToUpper(c.Request.Method)
|
|
|
|
var wroute models.RouteWrapper
|
|
err := bindRoute(c, method, &wroute)
|
|
if err != nil {
|
|
handleErrorResponse(c, err)
|
|
return
|
|
}
|
|
if method != http.MethodPatch {
|
|
err = s.ensureApp(ctx, &wroute, method)
|
|
if err != nil {
|
|
handleErrorResponse(c, err)
|
|
return
|
|
}
|
|
}
|
|
resp, err := s.ensureRoute(ctx, method, &wroute)
|
|
if err != nil {
|
|
handleErrorResponse(c, err)
|
|
return
|
|
}
|
|
|
|
c.JSON(http.StatusOK, resp)
|
|
}
|
|
|
|
func (s *Server) submitRoute(ctx context.Context, wroute *models.RouteWrapper) error {
|
|
r, err := s.datastore.InsertRoute(ctx, wroute.Route)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
wroute.Route = r
|
|
return nil
|
|
}
|
|
|
|
func (s *Server) changeRoute(ctx context.Context, wroute *models.RouteWrapper) error {
|
|
r, err := s.datastore.UpdateRoute(ctx, wroute.Route)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
wroute.Route = r
|
|
return nil
|
|
}
|
|
|
|
// ensureApp will only execute if it is on put
|
|
func (s *Server) ensureRoute(ctx context.Context, method string, wroute *models.RouteWrapper) (routeResponse, error) {
|
|
bad := new(routeResponse)
|
|
|
|
switch method {
|
|
case http.MethodPost:
|
|
err := s.submitRoute(ctx, wroute)
|
|
if err != nil {
|
|
return *bad, err
|
|
}
|
|
return routeResponse{"Route successfully created", wroute.Route}, nil
|
|
case http.MethodPut:
|
|
_, err := s.datastore.GetRoute(ctx, wroute.Route.AppName, wroute.Route.Path)
|
|
if err != nil && err == models.ErrRoutesNotFound {
|
|
err := s.submitRoute(ctx, wroute)
|
|
if err != nil {
|
|
return *bad, err
|
|
}
|
|
return routeResponse{"Route successfully created", wroute.Route}, nil
|
|
}
|
|
err = s.changeRoute(ctx, wroute)
|
|
if err != nil {
|
|
return *bad, err
|
|
}
|
|
return routeResponse{"Route successfully updated", wroute.Route}, nil
|
|
case http.MethodPatch:
|
|
err := s.changeRoute(ctx, wroute)
|
|
if err != nil {
|
|
return *bad, err
|
|
}
|
|
return routeResponse{"Route successfully updated", wroute.Route}, nil
|
|
}
|
|
return *bad, nil
|
|
}
|
|
|
|
// ensureApp will only execute if it is on post or put. Patch is not allowed to create apps.
|
|
func (s *Server) ensureApp(ctx context.Context, wroute *models.RouteWrapper, method string) error {
|
|
app, err := s.datastore.GetApp(ctx, wroute.Route.AppName)
|
|
if err != nil && err != models.ErrAppsNotFound {
|
|
return err
|
|
} else if app == nil {
|
|
// Create a new application
|
|
newapp := &models.App{Name: wroute.Route.AppName}
|
|
_, err = s.datastore.InsertApp(ctx, newapp)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
}
|
|
return nil
|
|
}
|
|
|
|
// bindRoute binds the RouteWrapper to the json from the request.
|
|
func bindRoute(c *gin.Context, method string, wroute *models.RouteWrapper) error {
|
|
err := c.BindJSON(wroute)
|
|
if err != nil {
|
|
if models.IsAPIError(err) {
|
|
return err
|
|
}
|
|
return models.ErrInvalidJSON
|
|
}
|
|
|
|
if wroute.Route == nil {
|
|
return models.ErrRoutesMissingNew
|
|
}
|
|
wroute.Route.AppName = c.MustGet(api.AppName).(string)
|
|
|
|
if method == http.MethodPut || method == http.MethodPatch {
|
|
p := path.Clean(c.MustGet(api.Path).(string))
|
|
|
|
if wroute.Route.Path != "" && wroute.Route.Path != p {
|
|
return models.ErrRoutesPathImmutable
|
|
}
|
|
wroute.Route.Path = p
|
|
}
|
|
if method == http.MethodPost {
|
|
if wroute.Route.Path == "" {
|
|
return models.ErrRoutesMissingPath
|
|
}
|
|
}
|
|
return nil
|
|
}
|