Page MenuHomeSoftware Heritage
Differential D3996

auth: Add web UI for API bearer tokens management
ClosedPublic
Actions

Authored by anlambert on Sep 18 2020, 4:59 PM.

Details

Reviewers
ardumont
Group Reviewers
Reviewers
Maniphest Tasks
T2569: Add Web UI to generate and revoke bearer tokens
Commits
rDWAPPSc45a6af2f7e0: auth: Add web UI for API bearer tokens management
Summary

Add HTML view reachable from the /api/tokens URL enabling a logged in user
to generate, display or revoke bearer tokens for Web API authentication.

Once generated, each token will be encrypted based on user password and
stored to the webapp database.

Each token related action is also protected by the user password.

Below are some screenshots of the result:

Closes T2569

Diff Detail

Repository
rDWAPPS Web applications
Branch
api-tokens-ui
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 15353
Build 23646: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 23645: arc lint + arc unit

Event Timeline

anlambert created this revision.Sep 18 2020, 4:59 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptSep 18 2020, 4:59 PM
swh-public-ci added a comment.Sep 18 2020, 5:07 PM

Build has FAILED

Patch application report for D3996 (id=14081)

Rebasing onto 626b1e5b75...

Current branch diff-target is up to date.
Changes applied before test
commit 0757f01b038f12d623818caf9534d6dacd0eea9c
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Fri Sep 4 14:51:41 2020 +0200

    auth: Add web UI for API bearer tokens management
    
    Add HTML view reachable from the /api/tokens URL enabling a logged in user
    to generate, display or revoke bearer tokens for Web API authentication.
    
    Once generated, each token will be encrypted based on user password and
    stored to the webapp database.
    
    Each token related action is also protected by the user password.
    
    Closes T2569

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/376/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/376/console

Harbormaster failed remote builds in B15348: Diff 14081!Sep 18 2020, 5:07 PM
anlambert updated this revision to Diff 14086.Sep 18 2020, 5:21 PM

Update: Fix cypress tests

swh-public-ci added a comment.Sep 18 2020, 5:28 PM

Build is green

Patch application report for D3996 (id=14086)

Rebasing onto 626b1e5b75...

Current branch diff-target is up to date.
Changes applied before test
commit c7aa4b2f3d374e1161884ddf29db262d52569831
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Fri Sep 4 14:51:41 2020 +0200

    auth: Add web UI for API bearer tokens management
    
    Add HTML view reachable from the /api/tokens URL enabling a logged in user
    to generate, display or revoke bearer tokens for Web API authentication.
    
    Once generated, each token will be encrypted based on user password and
    stored to the webapp database.
    
    Each token related action is also protected by the user password.
    
    Closes T2569

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/377/ for more details.

Harbormaster completed remote builds in B15353: Diff 14086.Sep 18 2020, 5:28 PM
ardumont accepted this revision.Sep 21 2020, 5:46 PM
ardumont added a subscriber: ardumont.

lgtm

some questions/remarks inline.

cypress/integration/api-tokens.spec.js
84

Is there some cypress magic behing that setup call?

swh/web/auth/apps.py
9

is the Swh prefix necessary (we are already in an swh module)?

swh/web/tests/auth/test_views.py
343

"anonymous user with credentials should be refused access with forbidden response"

360

"user with correct credentials should be allowed access"...

372

"user with wrong credentials should be refused access with 401 invalid response" (or something ;)

382

"anonymous access should be refused with forbidden response"

413

"anonymous access should be refused with forbidden response"

This revision is now accepted and ready to land.Sep 21 2020, 5:46 PM
ardumont added inline comments.Sep 21 2020, 5:47 PM
swh/web/auth/utils.py
51

for the reference: https://cryptography.io/en/latest/fernet/

anlambert added a comment.Sep 22 2020, 10:48 AM

@ardumont, thanks for the review. I will update the diff according to your comments before landing this.

cypress/integration/api-tokens.spec.js
84

not really, it simply tests the UI and mock the generate token request

swh/web/auth/apps.py
9

Right, it can be removed

anlambert updated this revision to Diff 14116.Sep 22 2020, 11:03 AM

Update: Address @ardumont comments

swh-public-ci added a comment.Sep 22 2020, 11:10 AM

Build is green

Patch application report for D3996 (id=14116)

Rebasing onto 626b1e5b75...

Current branch diff-target is up to date.
Changes applied before test
commit c45a6af2f7e0805bca99c6c0661931cee87cd1c8
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Fri Sep 4 14:51:41 2020 +0200

    auth: Add web UI for API bearer tokens management
    
    Add HTML view reachable from the /api/tokens URL enabling a logged in user
    to generate, display or revoke bearer tokens for Web API authentication.
    
    Once generated, each token will be encrypted based on user password and
    stored to the webapp database.
    
    Each token related action is also protected by the user password.
    
    Closes T2569

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/378/ for more details.

Harbormaster completed remote builds in B15382: Diff 14116.Sep 22 2020, 11:10 AM
Closed by commit rDWAPPSc45a6af2f7e0: auth: Add web UI for API bearer tokens management (authored by anlambert). · Explain WhySep 22 2020, 12:21 PM
This revision was automatically updated to reflect the committed changes.
anlambert added a commit: rDWAPPSc45a6af2f7e0: auth: Add web UI for API bearer tokens management.

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)
cypress/
integration/
211 lines--
1 line--
swh/
web/
api/
11 lines88%
Loading...
assets/
src/
bundles/
auth/
21 lines--
216 lines85%
Loading...
auth/
6 lines100%
Loading...
11 lines100%
Loading...
23 lines87%
Loading...
migrations/
44 lines100%
Loading...
migrations/
--
15 lines94%
Loading...
61 lines100%
Loading...
131 lines92%
Loading...
settings/
1 line94%
Loading...
templates/
api/
57 lines--
tests/
auth/
9 lines100%
Loading...
31 lines100%
Loading...
164 lines100%
Loading...

Diff 14086

View Options

cypress/integration/api-tokens.spec.js

Loading...
View Options

requirements.txt

Loading...
View Options

swh/web/api/urls.py

Loading...
View Options

swh/web/assets/src/bundles/auth/auth.css

Loading...
View Options

swh/web/assets/src/bundles/auth/index.js

Loading...
View Options

swh/web/auth/__init__.py

Loading...
View Options

swh/web/auth/apps.py

Loading...
View Options

swh/web/auth/keycloak.py

Loading...
View Options

swh/web/auth/migrations/0001_initial.py

Loading...
View Options

swh/web/auth/migrations/__init__.py

Loading...
View Options

swh/web/auth/models.py

Loading...
View Options

swh/web/auth/utils.py

Loading...
View Options

swh/web/auth/views.py

Loading...
View Options

swh/web/settings/common.py

Loading...
View Options

swh/web/templates/api/tokens.html

Loading...
View Options

swh/web/tests/auth/keycloak_mock.py

Loading...
View Options

swh/web/tests/auth/test_utils.py

Loading...
View Options

swh/web/tests/auth/test_views.py

Loading...
About · Developer information · Legal