entitlements-azure merge requestshttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests2020-05-06T15:06:21Zhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/1Added Capability and Readme instructions for manual deployment of service #12020-05-06T15:06:21ZDaniel SchollAdded Capability and Readme instructions for manual deployment of service #1## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull-requests/code-reviews/readme.md)?
* [YES] Have ...## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull-requests/code-reviews/readme.md)?
* [YES] Have you added an explanation of what your changes do and why you'd like us to include them?
* [YES] I have updated the documentation accordingly.
* [NA] I have added tests to cover my changes.
* [NA] All new and existing tests passed.
* [NA] My code follows the code style of this project.
* [NA] I ran lint checks locally prior to submission.
## What is the current behavior?
-------------------------------------
Currently Pipelines are configured to support ADO Pipelines and needs to be modified to support GitLab CI/CD for Continuous Delivery
## What is the new behavior?
-------------------------------------
- Modified the settings.xml file to support the Maven Plugin for Azure Web App Deploy
- Documented and tested a manual deploy process to prepare for the CI/CD Automated Process
## Does this introduce a breaking change?
-------------------------------------
- [NO]
## Any relevant logs, error output, etc?
-------------------------------------
## Other information
-------------------------------------Daniel SchollDaniel Schollhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/2Azure Pipeline Build Support2020-05-11T14:00:39ZDaniel SchollAzure Pipeline Build Support## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull-requests/code-reviews/readme.md)?
* [YES] Have ...## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull-requests/code-reviews/readme.md)?
* [YES] Have you added an explanation of what your changes do and why you'd like us to include them?
* [YES] I have updated the documentation accordingly.
* [NA] I have added tests to cover my changes.
* [YES] All new and existing tests passed.
* [YES] My code follows the code style of this project.
* [NA] I ran lint checks locally prior to submission.
## What is the current behavior?
-------------------------------------
This pull request is to update the pipeline file and supporting documentation necessary to support the deployment of this service into Azure.
This PR has a dependency upon a PR submission in CI/CD Pipelines where the Pipeline template for azure is maintained.
## What is the new behavior?
-------------------------------------
- Documentation Updates
- Pull Request Template
- Additional gitignore items due to adding .envrc support in README
## Does this introduce a breaking change?
-------------------------------------
- [NO]
## Any relevant logs, error output, etc?
-------------------------------------
NA
## Other information
NoneDaniel SchollDaniel Schollhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/3Azure2020-05-20T01:46:17ZDaniel SchollAzureThis merge request enables Pipeline Integration with Azure Cloud Integration Environment.This merge request enables Pipeline Integration with Azure Cloud Integration Environment.Daniel SchollDaniel Schollhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/4Trusted azure2020-05-22T22:28:50ZDaniel SchollTrusted azureRename azure-pipelines to azure-pipeline for consistency purposes.Rename azure-pipelines to azure-pipeline for consistency purposes.Daniel SchollDaniel Schollhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/5Fix notice2020-05-25T14:39:15ZDavid Diederichd.diederich@opengroup.orgFix noticeThis updates the NOTICE file that was failing from master.
The change to the "unknown" sections was likely due to some manual overrides I put in the FOSSA tool for Byte Buddy -- FOSSA was picking up some non-license files and interpreti...This updates the NOTICE file that was failing from master.
The change to the "unknown" sections was likely due to some manual overrides I put in the FOSSA tool for Byte Buddy -- FOSSA was picking up some non-license files and interpreting them incorrectly. The attribution is still present, so I believe this is fine, and now the tools will acknowledge it.
Also, I used this branch as a pilot test case for https://community.opengroup.org/osdu/platform/ci-cd-pipelines/-/merge_requests/10, which worked well.Daniel SchollDaniel Schollhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/6Azure Development Pipelines added2020-06-04T16:28:42ZDaniel SchollAzure Development Pipelines addedThis pull request adds in a Pipeline to be used for Microsoft Azure Development Work.This pull request adds in a Pipeline to be used for Microsoft Azure Development Work.Daniel SchollDaniel Schollhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/7Development Pipeline Updated2020-06-04T20:21:39ZDaniel SchollDevelopment Pipeline UpdatedDaniel SchollDaniel Schollhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/8Fixed License Copyright Symbol2020-06-04T21:48:04ZDaniel SchollFixed License Copyright SymbolThis is a change of the copyright symbol in order to test a developer workflow.This is a change of the copyright symbol in order to test a developer workflow.Daniel SchollDaniel Schollhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/9Eleckner/us/21812020-06-26T21:12:13ZChristian LecknerEleckner/us/2181## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull-requests/code-reviews/readme.md)?
* [YES] Have ...## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull-requests/code-reviews/readme.md)?
* [YES] Have you added an explanation of what your changes do and why you'd like us to include them?
* [YES] I have updated the documentation accordingly.
* [YES] I have added tests to cover my changes.
* [YES] All new and existing tests passed.
* [YES] My code follows the code style of this project.
* [NA] I ran lint checks locally prior to submission.
## What is the current behavior?
-------------------------------------
2181 Use OID for HTTP POST /entitlements/v1/groups/{group_email}/members for USERS
2763 Entitlement service implementation
2626 Merge entitlement service changes to Gitlab
1678 Entitlement Service CRUD Operations- Delete User Profile Operation
1679 Entitlement Service CRUD Operations- Update User Profile Operation
2581 Unit and integration tests for entitlement service APIs
## What is the new behavior?
-------------------------------------
See above
## Does this introduce a breaking change?
-------------------------------------
No
## Any relevant logs, error output, etc?
-------------------------------------
N./A
## Other information
-------------------------------------Christian LecknerChristian Lecknerhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/10Azure - fix error code when create a existed profile2020-07-10T20:09:02ZJohnny Guojguo26@slb.comAzure - fix error code when create a existed profile**This impacts only Azure Implementation**
## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull...**This impacts only Azure Implementation**
## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull-requests/code-reviews/readme.md)?
* [YES] Have you added an explanation of what your changes do and why you'd like us to include them?
* [YES] I have updated the documentation accordingly.
* [NA] I have added tests to cover my changes.
* [YES] All new and existing tests passed.
* [YES] My code follows the code style of this project.
* [NA] I ran lint checks locally prior to submission.
## What is the current behavior?
-------------------------------------
<!-- Please describe the current behavior that you are modifying or link to a relevant issue. -->
Fix error code when creating an existed user profile, change from 400 to 409.
## What is the new behavior?
-------------------------------------
<!-- Please describe the behavior or changes that are being added by this PR. -->
- No new behavior.
## Does this introduce a breaking change?
-------------------------------------
- [YES]
<!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. -->
When creating an existed user profile now the error code is 409, before this change it was 400.
## Any relevant logs, error output, etc?
-------------------------------------
(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)
N/A
## Other information
-------------------------------------
<!-- Any other information that is important to this PR such as screenshots of how the component looks before and after the change. -->
N/AM1 - Release 0.1ethiraj krishnamanaiduDania Kodeih (Microsoft)Daniel Schollashley kelhamHong Yanethiraj krishnamanaiduhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/11Azure - (1/4) - Adding missing context path for entitlements2020-07-16T02:33:10ZDuvelis CaraoAzure - (1/4) - Adding missing context path for entitlements**This impacts only Azure Implementation**
This MR is part of a group of MRs with dependencies between them
0. Update infrastructure (Azure / osdu-infrastructure) to have reference to the new URLs (https://github.com/Azure/osdu-inf...**This impacts only Azure Implementation**
This MR is part of a group of MRs with dependencies between them
0. Update infrastructure (Azure / osdu-infrastructure) to have reference to the new URLs (https://github.com/Azure/osdu-infrastructure/pull/23)
1. Adding missing context path for entitlements.
2. Adding missing context path for legal.
3. Adding missing context path for storage.
4. Adding missing context path for indexer
After merging this MR, the new Entitlement URL will be:
https://{host}**/entitlements/v1/**
Before merging this MR:
- Please make sure infrastructure is updated and executed in order to have the new URL for entitlement, otherwise, integration testing will fail.
- Update environment variable (AZURE - Common) with the new Entitlement URL:
ENTITLEMENT_URL: https://$(AZURE_ENTITLEMENTS_SERVICE_NAME).azurewebsites.net/entitlements/v1/
-------
This PR is related to 6 other requests for changing a URL structure and adding in context paths for services.
1. osdu-infrastructure 1/6 PR https://github.com/Azure/osdu-infrastructure/pull/23
1. CI-CD Pipelines 2/6 This PR
1. Entitlements Azure 3/6 https://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/11
1. Legal 4/6 https://community.opengroup.org/osdu/platform/security-and-compliance/legal/-/merge_requests/32
1. Storage 5/6 https://community.opengroup.org/osdu/platform/system/storage/-/merge_requests/33
1. Indexer 6/6 https://community.opengroup.org/osdu/platform/system/indexer-service
They are all required and any one but not the other will break builds and tests across the entire ecosystem for Azure.M1 - Release 0.1ethiraj krishnamanaiduDania Kodeih (Microsoft)Daniel Schollethiraj krishnamanaiduhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/12making changes to use CosmosStore instead of CosmosFacade2020-07-29T13:11:43Zharshit aggarwalmaking changes to use CosmosStore instead of CosmosFacadeThis change will make Entitlement service to use CosmosStore class instead of CosmosFacade for
accessing cosmosDbThis change will make Entitlement service to use CosmosStore class instead of CosmosFacade for
accessing cosmosDbharshit aggarwalharshit aggarwalhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/13Enabled transaction logger and slf4jlogger2020-08-17T14:26:22ZKishore BattulaEnabled transaction logger and slf4jlogger## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull-requests/code-reviews/readme.md)?
* [YES] Have ...## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull-requests/code-reviews/readme.md)?
* [YES] Have you added an explanation of what your changes do and why you'd like us to include them?
* [YES] I have updated the documentation accordingly.
* [NA] I have added tests to cover my changes.
* [YES] All new and existing tests passed.
* [YES] My code follows the code style of this project.
* [YES] I ran lint checks locally prior to submission.
## What is the current behavior?
-------------------------------------
<!-- Please describe the current behavior that you are modifying, or link to a relevant issue. -->
Issue: Logs are not written to azure app insights.
## What is the new behavior?
-------------------------------------
<!-- Please describe the behavior or changes that are being added by this PR. -->
- Logs are written to azure app insights
- Added new logs to log start and end of web request
-
## Does this introduce a breaking change?
-------------------------------------
- [NO]
<!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. -->
## Any relevant logs, error output, etc?
-------------------------------------
(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)
## Other information
-------------------------------------
<!-- Any other information that is important to this PR such as screenshots of how the component looks before and after the change. -->
https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/11Daniel SchollHema Vishnu Pola [Microsoft]Daniel Schollhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/14Enabled slf4j MDC Context filter2020-08-18T16:31:52ZKishore BattulaEnabled slf4j MDC Context filterhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/15Azure aks2020-08-31T12:08:13ZDaniel SchollAzure aksDaniel SchollDaniel Schollhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/16Azure aks2020-08-31T15:05:01ZDaniel SchollAzure aksBug Fixing Azure AKS Pipeline supportBug Fixing Azure AKS Pipeline supportDaniel SchollDaniel Schollhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/18Draft: Resolve "Disabling AAD Authentication from R2 Services"2020-09-11T12:14:03ZKiran VeerapaneniDraft: Resolve "Disabling AAD Authentication from R2 Services"Closes #1Closes #1https://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/17Disabling AAD authentication In Entitlement service2020-09-11T17:12:10ZKiran VeerapaneniDisabling AAD authentication In Entitlement serviceAs Istio AuthN got enabled in R2 services,Disabling AAD authentication using Istio auth property.
Added a new Filter to decode payload coming from Istio and set security context to authorize requests.
Added Istio auth Env variable and se...As Istio AuthN got enabled in R2 services,Disabling AAD authentication using Istio auth property.
Added a new Filter to decode payload coming from Istio and set security context to authorize requests.
Added Istio auth Env variable and seti to true in helm charts templatehttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/19Adding conditional check on the component2020-09-12T07:36:48ZKiran VeerapaneniAdding conditional check on the componentAdding condition to Inject Filter only Istio is enabledAdding condition to Inject Filter only Istio is enabledhttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements-azure/-/merge_requests/20Updaing readme doc with Istio Auth filter2020-09-16T07:06:50ZKiran VeerapaneniUpdaing readme doc with Istio Auth filter