apps: log viewmode in the returning app URL message

glpatcern · glpatcern · commit 8028fa0f0ec7 · 2025-02-19T17:25:15.000+01:00
diff --git a/changelog/unreleased/fix-app-log.md b/changelog/unreleased/fix-app-log.md
@@ -0,0 +1,3 @@
+Enhancement: log viewmode in the returning app URL message
+
+https://github.com/cs3org/reva/pull/5080
diff --git a/internal/http/services/appprovider/appprovider.go b/internal/http/services/appprovider/appprovider.go
@@ -435,7 +435,7 @@ func (s *svc) handleOpen(w http.ResponseWriter, r *http.Request) {
 	}
 
 	log := appctx.GetLogger(ctx)
-	log.Info().Interface("resource", fileRef).Str("url", openRes.AppUrl.AppUrl).Str("method", openRes.AppUrl.Method).Interface("target", openRes.AppUrl.Target).Msg("returning app URL for file")
+	log.Info().Interface("resource", fileRef).Str("url", openRes.AppUrl.AppUrl).Str("method", openRes.AppUrl.Method).Interface("viewMode", viewMode).Msg("returning app URL for file")
 
 	w.Header().Set("Content-Type", "application/json")
 	if _, err = w.Write(js); err != nil {

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+Enhancement: log viewmode in the returning app URL message`
	`2`	`+`
	`3`	`+https://github.com/cs3org/reva/pull/5080`
Original file line number	Diff line number	Diff line change
`@@ -435,7 +435,7 @@ func (s svc) handleOpen(w http.ResponseWriter, r http.Request) {`
`435`	`435`	`}`
`436`	`436`
`437`	`437`	`log := appctx.GetLogger(ctx)`
`438`		`- log.Info().Interface("resource", fileRef).Str("url", openRes.AppUrl.AppUrl).Str("method", openRes.AppUrl.Method).Interface("target", openRes.AppUrl.Target).Msg("returning app URL for file")`
	`438`	`+ log.Info().Interface("resource", fileRef).Str("url", openRes.AppUrl.AppUrl).Str("method", openRes.AppUrl.Method).Interface("viewMode", viewMode).Msg("returning app URL for file")`
`439`	`439`
`440`	`440`	`w.Header().Set("Content-Type", "application/json")`
`441`	`441`	`if _, err = w.Write(js); err != nil {`