From 68c5e269316fd0b3d2d487b1c1fda8c346cd28be Mon Sep 17 00:00:00 2001
From: Marcin Rataj <lidel@lidel.org>
Date: Fri, 22 Apr 2022 01:05:30 +0200
Subject: [PATCH] chore: remove Gateway.PathPrefixes

Closes #7702
---
 cmd/ipfs/daemon.go       |  2 +-
 config/gateway.go        | 20 +++-----------------
 core/corehttp/gateway.go |  2 --
 docs/config.md           | 35 +----------------------------------
 4 files changed, 5 insertions(+), 54 deletions(-)

diff --git a/cmd/ipfs/daemon.go b/cmd/ipfs/daemon.go
index 907bdabf3a9f..d37d6407d814 100644
--- a/cmd/ipfs/daemon.go
+++ b/cmd/ipfs/daemon.go
@@ -798,7 +798,7 @@ func serveHTTPGateway(req *cmds.Request, cctx *oldcmds.Context) (<-chan error, e
 	}
 
 	if len(cfg.Gateway.PathPrefixes) > 0 {
-		log.Error("Support for X-Ipfs-Gateway-Prefix and Gateway.PathPrefixes is deprecated and will be removed in the next release. Please comment on the issue if you're using this feature: https://github.com/ipfs/kubo/issues/7702")
+		log.Fatal("Support for custom Gateway.PathPrefixes was removed: https://github.com/ipfs/go-ipfs/issues/7702")
 	}
 
 	node, err := cctx.ConstructNode()
diff --git a/config/gateway.go b/config/gateway.go
index 574a7e3dfe8f..0073ec44bcfb 100644
--- a/config/gateway.go
+++ b/config/gateway.go
@@ -8,7 +8,7 @@ type GatewaySpec struct {
 	// UseSubdomains indicates whether or not this gateway uses subdomains
 	// for IPFS resources instead of paths. That is: http://CID.ipfs.GATEWAY/...
 	//
-	// If this flag is set, any /ipns/$id and/or /ipfs/$id paths in PathPrefixes
+	// If this flag is set, any /ipns/$id and/or /ipfs/$id paths in Paths
 	// will be permanently redirected to http://$id.[ipns|ipfs].$gateway/.
 	//
 	// We do not support using both paths and subdomains for a single domain
@@ -35,22 +35,8 @@ type Gateway struct {
 	// writing is done through the API, not the gateway.
 	Writable bool
 
-	// PathPrefixes  is an array of acceptable url paths that a client can
-	// specify in X-Ipfs-Path-Prefix header.
-	//
-	// The X-Ipfs-Path-Prefix header is used to specify a base path to prepend
-	// to links in directory listings and for trailing-slash redirects. It is
-	// intended to be set by a frontend http proxy like nginx.
-	//
-	// Example: To mount blog.ipfs.io (a DNSLink site) at ipfs.io/blog
-	// set PathPrefixes to ["/blog"] and nginx config to translate paths
-	// and pass Host header (for DNSLink):
-	//  location /blog/ {
-	//    rewrite "^/blog(/.*)$" $1 break;
-	//    proxy_set_header Host blog.ipfs.io;
-	//    proxy_set_header X-Ipfs-Gateway-Prefix /blog;
-	//    proxy_pass http://127.0.0.1:8080;
-	//  }
+	// PathPrefixes  was removed: https://github.com/ipfs/go-ipfs/issues/7702
+	// TODO: can we remove it from this struct?
 	PathPrefixes []string
 
 	// FastDirIndexThreshold is the maximum number of items in a directory
diff --git a/core/corehttp/gateway.go b/core/corehttp/gateway.go
index 2d300183ae8f..c2d5ea432691 100644
--- a/core/corehttp/gateway.go
+++ b/core/corehttp/gateway.go
@@ -18,7 +18,6 @@ import (
 type GatewayConfig struct {
 	Headers               map[string][]string
 	Writable              bool
-	PathPrefixes          []string
 	FastDirIndexThreshold int
 }
 
@@ -96,7 +95,6 @@ func GatewayOption(writable bool, paths ...string) ServeOption {
 		gateway, err = newGatewayHandler(GatewayConfig{
 			Headers:               headers,
 			Writable:              writable,
-			PathPrefixes:          cfg.Gateway.PathPrefixes,
 			FastDirIndexThreshold: int(cfg.Gateway.FastDirIndexThreshold.WithDefault(100)),
 		}, api)
 		if err != nil {
diff --git a/docs/config.md b/docs/config.md
index 37bbf45b140d..a034a097baf2 100644
--- a/docs/config.md
+++ b/docs/config.md
@@ -675,40 +675,7 @@ Type: `bool`
 
 ### `Gateway.PathPrefixes`
 
-**DEPRECATED:** see [kubo#7702](https://github.com/ipfs/kubo/issues/7702)
-
-<!--
-An array of acceptable url paths that a client can specify in X-Ipfs-Path-Prefix
-header.
-
-The X-Ipfs-Path-Prefix header is used to specify a base path to prepend to links
-in directory listings and for trailing-slash redirects. It is intended to be set
-by a frontend http proxy like nginx.
-
-Example: We mount `blog.ipfs.io` (a dnslink page) at `ipfs.io/blog`.
-
-**.ipfs/config**
-```json
-"Gateway": {
-  "PathPrefixes": ["/blog"],
-}
-```
-
-**nginx_ipfs.conf**
-```nginx
-location /blog/ {
-  rewrite "^/blog(/.*)$" $1 break;
-  proxy_set_header Host blog.ipfs.io;
-  proxy_set_header X-Ipfs-Gateway-Prefix /blog;
-  proxy_pass http://127.0.0.1:8080;
-}
-```
-
--->
-
-Default: `[]`
-
-Type: `array[string]`
+**REMOVED:** see [go-ipfs#7702](https://github.com/ipfs/go-ipfs/issues/7702)
 
 ### `Gateway.PublicGateways`