Skip to content

Merge Develop into Master for v25.11.1 release#1247

Merged
johnnyq merged 32 commits intomasterfrom
develop
Nov 17, 2025
Merged

Merge Develop into Master for v25.11.1 release#1247
johnnyq merged 32 commits intomasterfrom
develop

Conversation

@johnnyq
Copy link
Collaborator

@johnnyq johnnyq commented Nov 17, 2025

Merge Develop into Master for v25.11.1 release

…f not defined, add product type if not defined
…ted recurring payments, related saved cards and client payment provider relation
… selected, dont show relations in ticket edit if no client assigned to ticket, also dont display public and email response type if no contact_email exists
@johnnyq johnnyq requested a review from wrongecho November 17, 2025 17:13
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 17, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
E Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

}

// Enable crypto on the stream
if (!stream_socket_enable_crypto($fp, true, STREAM_CRYPTO_METHOD_TLS_CLIENT)) {

Check failure

Code scanning / SonarCloud

Weak SSL/TLS protocols should not be used High

Change this code to use a stronger protocol. See more on SonarQube Cloud
Comment on lines +297 to +306
$sql_tags_filter = mysqli_query($mysqli, "
SELECT tag_id, tag_name
FROM tags
LEFT JOIN asset_tags ON asset_tag_tag_id = tag_id
LEFT JOIN assets ON asset_tag_asset_id = asset_id
WHERE tag_type = 5
$client_query OR tag_id IN ($tag_filter)
GROUP BY tag_id
HAVING COUNT(asset_tag_asset_id) > 0 OR tag_id IN ($tag_filter)
");

Check failure

Code scanning / SonarCloud

Database queries should not be vulnerable to injection attacks High

Change this code to not construct SQL queries directly from user-controlled data. See more on SonarQube Cloud
@wrongecho
Copy link
Collaborator

Security alerts are false positives.
LGTM.

@johnnyq johnnyq merged commit c7ef362 into master Nov 17, 2025
3 of 5 checks passed
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.

2 participants