Bug 1574792 - Bugzilla has suffered an internal error: Bugzilla cannot log you into...

diff --git a/Bugzilla/CGI.pm b/Bugzilla/CGI.pm
index e4245a86221695fccfc18f6f3ed236ba7b30a56f..dc279be46f9d772c7fa7d14380dbdba252cc64bc 100644 (file)
--- a/Bugzilla/CGI.pm
+++ b/Bugzilla/CGI.pm
@@ -123,21 +123,6 @@ sub new {
   return $self;
 }
 
-sub target_uri {
-  my ($self) = @_;
-
-  my $base = Bugzilla->localconfig->urlbase;
-  if (my $request_uri = $self->request_uri) {
-    my $base_uri = URI->new($base);
-    $base_uri->path('');
-    $base_uri->query(undef);
-    return $base_uri . $request_uri;
-  }
-  else {
-    return $base . ($self->url(-relative => 1, -query => 1) || 'index.cgi');
-  }
-}
-
 # We want this sorted plus the ability to exclude certain params
 sub canonicalize_query {
   my ($self, @exclude) = @_;

diff --git a/extensions/GitHubAuth/lib/Client.pm b/extensions/GitHubAuth/lib/Client.pm
index bc429f37962f536130cb8e60012dbc27b3864b63..a084b14443812386eb045ba8dce51a4993854252 100644 (file)
--- a/extensions/GitHubAuth/lib/Client.pm
+++ b/extensions/GitHubAuth/lib/Client.pm
@@ -37,14 +37,6 @@ sub new {
   return $self;
 }
 
-sub login_uri {
-  my ($class, $target_uri) = @_;
-
-  my $uri = URI->new(Bugzilla->localconfig->urlbase . "github.cgi");
-  $uri->query_form(target_uri => $target_uri);
-  return $uri;
-}
-
 sub authorize_uri {
   my ($class, $state) = @_;
 

diff --git a/extensions/GitHubAuth/template/en/default/hook/account/auth/login-additional_methods.html.tmpl b/extensions/GitHubAuth/template/en/default/hook/account/auth/login-additional_methods.html.tmpl
index f5e5ab11b261f86a37c95e63c6d376f9271bd693..8795d7a4a1361737785e4ea94a3c893d4f8a77aa 100644 (file)
--- a/extensions/GitHubAuth/template/en/default/hook/account/auth/login-additional_methods.html.tmpl
+++ b/extensions/GitHubAuth/template/en/default/hook/account/auth/login-additional_methods.html.tmpl
@@ -9,7 +9,7 @@
 [% IF Param('user_info_class').split(',').contains('GitHubAuth') %]
   <form method="post" action="[% basepath FILTER html %]github.cgi">
     <input type="hidden" name="github_secret" value="[% Bugzilla.github_secret FILTER html %]">
-    <input type="hidden" name="target_uri" value="[% Bugzilla.cgi.target_uri FILTER html %]">
+    <input type="hidden" name="target_uri" value="[% urlbase FILTER html %]index.cgi">
     <input type="image" src="[% basepath FILTER none %]extensions/GitHubAuth/web/images/github_sign_in.png"
            alt="Sign in with GitHub"
            title="Sign in with GitHub"

diff --git a/extensions/GitHubAuth/template/en/default/hook/account/auth/login-small-additional_methods.html.tmpl b/extensions/GitHubAuth/template/en/default/hook/account/auth/login-small-additional_methods.html.tmpl
index ba69911758d3bc44b2694e13a8ce0487a918f0cf..fb1d6866b88f10b7794aa01d935fb9a95a74aadd 100644 (file)
--- a/extensions/GitHubAuth/template/en/default/hook/account/auth/login-small-additional_methods.html.tmpl
+++ b/extensions/GitHubAuth/template/en/default/hook/account/auth/login-small-additional_methods.html.tmpl
@@ -11,7 +11,7 @@
   <span id="github_mini_login[% qs_suffix FILTER html %]" class="mini_login[% qs_suffix FILTER html %]">
     <form method="post" action="[% basepath FILTER html %]github.cgi">
       <input type="hidden" name="github_secret" value="[% Bugzilla.github_secret FILTER html %]">
-      <input type="hidden" name="target_uri" value="[% Bugzilla.cgi.target_uri FILTER html %]">
+      <input type="hidden" name="target_uri" value="[% urlbase FILTER html %]index.cgi">
       <input type="image" src="[% basepath FILTER none %]extensions/GitHubAuth/web/images/sign_in.png" height="22" width="75" align="absmiddle"
              alt="Sign in with GitHub"
              title="Sign in with GitHub"> or