Merge pull request metacpan#2566 from metacpan/haarg/remove-openid

remove openid login support

Author: haarg

lib/MetaCPAN/Web/Controller/Login.pm

@@ -31,19 +31,6 @@ sub index : Chained('login_root') : PathPart('') : Args(0) {
     }
 }
 
-sub openid : Chained('login_root') : PathPart : Args(0) {
-    my ( $self, $c ) = @_;
-
-    # Never cache at CDN
-    $c->cdn_never_cache(1);
-
-    my $model = $c->model('API::User');
-    $c->stash( {
-        consumer_key => $model->consumer_key,
-        openid_url   => $model->openid_url,
-    } );
-}
-
 sub pause : Chained('login_root') : PathPart : Args(0) {
     my ( $self, $c ) = @_;
 

lib/MetaCPAN/Web/Model/API/User.pm

@@ -23,11 +23,6 @@ sub COMPONENT {
     return $class->SUPER::COMPONENT( $app, $args );
 }
 
-sub openid_url {
-    my ($self) = @_;
-    return $self->api_public . '/login/openid';
-}
-
 sub oauth_url {
     my ($self) = @_;
     return $self->api_public . '/oauth2/authorize';

root/account/identities.tx

@@ -6,7 +6,7 @@
       Identities you are connected to allow you to log into MetaCPAN. MetaCPAN also drags in some information from them to help you with filling in your <a href="/account/profile">profile</a>. This information is not exposed to anyone unless you explicitly publish it.
     </div>
     <table width="300">
-%%  for ['GitHub', 'PAUSE', 'Twitter', 'Google', 'OpenID'] -> $identity {
+%%  for ['GitHub', 'PAUSE', 'Twitter', 'Google'] -> $identity {
 %%  my $connected = !!$user.identity[$identity.lc()];
     <tr>
         <td><big>[% $identity %]</big></td>

root/base.tx

@@ -146,7 +146,7 @@
                         <b class="caret"></b>
                         </a>
                         <ul class="dropdown-menu">
-                            %%  for ['GitHub', 'Twitter', 'Google', 'OpenID'] -> $identity {
+                            %%  for ['GitHub', 'Twitter', 'Google'] -> $identity {
                             <li>
                                 <a href="/login/[% $identity.lc() %]">
                                     <i class="fab fa-[% $identity.lc() %] fa-fw"></i>