repobrowse: switch to new URL format to avoid query strings

Query strings make endpoint caching more difficult since
they're order-independent.  They are also more likely lost
or truncated inadvertantly when copy+pasting, so try to
avoid them for default endpoints.

There's still some things which are broken and followup
commits will be needed to fix them.

1 files changed, 1 insertions, 4 deletions

diff --git a/lib/PublicInbox/RepoGitDiff.pm b/lib/PublicInbox/RepoGitDiff.pm
index bb71e738..ef4717ac 100644
--- a/lib/PublicInbox/RepoGitDiff.pm
+++ b/lib/PublicInbox/RepoGitDiff.pm
@@ -34,11 +34,8 @@ sub git_diff_sed ($$) {
 
 sub call_git_diff {
         my ($self, $req) = @_;
+        my ($id, $id2) = split(/\.\./, $req->{h});
         my $env = $req->{env};
-        my $q = PublicInbox::RepoGitQuery->new($env);
-        my $id = $q->{id};
-        my $id2 = $q->{id2};
-
         my $git = $req->{repo_info}->{git};
         my $cmd = $git->cmd(qw(diff-tree -z --numstat -p --encoding=UTF-8
                                 --no-color -M -B -D -r), $id2, $id, '--');