Skip to content

Conversation

@nolanpro
Copy link
Contributor

@nolanpro nolanpro commented Oct 21, 2025

Since we're using a single horizon instance for all tenants, this is very important

ci:deploy
ci:multitenancy

@nolanpro nolanpro requested a review from gusys October 21, 2025 15:17
cursor[bot]

This comment was marked as outdated.

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-c79f332172.engk8s.processmaker.net

@nolanpro nolanpro closed this Oct 22, 2025
@nolanpro nolanpro reopened this Oct 22, 2025
@vladyrichter
Copy link

QA server K8S was successfully deployed https://tenant-1.ci-c79f332172.engk8s.processmaker.net

@cursor
Copy link

cursor bot commented Oct 23, 2025

Bug: Environment Variable Ignored, Error Handling Missing

The getOriginalValue method now reads values solely from the .env file, ignoring runtime environment variables (e.g., from Docker/K8s). This can cause incorrect configurations and tenant switching failures. Additionally, it lacks error handling for file_get_contents, which may lead to fatal errors if the .env file is missing or unreadable.

Fix in Cursor Fix in Web

@vladyrichter
Copy link

QA server K8S was successfully deployed https://tenant-1.ci-c79f332172.engk8s.processmaker.net

@processmaker-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@nolanpro nolanpro merged commit a47b94b into develop Oct 23, 2025
9 checks passed
@nolanpro nolanpro deleted the task/FOUR-27380 branch October 23, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants