user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download mbox.gz: |
* [PATCH 3/4] solver: join multiple URLs with "||"
  2020-02-01  9:12  6% [PATCH 0/4] flesh out multi-URL support Eric Wong
@ 2020-02-01  9:12  7% ` Eric Wong
  0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2020-02-01  9:12 UTC (permalink / raw)
  To: meta

It seems to make sense to the target audience that any of
the URLs displayed could work.
---
 lib/PublicInbox/SolverGit.pm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/PublicInbox/SolverGit.pm b/lib/PublicInbox/SolverGit.pm
index 06b4cdc4..bf806f65 100644
--- a/lib/PublicInbox/SolverGit.pm
+++ b/lib/PublicInbox/SolverGit.pm
@@ -530,7 +530,8 @@ sub resolve_patch ($$) {
 	if (my $existing = solve_existing($self, $want)) {
 		my ($found_git, undef, $type, undef) = @$existing;
 		dbg($self, "found $cur_want in " .
-			join("\n", $found_git->pub_urls($self->{psgi_env})));
+			join(" ||\n\t",
+				$found_git->pub_urls($self->{psgi_env})));
 
 		if ($cur_want eq $self->{oid_want} || $type ne 'blob') {
 			eval { done($self, $existing) };
@@ -548,7 +549,7 @@ sub resolve_patch ($$) {
 
 		unshift @{$self->{patches}}, @$diffs;
 		dbg($self, "found $cur_want in ".
-			join("\n\t", map { di_url($self, $_) } @$diffs));
+			join(" ||\n\t", map { di_url($self, $_) } @$diffs));
 
 		# good, we can find a path to the oid we $want, now
 		# lets see if we need to apply more patches:

^ permalink raw reply related	[relevance 7%]

* [PATCH 0/4] flesh out multi-URL support
@ 2020-02-01  9:12  6% Eric Wong
  2020-02-01  9:12  7% ` [PATCH 3/4] solver: join multiple URLs with "||" Eric Wong
  0 siblings, 1 reply; 2+ results
From: Eric Wong @ 2020-02-01  9:12 UTC (permalink / raw)
  To: meta

Better support for multiple URLs will hopefully discourage SPOFs
and give users more options when hosts go down.  It also gives
users and admins more agility when it comes to switching domain
names or having .onion mirrors.

Eric Wong (4):
  wwwtext: show multiple infourl values properly
  wwwtext: give "url" examples in sample config
  solver: join multiple URLs with "||"
  config: assume multiple cgit URLs, too

 lib/PublicInbox/Config.pm    |  8 ++++++--
 lib/PublicInbox/SolverGit.pm |  5 +++--
 lib/PublicInbox/WwwText.pm   | 24 +++++++++++++++---------
 3 files changed, 24 insertions(+), 13 deletions(-)

^ permalink raw reply	[relevance 6%]

Results 1-2 of 2 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2020-02-01  9:12  6% [PATCH 0/4] flesh out multi-URL support Eric Wong
2020-02-01  9:12  7% ` [PATCH 3/4] solver: join multiple URLs with "||" Eric Wong

Code repositories for project(s) associated with this public inbox

	https://80x24.org/public-inbox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).