Add API endpoint to revoke PATs

Adds the DELETE /personal_access_tokens/:id
REST endpoint for users to revoke their own
PATs or admins to do it on their behalf

doc/api/personal_access_tokens.md

@@ -4,7 +4,7 @@ You can read more about [personal access tokens](../user/profile/personal_access
 
 ## List personal access tokens
 
-> [Introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/22726) in [GitLab Ultimate](https://about.gitlab.com/pricing/) 13.3.
+> [Introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/227264) in [GitLab Ultimate](https://about.gitlab.com/pricing/) 13.3.
 
 Get a list of personal access tokens.
 
@@ -60,3 +60,29 @@ curl --header "PRIVATE-TOKEN: <your_access_token>" "https://gitlab.example.com/a
     }
 ]
 ```
+
+## Revoke a personal access token
+
+> [Introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/216004) in [GitLab Ultimate](https://about.gitlab.com/pricing/) 13.3.
+
+Revoke a personal access token.
+
+```plaintext
+DELETE /personal_access_tokens/:id
+```
+
+| Attribute | Type    | required | Description         |
+|-----------|---------|----------|---------------------|
+| `id` | integer/string | yes | ID of personal access token |
+
+NOTE: **Note:**
+Non-administrators can revoke their own tokens. Administrators can revoke tokens of any user.
+
+```shell
+curl --request DELETE --header "PRIVATE-TOKEN: <your_access_token>" "https://gitlab.example.com/api/v4/personal_access_tokens/<personal_access_token_id>"
+```
+
+### Responses
+
+- `204: No Content` if successfully revoked.
+- `400 Bad Request` if not revoked successfully.

ee/changelogs/unreleased/216004-revoke-pats-api.yml

+---
+title: Add API endpoint to revoke PATs
+merge_request: 39072
+author:
+type: added

ee/lib/api/personal_access_tokens.rb

@@ -34,16 +34,31 @@ module API
         unauthorized! unless Ability.allowed?(current_user, :read_user_personal_access_tokens, user(params[:user_id]))
       end
 
+      def find_token(id)
+        PersonalAccessToken.find(id) || not_found!
+      end
+
       def authenticate!
         unauthorized! unless ::License.feature_available?(:personal_access_token_api_management)
         super
       end
     end
 
-    get :personal_access_tokens do
+    resources :personal_access_tokens do
+      get do
         tokens = PersonalAccessTokensFinder.new(finder_params(current_user), current_user).execute
 
         present paginate(tokens), with: Entities::PersonalAccessToken
       end
+
+      delete ':id' do
+        service = ::PersonalAccessTokens::RevokeService.new(
+          current_user,
+          { token: find_token(params[:id]) }
+        ).execute
+
+        service.success? ? no_content! : bad_request!(nil)
+      end
+    end
   end
 end

ee/spec/requests/api/personal_access_tokens_spec.rb

@@ -8,6 +8,7 @@ RSpec.describe API::PersonalAccessTokens do
   let_it_be(:token2) { create(:personal_access_token) }
   let_it_be(:current_user) { create(:user) }
 
+  describe 'GET /personal_access_tokens' do
     context 'when unlicensed' do
       before do
         stub_licensed_features(personal_access_token_api_management: false)
@@ -85,4 +86,63 @@ RSpec.describe API::PersonalAccessTokens do
         end
       end
     end
+  end
+
+  describe 'DELETE /personal_access_tokens/:id' do
+    let(:path) { "/personal_access_tokens/#{token1.id}" }
+
+    context 'when unlicensed' do
+      before do
+        stub_licensed_features(personal_access_token_api_management: false)
+      end
+
+      it 'responds with unauthorized' do
+        delete api(path, current_user)
+
+        expect(response).to have_gitlab_http_status(:unauthorized)
+      end
+    end
+
+    context 'when licensed' do
+      before do
+        stub_licensed_features(personal_access_token_api_management: true)
+      end
+
+      context 'when current_user is an administrator', :enable_admin_mode do
+        let_it_be(:admin_user) { create(:admin) }
+        let_it_be(:admin_token) { create(:personal_access_token, user: admin_user) }
+        let_it_be(:admin_path) { "/personal_access_tokens/#{admin_token.id}" }
+
+        it 'revokes a different users token' do
+          delete api(path, admin_user)
+
+          expect(response).to have_gitlab_http_status(:no_content)
+          expect(token1.reload.revoked?).to be true
+        end
+
+        it 'revokes their own token' do
+          delete api(admin_path, admin_user)
+
+          expect(response).to have_gitlab_http_status(:no_content)
+        end
+      end
+
+      context 'when current_user is not an administrator' do
+        let_it_be(:user_token) { create(:personal_access_token, user: current_user) }
+        let_it_be(:user_token_path) { "/personal_access_tokens/#{user_token.id}" }
+
+        it 'fails revokes a different users token' do
+          delete api(path, current_user)
+
+          expect(response).to have_gitlab_http_status(:bad_request)
+        end
+
+        it 'revokes their own token' do
+          delete api(user_token_path, current_user)
+
+          expect(response).to have_gitlab_http_status(:no_content)
+        end
+      end
+    end
+  end
 end