hugo

commit 4c8552b11477141777101e0e0609dd1f32d191e9
parent 34e4742f0caab0d3eb9efd00fce4157d112617b5
Author: Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Date:   Wed, 30 Jun 2021 12:55:29 +0200

Fix Cloudflare vs Netlify cache dir issue

Re-add the additional environment checks to determine if its Netlify. Seems that Cloudflare also sets `NETLIFY=true`.

This makes it look, basically, like a variant of the conditional we had before we started fixing this, but I have checked this logic on Netlify now and it should work.

Fixes #8714

Diffstat:
M
helpers/path.go
 | 
6
++++--

1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/helpers/path.go b/helpers/path.go
@@ -412,8 +412,10 @@ func getCacheDir(cfg config.Provider) string {
 		return addTrailingFileSeparator(cacheDir)
 	}
 
-	// This is always set to true when running on Netlify.
-	if os.Getenv("NETLIFY") == "true" {
+	// See Issue #8714.
+	// Turns out that Cloudflare also sets NETLIFY=true in its build environment,
+	// but all of these 3 should not give any false positives.
+	if os.Getenv("NETLIFY") == "true" && os.Getenv("PULL_REQUEST") != "" && os.Getenv("DEPLOY_PRIME_URL") != "" {
 		// Netlify's cache behaviour is not documented, the currently best example
 		// is this project:
 		// https://github.com/philhawksworth/content-shards/blob/master/gulpfile.js