yasithdev opened a new pull request, #207: URL: https://github.com/apache/airavata-portals/pull/207
Track D endgame — the portal no longer imports any Thrift model type (`airavata.model.*.ttypes`) or the `thrift` library **anywhere**. ## What - **AuthzToken**: replace the Thrift security value type with a plain carrier in `apps/auth/utils.py` (same `accessToken` / `claimsMap` attributes, so the gRPC client factory + IAM REST helpers are unchanged); `token_authentication.py` imports it. - **exceptions.py**: drop the dead Thrift exception handlers (`AuthorizationException` / `ExperimentNotFoundException` / `TTransportException` / `TException`) + their imports — no Thrift call can raise them now; the gRPC `RpcError` handler covers the views. - **Delete `thrift_utils.py`** (the auto-serializer factory) — every serializer is proto-native, so nothing imports it. - **Test fixtures** (`apps/auth/tests/test_signals.py`, `apps/auth` + `apps/api` `tests/test_views.py`) build proto `UserProfile`/`GroupModel`/`GatewayGroups` with proto field names. `test_signals` now passes; the IAMUser/Group view tests remain **pre-existing** failures (they mock the long-removed `profile_service`/`airavata_client` Thrift surfaces — a separate gRPC-facade test rewrite). ## Validation - `manage.py check` green; the api test failure set is unchanged vs `origin/main` (same 6 pre-existing). - `grep` confirms zero `airavata.model` / `thrift` imports remain in the portal. The legacy `airavata-python-sdk==2.2.7` + `thrift` deps can be dropped once `airavata_django_portal_sdk` (experiment_util / queue_settings_calculators) is folded in — the final increment. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
