SIMS
SIMS copied to clipboard
#3193 - New OCP Routes
Main changes
- Added
certificate,key, andcaCertificateto routes. - Added the proposed annotations to the same routes. Information about the route annotations and its intentions can be found in route-specific-annotations_route-configuration.
- Renamed the routes (now sims.studentaidbc.ca) to allow both routes to exist at the same time for now.
- Only the certificate contents are provided as parameters to allow the correct line-breaking.
Additional Steps
- Change Keycloak properties for:
aest,institution,student, andsupporting-users.- Valid Redirect URIs:
https://dev.sims.studentaidbc.ca/* - Web Origins:
https://dev.sims.studentaidbc.ca
- Valid Redirect URIs:
- Created a new GitHub variable
DOMAIN_PREFIX(created as a regular var because it is not sensitive). - Create the new variables
TLS_CERTIFICATE,TLS_KEY,TLS_CA_CERTIFICATE. These were not added to the sample config because there is no reason for them to exist in the local environment.
Non-directly related change
FORMIO_ROOT_EMAILandFORMIO_ROOT_PASSWORDare no longer regenerated at every form.io server deployment to avoid credentials reconfigurations along the applications. The credentials should be changed due to a password rotation process and not during the server update.
TODO
- Ensure release instructions for the
Additional Stepsabove.
E2E SIMS API Coverage Report
| Totals | |
|---|---|
| Statements: | 64.03% ( 5230 / 8168 ) |
| Methods: | 60.73% ( 631 / 1039 ) |
| Lines: | 68.3% ( 4144 / 6067 ) |
| Branches: | 42.84% ( 455 / 1062 ) |
Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
Backend Unit Tests Coverage Report
| Totals | |
|---|---|
| Statements: | 21.91% ( 3412 / 15576 ) |
| Methods: | 10.01% ( 195 / 1949 ) |
| Lines: | 25.59% ( 3004 / 11740 ) |
| Branches: | 11.29% ( 213 / 1887 ) |
E2E Workflow Workers Coverage Report
| Totals | |
|---|---|
| Statements: | 58.64% ( 509 / 868 ) |
| Methods: | 52.88% ( 55 / 104 ) |
| Lines: | 62.27% ( 411 / 660 ) |
| Branches: | 41.35% ( 43 / 104 ) |
E2E Queue Consumers Coverage Report
| Totals | |
|---|---|
| Statements: | 82.14% ( 952 / 1159 ) |
| Methods: | 83.47% ( 101 / 121 ) |
| Lines: | 83.93% ( 820 / 977 ) |
| Branches: | 50.82% ( 31 / 61 ) |
@andrewsignori-aot
E2E test seems failed in this PR, it needs to pass all E2E test?
@andrewsignori-aot
E2E test seems failed in this PR, it needs to pass all E2E test?
The failed tests are intermittent issues that also exist in other PRs. The ticket #3652 was created a few days ago to investigate those.