entitlements v2 cutover
Merge request reports
Activity
added 2 commits
added 9 commits
-
55ab224a...cbf398f0 - 8 commits from branch
master
- 26c38230 - Merge branch 'master' of...
-
55ab224a...cbf398f0 - 8 commits from branch
@ethiraj @komakkar @harshit283 Pipeline is green now. Can you please review/comment/approve ?
- Resolved by Neelesh Thakur
- Resolved by Neelesh Thakur
- Resolved by Neelesh Thakur
- Resolved by Neelesh Thakur
- Resolved by Neelesh Thakur
- Resolved by Neelesh Thakur
- Resolved by Neelesh Thakur
- Resolved by Neelesh Thakur
23 import org.opengroup.osdu.core.common.logging.JaxRsDpsLog; 24 import org.opengroup.osdu.core.common.model.entitlements.AuthorizationResponse; 25 import org.opengroup.osdu.core.common.model.entitlements.EntitlementsException; 26 import org.opengroup.osdu.core.common.model.entitlements.GroupInfo; 27 import org.opengroup.osdu.core.common.model.entitlements.Groups; 28 import org.opengroup.osdu.core.common.model.http.AppException; 29 import org.opengroup.osdu.core.common.model.http.DpsHeaders; 30 import org.opengroup.osdu.core.common.provider.interfaces.IAuthorizationService; 31 import org.springframework.context.annotation.Lazy; 32 import org.springframework.context.annotation.Primary; 33 import org.springframework.stereotype.Service; 34 35 import javax.inject.Inject; 36 import java.util.ArrayList; 37 import java.util.List; 38 import java.util.stream.Collectors; @nthakur can you help me understand why we are writing this new code to interface with entitlements v2. The new service has the same getgroups API interface as V1, so I am not understanding why this is needed. Can you help me understand? We were expecting the only changes needed to be (1) updated os-core-common version, (2) update deployment.yaml with new endpoints and (3) update os-core-common libraries used to entitlements v2 where relevant.
changed this line in version 9 of the diff
Please register or sign in to reply