Skip to content

Commit

Permalink
Fix manifest is missing "configurationRequired"
Browse files Browse the repository at this point in the history
  • Loading branch information
doingodswork committed Oct 10, 2020
1 parent 5878621 commit b6ed96e
Showing 1 changed file with 15 additions and 10 deletions.
25 changes: 15 additions & 10 deletions handlers.go
Expand Up @@ -45,18 +45,23 @@ func createManifestHandler(manifest Manifest, logger *zap.Logger, manifestCallba
var userData interface{}
userDataString := c.Params("userData")
configured := false
if userDataType == nil {
userData = userDataString
configured = true
} else if userDataString == "" {
userData = nil
} else {
var err error
if userData, err = decodeUserData(userDataString, userDataType, logger, userDataIsBase64); err != nil {
c.Status(fiber.StatusBadRequest)
return
if userDataString == "" {
if userDataType == nil {
userData = ""
} else {
userData = nil
}
} else {
configured = true
if userDataType == nil {
userData = userDataString
} else {
var err error
if userData, err = decodeUserData(userDataString, userDataType, logger, userDataIsBase64); err != nil {
c.Status(fiber.StatusBadRequest)
return
}
}
}
if manifestCallback != nil {
if status := manifestCallback(c.Context(), userData); status >= 400 {
Expand Down

0 comments on commit b6ed96e

Please sign in to comment.