From e8669f41d09b67ed93f8253c8d4fc179d6e915c0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20Sw=C3=A4rd?= Date: Wed, 19 Apr 2023 10:03:47 +0200 Subject: [PATCH] graph: Add comment to fix and remove expected test failures. --- services/graph/pkg/service/v0/graph.go | 2 ++ .../expected-failures-localAPI-on-OCIS-storage.md | 6 ------ 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/services/graph/pkg/service/v0/graph.go b/services/graph/pkg/service/v0/graph.go index e43214aacd0..f058984fee1 100644 --- a/services/graph/pkg/service/v0/graph.go +++ b/services/graph/pkg/service/v0/graph.go @@ -76,7 +76,9 @@ type Graph struct { // ServeHTTP implements the Service interface. func (g Graph) ServeHTTP(w http.ResponseWriter, r *http.Request) { + // Adding this as a fix for some weird unescaping issues we've had. r.URL.RawPath = r.URL.EscapedPath() + g.mux.ServeHTTP(w, r) } diff --git a/tests/acceptance/expected-failures-localAPI-on-OCIS-storage.md b/tests/acceptance/expected-failures-localAPI-on-OCIS-storage.md index 826e4273482..514f388f220 100644 --- a/tests/acceptance/expected-failures-localAPI-on-OCIS-storage.md +++ b/tests/acceptance/expected-failures-localAPI-on-OCIS-storage.md @@ -53,12 +53,6 @@ The expected failures in this file are from features in the owncloud/ocis repo. - [apiGraph/assignRole.feature:32](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/assignRole.feature#L32) - [apiGraph/assignRole.feature:33](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/assignRole.feature#L33) -### [Group having percentage (%) can be created but cannot be GET](https://github.com/owncloud/ocis/issues/5083) -- [apiGraph/deleteGroup.feature:49](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/deleteGroup.feature#L49) -- [apiGraph/deleteGroup.feature:50](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/deleteGroup.feature#L50) -- [apiGraph/deleteGroup.feature:51](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/deleteGroup.feature#L51) -- [apiGraph/editGroup.feature:25](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/editGroup.feature#L25) - #### [Share lists deleted user as 'user'](https://github.com/owncloud/ocis/issues/903) - [apiGraph/deleteGroup.feature:68](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/deleteGroup.feature#L68)