From a861f9c12625af37c2fa103a5b7c6871cbb9e868 Mon Sep 17 00:00:00 2001 From: Willy Kloucek Date: Thu, 19 May 2022 07:56:52 +0200 Subject: [PATCH] remove legacy accounts route from proxy default routes --- .../unreleased/fix-remove-legacy-accounts-routes.md | 8 ++++++++ extensions/proxy/pkg/config/defaults/defaultconfig.go | 10 ---------- 2 files changed, 8 insertions(+), 10 deletions(-) create mode 100644 changelog/unreleased/fix-remove-legacy-accounts-routes.md diff --git a/changelog/unreleased/fix-remove-legacy-accounts-routes.md b/changelog/unreleased/fix-remove-legacy-accounts-routes.md new file mode 100644 index 0000000000..f8aa3a0fa7 --- /dev/null +++ b/changelog/unreleased/fix-remove-legacy-accounts-routes.md @@ -0,0 +1,8 @@ +Bugfix: remove legacy accounts proxy routes + +We've removed the legacy accounts routes from the proxy default config. +There were no longer used since the switch to IDM as the default user +backend. Also accounts is no longer part of the oCIS binary and therefore +should not be part of the proxy default route config. + +https://github.com/owncloud/ocis/pull/3831 diff --git a/extensions/proxy/pkg/config/defaults/defaultconfig.go b/extensions/proxy/pkg/config/defaults/defaultconfig.go index 87e803d706..64934fb895 100644 --- a/extensions/proxy/pkg/config/defaults/defaultconfig.go +++ b/extensions/proxy/pkg/config/defaults/defaultconfig.go @@ -156,16 +156,6 @@ func DefaultPolicies() []config.Policy { Endpoint: "/graph-explorer", Backend: "http://localhost:9135", }, - // if we were using the go micro api gateway we could look up the endpoint in the registry dynamically - { - Endpoint: "/api/v0/accounts", - Backend: "http://localhost:9181", - }, - // TODO the lookup needs a better mechanism - { - Endpoint: "/accounts.js", - Backend: "http://localhost:9181", - }, { Endpoint: "/api/v0/settings", Backend: "http://localhost:9190",