[stable30] Fix federated reshares #75342
provokateurinstatic-code-analysis.yml
on: pull_request
static-code-analysis
4m 37s
static-code-analysis-security
12m 54s
static-code-analysis-ocp
2m 28s
Annotations
16 errors and 1 warning
|
static-code-analysis
Process completed with exit code 2.
|
|
TooFewArguments:
apps/federatedfilesharing/lib/FederatedShareProvider.php#L241
apps/federatedfilesharing/lib/FederatedShareProvider.php:241:25: TooFewArguments: Too few arguments for method OCA\FederatedFileSharing\FederatedShareProvider::getsharefromexternalsharetable saw 1 (see https://psalm.dev/025)
|
|
TooFewArguments:
apps/federatedfilesharing/lib/FederatedShareProvider.php#L241
apps/federatedfilesharing/lib/FederatedShareProvider.php:241:25: TooFewArguments: Too few arguments for OCA\FederatedFileSharing\FederatedShareProvider::getShareFromExternalShareTable - expecting target to be passed (see https://psalm.dev/025)
|
|
InvalidScalarArgument:
apps/federatedfilesharing/lib/FederatedShareProvider.php#L223
apps/federatedfilesharing/lib/FederatedShareProvider.php:223:32: InvalidScalarArgument: Argument 1 of OCA\FederatedFileSharing\FederatedShareProvider::removeShareFromTable expects string, but int provided (see https://psalm.dev/012)
|
|
InvalidScalarArgument:
apps/federatedfilesharing/lib/FederatedShareProvider.php#L156
apps/federatedfilesharing/lib/FederatedShareProvider.php:156:33: InvalidScalarArgument: Argument 1 of OCA\FederatedFileSharing\FederatedShareProvider::removeShareFromTable expects string, but int provided (see https://psalm.dev/012)
|
|
TooManyArguments:
apps/federatedfilesharing/lib/FederatedShareProvider.php#L141
apps/federatedfilesharing/lib/FederatedShareProvider.php:141:48: TooManyArguments: Too many arguments for method OCA\FederatedFileSharing\Notifications::requestreshare - saw 8 (see https://psalm.dev/026)
|
|
TaintedHeader:
lib/base.php#L150
lib/base.php:150:12: TaintedHeader: Detected tainted header (see https://psalm.dev/256)
|
|
TaintedCallable:
apps/workflowengine/lib/Manager.php#L506
apps/workflowengine/lib/Manager.php:506:39: TaintedCallable: Detected tainted text (see https://psalm.dev/243)
|
|
TaintedCallable:
apps/workflowengine/lib/Manager.php#L469
apps/workflowengine/lib/Manager.php:469:38: TaintedCallable: Detected tainted text (see https://psalm.dev/243)
|
|
TaintedCallable:
apps/workflowengine/lib/Manager.php#L421
apps/workflowengine/lib/Manager.php:421:38: TaintedCallable: Detected tainted text (see https://psalm.dev/243)
|
|
TaintedFile:
apps/theming/lib/IconBuilder.php#L209
apps/theming/lib/IconBuilder.php:209:28: TaintedFile: Detected tainted file handling (see https://psalm.dev/255)
|
|
TaintedFile:
apps/theming/lib/IconBuilder.php#L116
apps/theming/lib/IconBuilder.php:116:40: TaintedFile: Detected tainted file handling (see https://psalm.dev/255)
|
|
TaintedCookie:
apps/files_sharing/lib/Controller/ShareController.php#L431
apps/files_sharing/lib/Controller/ShareController.php:431:35: TaintedCookie: Detected tainted cookie (see https://psalm.dev/257)
|
|
TaintedFile:
apps/files_external/lib/Lib/Storage/SFTP.php#L385
apps/files_external/lib/Lib/Storage/SFTP.php:385:22: TaintedFile: Detected tainted file handling (see https://psalm.dev/255)
|
|
TaintedCallable:
apps/files_external/lib/Config/ConfigAdapter.php#L53
apps/files_external/lib/Config/ConfigAdapter.php:53:50: TaintedCallable: Detected tainted text (see https://psalm.dev/243)
|
|
TaintedHtml:
apps/admin_audit/lib/Actions/Action.php#L42
apps/admin_audit/lib/Actions/Action.php:42:16: TaintedHtml: Detected tainted HTML (see https://psalm.dev/245)
|
|
static-code-analysis
CodeQL Action v3 will be deprecated in December 2026. Please update all occurrences of the CodeQL Action in your workflow files to v4. For more information, see https://github.blog/changelog/2025-10-28-upcoming-deprecation-of-codeql-action-v3/
|