yasithdev opened a new pull request, #191:
URL: https://github.com/apache/airavata-portals/pull/191

   ## Summary
   
   After the D5 migration of IAM / group / sharing to the gRPC `iam` facade, 
nothing reads `request.profile_service` anymore. This removes the now-dead 
Thrift plumbing:
   
   - the `profile_service_client` middleware (and its registration in 
`settings.MIDDLEWARE`)
   - the four profile Thrift client pools (group manager, IAM admin, tenant 
profile, user profile), their factory/getter functions, and the `*ThriftClient` 
connection classes in `utils.py`
   - the corresponding `airavata.service.profile.*` imports
   
   Net −149 lines of dead code.
   
   The airavata API Thrift client (`AiravataClientMiddleware` / 
`request.airavata_client`) **stays** for now: two admin-only per-protocol views 
still use it — `GlobusJobSubmissionView`'s pre-broken `getClo` and 
`UnicoreDataMovementView`'s missing facade getter. The airavata Thrift pool and 
the `airavata-python-sdk==2.2.7` dependency are removed in a later step once 
those two views are resolved.
   
   ## Validation
   
   - `manage.py check` clean.
   - The middleware chain loads with `profile_service_client` gone and 
`AiravataClientMiddleware` + the gRPC `airavata_grpc_client` middleware intact 
(verified the resolved `MIDDLEWARE` order and that the removed symbols no 
longer exist).
   - No remaining references to any removed symbol anywhere in 
`django_airavata`.


-- 
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]

Reply via email to