update ci file #42
update ci file #42
Conversation
WalkthroughUpdated environment variable placeholders in Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Pre-merge checks (3 passed)✅ Passed checks (3 passed)
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
✨ Finishing touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
| spring.datasource.dbidentity.url=@env.DATABASE_IDENTITY_URL@ | ||
| spring.datasource.dbidentity.username=@env.DATABASE_IDENTITY_USERNAME@ | ||
| spring.datasource.dbidentity.password=@env.DATABASE_IDENTITY_USERNAME@ | ||
| spring.datasource.dbidentity.driver-class-name=com.mysql.cj.jdbc.Driver |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
Password points to USERNAME env var; fix to use DATABASE_IDENTITY_PASSWORD.
Line 10 maps the password to @env.DATABASE_IDENTITY_USERNAME@. This will fail auth and risks masking misconfigured secrets in CI.
Apply this diff:
spring.datasource.dbidentity.url=@env.DATABASE_IDENTITY_URL@
spring.datasource.dbidentity.username=@env.DATABASE_IDENTITY_USERNAME@
-spring.datasource.dbidentity.password=@env.DATABASE_IDENTITY_USERNAME@
+spring.datasource.dbidentity.password=@env.DATABASE_IDENTITY_PASSWORD@Run to confirm the corresponding CI/env vars exist:
🏁 Script executed:
#!/bin/bash
# Verify references to new Identity vars across the repo
rg -nP 'DATABASE_IDENTITY_(URL|USERNAME|PASSWORD)' -S -C1Length of output: 988
Fix password env var mapping: use DATABASE_IDENTITY_PASSWORD (2 locations)
Password properties currently reference DATABASE_IDENTITY_USERNAME — change to DATABASE_IDENTITY_PASSWORD to avoid CI auth failures and secret-masking.
File: src/main/environment/common_ci.properties — lines 8–11 and 20–23
spring.datasource.dbidentity.url=@env.DATABASE_IDENTITY_URL@
spring.datasource.dbidentity.username=@env.DATABASE_IDENTITY_USERNAME@
-spring.datasource.dbidentity.password=@env.DATABASE_IDENTITY_USERNAME@
+spring.datasource.dbidentity.password=@env.DATABASE_IDENTITY_PASSWORD@
spring.datasource.dbidentity.driver-class-name=com.mysql.cj.jdbc.Driver spring.datasource.db1097identity.url=@env.DATABASE_1097_IDENTITY_URL@
spring.datasource.db1097identity.username=@env.DATABASE_IDENTITY_USERNAME@
-spring.datasource.db1097identity.password=@env.DATABASE_IDENTITY_USERNAME@
+spring.datasource.db1097identity.password=@env.DATABASE_IDENTITY_PASSWORD@
spring.datasource.db1097identity.driver-class-name=com.mysql.cj.jdbc.Driver📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| spring.datasource.dbidentity.url=@env.DATABASE_IDENTITY_URL@ | |
| spring.datasource.dbidentity.username=@env.DATABASE_IDENTITY_USERNAME@ | |
| spring.datasource.dbidentity.password=@env.DATABASE_IDENTITY_USERNAME@ | |
| spring.datasource.dbidentity.driver-class-name=com.mysql.cj.jdbc.Driver | |
| spring.datasource.dbidentity.url=@env.DATABASE_IDENTITY_URL@ | |
| spring.datasource.dbidentity.username=@env.DATABASE_IDENTITY_USERNAME@ | |
| spring.datasource.dbidentity.password=@env.DATABASE_IDENTITY_PASSWORD@ | |
| spring.datasource.dbidentity.driver-class-name=com.mysql.cj.jdbc.Driver |
🤖 Prompt for AI Agents
In src/main/environment/common_ci.properties around lines 8–11 and 20–23 the
password property is incorrectly mapped to @env.DATABASE_IDENTITY_USERNAME@;
change both occurrences to @env.DATABASE_IDENTITY_PASSWORD@ so
spring.datasource.dbidentity.password uses the correct env var; verify both
replacements and commit.
| spring.datasource.db1097identity.url=@env.DATABASE_1097_IDENTITY_URL@ | ||
| spring.datasource.db1097identity.username=@env.DATABASE_IDENTITY_USERNAME@ | ||
| spring.datasource.db1097identity.password=@env.DATABASE_IDENTITY_USERNAME@ | ||
| spring.datasource.db1097identity.driver-class-name=com.mysql.cj.jdbc.Driver |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
1097 Identity: password maps to USERNAME; and USERNAME may need 1097-specific var.
- Line 22 should reference a password variable, not username.
- Given URL uses DATABASE_1097_IDENTITY_URL, make USERNAME/PASSWORD symmetric unless credentials are intentionally shared.
Preferred symmetric fix:
spring.datasource.db1097identity.url=@env.DATABASE_1097_IDENTITY_URL@
-spring.datasource.db1097identity.username=@env.DATABASE_IDENTITY_USERNAME@
-spring.datasource.db1097identity.password=@env.DATABASE_IDENTITY_USERNAME@
+spring.datasource.db1097identity.username=@env.DATABASE_1097_IDENTITY_USERNAME@
+spring.datasource.db1097identity.password=@env.DATABASE_1097_IDENTITY_PASSWORD@If sharing creds with the primary identity store is intended, at minimum fix the password and add a clarifying comment:
spring.datasource.db1097identity.url=@env.DATABASE_1097_IDENTITY_URL@
-spring.datasource.db1097identity.username=@env.DATABASE_IDENTITY_USERNAME@
-spring.datasource.db1097identity.password=@env.DATABASE_IDENTITY_USERNAME@
+spring.datasource.db1097identity.username=@env.DATABASE_IDENTITY_USERNAME@
+spring.datasource.db1097identity.password=@env.DATABASE_IDENTITY_PASSWORD@
+# Note: 1097 Identity uses shared Identity credentials by design.Quick check for 1097 vars:
🏁 Script executed:
#!/bin/bash
rg -nP 'DATABASE_1097_IDENTITY_(URL|USERNAME|PASSWORD)' -S -C1Length of output: 294
🏁 Script executed:
#!/bin/bash
set -euo pipefail
f="src/main/environment/common_ci.properties"
echo "== $f lines 18-24 =="
nl -ba "$f" | sed -n '18,24p' || true
echo
echo "== grep for relevant DATABASE_* vars in the file =="
rg -n --hidden -S 'DATABASE_1097_IDENTITY_(URL|USERNAME|PASSWORD)|DATABASE_IDENTITY_(USERNAME|PASSWORD)|DATABASE_1097_IDENTITY_URL' "$f" || trueLength of output: 529
Fix incorrect password placeholders and add 1097-specific credentials
Passwords are pointing at USERNAME vars; change db1097 to use DATABASE_1097_IDENTITY_USERNAME/PASSWORD and fix the primary identity password.
Location: src/main/environment/common_ci.properties — lines 9–10 and 20–22.
Preferred fix for db1097:
spring.datasource.db1097identity.url=@env.DATABASE_1097_IDENTITY_URL@
-spring.datasource.db1097identity.username=@env.DATABASE_IDENTITY_USERNAME@
-spring.datasource.db1097identity.password=@env.DATABASE_IDENTITY_USERNAME@
+spring.datasource.db1097identity.username=@env.DATABASE_1097_IDENTITY_USERNAME@
+spring.datasource.db1097identity.password=@env.DATABASE_1097_IDENTITY_PASSWORD@Also update primary identity password (line 10):
spring.datasource.dbidentity.password=@env.DATABASE_IDENTITY_PASSWORD@
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| spring.datasource.db1097identity.url=@env.DATABASE_1097_IDENTITY_URL@ | |
| spring.datasource.db1097identity.username=@env.DATABASE_IDENTITY_USERNAME@ | |
| spring.datasource.db1097identity.password=@env.DATABASE_IDENTITY_USERNAME@ | |
| spring.datasource.db1097identity.driver-class-name=com.mysql.cj.jdbc.Driver | |
| spring.datasource.db1097identity.url=@env.DATABASE_1097_IDENTITY_URL@ | |
| spring.datasource.db1097identity.username=@env.DATABASE_1097_IDENTITY_USERNAME@ | |
| spring.datasource.db1097identity.password=@env.DATABASE_1097_IDENTITY_PASSWORD@ | |
| spring.datasource.db1097identity.driver-class-name=com.mysql.cj.jdbc.Driver |
🤖 Prompt for AI Agents
In src/main/environment/common_ci.properties (lines ~9–10 and 20–23), the
password placeholders are incorrect: update the primary identity password line
to use @env.DATABASE_IDENTITY_PASSWORD@ instead of the username var, and update
the db1097 datasource to use @env.DATABASE_1097_IDENTITY_USERNAME@ for username
and @env.DATABASE_1097_IDENTITY_PASSWORD@ for password so the 1097-specific
credentials are used.
|
|
@SauravBizbRolly there are conflicts. |
2 participants
📋 Description
JIRA ID:
Please provide a summary of the change and the motivation behind it. Include relevant context and details.
✅ Type of Change
ℹ️ Additional Information
Please describe how the changes were tested, and include any relevant screenshots, logs, or other information that provides additional context.
Summary by CodeRabbit