diff --git a/java/google/registry/env/common/backend/WEB-INF/web.xml b/java/google/registry/env/common/backend/WEB-INF/web.xml index 8eb70695d..18bcc43e7 100644 --- a/java/google/registry/env/common/backend/WEB-INF/web.xml +++ b/java/google/registry/env/common/backend/WEB-INF/web.xml @@ -284,16 +284,34 @@ Internal - Any request path starting with `/_dr/` will be restricted to requests originating - from the backend or by anyone authenticated to a Google account that's listed in - the AppEngine control panel settings for this project as a Viewer/Owner/Developer. - The `_dr` is short for domain registry to follow AppEngine naming conventions. + Admin-only internal section. Requests for paths covered by the URL patterns below will be + checked for a logged-in user account that's allowed to access the AppEngine admin console + (NOTE: this includes Editor/Viewer permissions in addition to Owner and the new IAM + App Engine Admin role. See https://cloud.google.com/appengine/docs/java/access-control + specifically the "Access handlers that have a login:admin restriction" line.) + + TODO(b/28219927): lift some of these restrictions so that we can allow OAuth authentication + for endpoints that need to be accessed by open-source automated processes. + + + /_ah/* + + /_dr/* + + + /registrar* + + + /assets/sources/* + admin + + CONFIDENTIAL @@ -313,39 +331,6 @@ - - - - Internal - - Go home rogue registrar! - - /registrar* - - - admin - - - CONFIDENTIAL - - - - - - - - Internal - No soup for you! - /assets/sources/* - - - admin - - - CONFIDENTIAL - - - ObjectifyFilter diff --git a/java/google/registry/env/common/default/WEB-INF/web.xml b/java/google/registry/env/common/default/WEB-INF/web.xml index 9e5840bb9..ddaa6d477 100644 --- a/java/google/registry/env/common/default/WEB-INF/web.xml +++ b/java/google/registry/env/common/default/WEB-INF/web.xml @@ -83,68 +83,58 @@ Internal - Any request path starting with `/_dr/` will be restricted to requests originating - from the backend or by anyone authenticated to a Google account that's listed in - the AppEngine control panel settings for this project as a Viewer/Owner/Developer. - The `_dr` is short for domain registry to follow AppEngine naming conventions. + Admin-only internal section. Requests for paths covered by the URL patterns below will be + checked for a logged-in user account that's allowed to access the AppEngine admin console + (NOTE: this includes Editor/Viewer permissions in addition to Owner and the new IAM + App Engine Admin role. See https://cloud.google.com/appengine/docs/java/access-control + specifically the "Access handlers that have a login:admin restriction" line.) + + TODO(b/28219927): lift some of these restrictions so that we can allow OAuth authentication + for endpoints that need to be accessed by open-source automated processes. + + + /_ah/* + + /_dr/* + + + /assets/sources/* + + + /assets/js/registrar_bin.js.map + /assets/js/registrar_dbg.js + /assets/js/brain_bin.js.map + /assets/css/registrar_dbg.css + admin + + + + CONFIDENTIAL + + - profile-registrar-xhr + Registrar console - Only allow logged-in users to even try to issue EPP commands. This is an additional - layer of safety on top of in-servlet authentication and XSRF protection. + Registrar console requires user login. This is in addition to the + code-level "requireLogin" configuration on individual @Actions. - /registrar-xhr - - - * - - - - - profile-registrar-settings - - Only allow logged-in users to even try to change registrar settings. This is an additional - layer of safety on top of in-servlet authentication and XSRF protection. - - /registrar-settings - - - * - - - - - profile-registrar-payment - - Only allow logged-in users to even try to change registrar settings. This is an additional - layer of safety on top of in-servlet authentication and XSRF protection. - - /registrar-payment - - - * - - - - - profile-registrar-payment-token - - Only allow logged-in users to even try to change registrar settings. This is an additional - layer of safety on top of in-servlet authentication and XSRF protection. - - /registrar-payment-token + /registrar* * + + + CONFIDENTIAL + @@ -161,61 +151,6 @@ - - - - - internal-sources - No soup for you! - /assets/sources/* - - - admin - - - - - - - registrar-bin-js-map - No soup for you! - /assets/js/registrar_bin.js.map - - - admin - - - - - registrar-dbg-js - No soup for you! - /assets/js/registrar_dbg.js - - - admin - - - - - brain-bin-js-map - No soup for you! - /assets/js/brain_bin.js.map - - - admin - - - - - registrar-dbg-css - No soup for you! - /assets/css/registrar_dbg.css - - - admin - - - ObjectifyFilter diff --git a/java/google/registry/env/common/tools/WEB-INF/web.xml b/java/google/registry/env/common/tools/WEB-INF/web.xml index ff4adc638..4c8b24382 100644 --- a/java/google/registry/env/common/tools/WEB-INF/web.xml +++ b/java/google/registry/env/common/tools/WEB-INF/web.xml @@ -128,16 +128,34 @@ Internal - Any request path starting with `/_dr/` will be restricted to requests originating - from the backend or by anyone authenticated to a Google account that's listed in - the AppEngine control panel settings for this project as a Viewer/Owner/Developer. - The `_dr` is short for domain registry to follow AppEngine naming conventions. - + Admin-only internal section. Requests for paths covered by the URL patterns below will be + checked for a logged-in user account that's allowed to access the AppEngine admin console + (NOTE: this includes Editor/Viewer permissions in addition to Owner and the new IAM + App Engine Admin role. See https://cloud.google.com/appengine/docs/java/access-control + specifically the "Access handlers that have a login:admin restriction" line.) + + TODO(b/28219927): lift some of these restrictions so that we can allow OAuth authentication + for endpoints that need to be accessed by open-source automated processes. + + + + /_ah/* + + /_dr/* + + + /registrar* + + + /assets/sources/* + admin + + CONFIDENTIAL @@ -157,39 +175,6 @@ - - - - Internal - - Go home rogue registrar! - - /registrar* - - - admin - - - CONFIDENTIAL - - - - - - - - Internal - No soup for you! - /assets/sources/* - - - admin - - - CONFIDENTIAL - - - ObjectifyFilter