[Fix] Encode uri for use in blink key (#209)

dnwjn · web-flow · commit 00ff4116da0f · 2023-10-10T07:56:19.000+01:00
diff --git a/src/Entries/EntryRepository.php b/src/Entries/EntryRepository.php
@@ -35,7 +35,7 @@ public function find($id): ?EntryContract
 
     public function findByUri(string $uri, string $site = null): ?EntryContract
     {
-        $blinkKey = "eloquent-entry-{$uri}".($site ? '-'.$site : '');
+        $blinkKey = 'eloquent-entry-'.md5(urlencode($uri)).($site ? '-'.$site : '');
         $item = Blink::once($blinkKey, function () use ($uri, $site) {
             return parent::findByUri($uri, $site);
         });
diff --git a/src/Taxonomies/TermRepository.php b/src/Taxonomies/TermRepository.php
@@ -73,7 +73,7 @@ public function findByUri(string $uri, string $site = null): ?TermContract
             return null;
         }
 
-        $blinkKey = "eloquent-term-{$uri}".($site ? '-'.$site : '');
+        $blinkKey = 'eloquent-term-'.md5(urlencode($uri)).($site ? '-'.$site : '');
         $term = Blink::once($blinkKey, function () use ($slug, $taxonomy) {
             return $this->query()
                 ->where('slug', $slug)

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ public function find($id): ?EntryContract`
`35`	`35`
`36`	`36`	`public function findByUri(string $uri, string $site = null): ?EntryContract`
`37`	`37`	`{`
`38`		`- $blinkKey = "eloquent-entry-{$uri}".($site ? '-'.$site : '');`
	`38`	`+ $blinkKey = 'eloquent-entry-'.md5(urlencode($uri)).($site ? '-'.$site : '');`
`39`	`39`	`$item = Blink::once($blinkKey, function () use ($uri, $site) {`
`40`	`40`	`return parent::findByUri($uri, $site);`
`41`	`41`	`});`
Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ public function findByUri(string $uri, string $site = null): ?TermContract`
`73`	`73`	`return null;`
`74`	`74`	`}`
`75`	`75`
`76`		`- $blinkKey = "eloquent-term-{$uri}".($site ? '-'.$site : '');`
	`76`	`+ $blinkKey = 'eloquent-term-'.md5(urlencode($uri)).($site ? '-'.$site : '');`
`77`	`77`	`$term = Blink::once($blinkKey, function () use ($slug, $taxonomy) {`
`78`	`78`	`return $this->query()`
`79`	`79`	`->where('slug', $slug)`