branchmispredictor

Okay, I've updated the PR to make the IAP middleware more transparent as requested. Now, all the frontend needs to do is call `/user/token` on refresh and it will get an IAP derived token if there is…

1 year ago

branchmispredictor pushed to feature/identity-aware-proxy at branchmispredictor/api

8840525b4a Make IAP middleware an optional follow-up to the JWT middleware
f75e8fa764 Fix lint
46845897d5 Merge branch 'main' into feature/identity-aware-proxy
440e0e2b27 Fix rebase errors
93dee49b25 Fix golint errors
Compare 12 commits »

1 year ago

branchmispredictor commented on pull request vikunja/api#715

Add support to login using identity from an identity-aware proxy

Following up just to make sure I capture the implicit decisions here, so you want for the backend: 1. If a valid jwt token is presented from the frontend, use that (e.g. ignore the IAP header) 2.…

1 year ago

branchmispredictor commented on pull request vikunja/api#715

Add support to login using identity from an identity-aware proxy

Okay, I'll give it a shot.

1 year ago

branchmispredictor commented on pull request vikunja/api#715

Add support to login using identity from an identity-aware proxy

Yes, or at least if not already logged in / if current jwt token is expired. Essentially it's similar to an open-id auth from the frontend's point of view except it 1) is hitting vikunja-backend…

1 year ago

branchmispredictor commented on pull request vikunja/api#715

Add support to login using identity from an identity-aware proxy

What I can do, is abtract this away a bit and make this an `/auth/externalprovider/loggedin` endpoint, so any future external auth source might also reuse the same endpoint.

1 year ago

branchmispredictor commented on pull request vikunja/api#715

Add support to login using identity from an identity-aware proxy

> Since you unified the external auth providers, could you add some docs about how to add a new auth provider in the future? > @branchmispredictor I'll give it another review, I just haven't…

1 year ago

branchmispredictor commented on pull request vikunja/api#715

Add support to login using identity from an identity-aware proxy

There's a problem with bootstrapping auth here. The way IAPs work is by setting an http header with claims to the downstream service (vikunja-api), however javascript or front-end code does not have…

1 year ago