extend app /notify endpoint to allow reporting errors (#5085)

Author: glpatcern

changelog/unreleased/app-notify.md

@@ -0,0 +1,3 @@
+Enhancement: extend app /notify endpoint to allow reporting errors
+
+https://github.com/cs3org/reva/pull/5085

internal/http/services/appprovider/appprovider.go

@@ -22,6 +22,7 @@ import (
 	"context"
 	"crypto/tls"
 	"encoding/json"
+	"io"
 	"net/http"
 	"path"
 
@@ -469,10 +470,17 @@ func (s *svc) handleNotify(w http.ResponseWriter, r *http.Request) {
 		fileRef.ResourceId = resourceID
 	}
 
+	// the body of the request may contain any error the client got when attempting to open the app
+	failure, _ := io.ReadAll(r.Body)
+
 	// log the fileid for later correlation / monitoring
 	ctx := r.Context()
 	log := appctx.GetLogger(ctx)
-	log.Info().Interface("resource", fileRef).Msg("file successfully opened in app")
+	if len(failure) == 0 {
+		log.Info().Interface("resource", fileRef).Msg("file successfully opened in app")
+	} else {
+		log.Info().Interface("resource", fileRef).Str("failure", string(failure)).Msg("failed to open file in app")
+	}
 
 	w.WriteHeader(http.StatusOK)
 }