This project does not seem to handle request data directly as such no vulnerable execution paths were found.
include
, or for example
via PHP's auto-loading mechanism.
1 | <?php |
||||
2 | |||||
3 | namespace NeedleProject\LaravelRabbitMq\Entity; |
||||
4 | |||||
5 | use NeedleProject\LaravelRabbitMq\AMQPConnection; |
||||
6 | use NeedleProject\LaravelRabbitMq\ConsumerInterface; |
||||
7 | use NeedleProject\LaravelRabbitMq\Interpreter\EntityArgumentsInterpreter; |
||||
8 | use NeedleProject\LaravelRabbitMq\Processor\AbstractMessageProcessor; |
||||
9 | use NeedleProject\LaravelRabbitMq\Processor\MessageProcessorInterface; |
||||
10 | use NeedleProject\LaravelRabbitMq\PublisherInterface; |
||||
11 | use PhpAmqpLib\Channel\AMQPChannel; |
||||
12 | use PhpAmqpLib\Exception\AMQPProtocolChannelException; |
||||
13 | use PhpAmqpLib\Exception\AMQPTimeoutException; |
||||
14 | use PhpAmqpLib\Message\AMQPMessage; |
||||
15 | use PhpAmqpLib\Wire\AMQPTable; |
||||
16 | use Psr\Log\LoggerAwareInterface; |
||||
17 | use Psr\Log\LoggerAwareTrait; |
||||
18 | use PhpAmqpLib\Exception\AMQPChannelClosedException; |
||||
19 | |||||
20 | /** |
||||
21 | * Class QueueEntity |
||||
22 | * |
||||
23 | * @package NeedleProject\LaravelRabbitMq\Entity |
||||
24 | * @author Adrian Tilita <[email protected]> |
||||
25 | */ |
||||
26 | class QueueEntity implements PublisherInterface, ConsumerInterface, AMQPEntityInterface, LoggerAwareInterface |
||||
27 | { |
||||
28 | use LoggerAwareTrait; |
||||
29 | |||||
30 | /** |
||||
31 | * @const int Retry count when a Channel Closed exeption is thrown |
||||
32 | */ |
||||
33 | const MAX_RETRIES = 3; |
||||
34 | |||||
35 | /** |
||||
36 | * @const array Default connections parameters |
||||
37 | */ |
||||
38 | const DEFAULTS = [ |
||||
39 | // Whether to check if it exists or to verify existance using argument types (Throws PRECONDITION_FAILED) |
||||
40 | 'passive' => false, |
||||
41 | // Entities with durable will be re-created uppon server restart |
||||
42 | 'durable' => false, |
||||
43 | // whether to use it by only one channel, then it gets deleted |
||||
44 | 'exclusive' => false, |
||||
45 | // Whether to delete it when the queue has no event on it |
||||
46 | 'auto_delete' => false, |
||||
47 | // Whether to receive a Declare confirmation |
||||
48 | 'nowait' => false, |
||||
49 | // Additional arguments for queue creation |
||||
50 | 'arguments' => [], |
||||
51 | // Whether to auto create the entity before publishing/consuming it |
||||
52 | 'auto_create' => false, |
||||
53 | // whether to "hide" the exception on re-declare. |
||||
54 | // if the `passive` filter is set true, this is redundant |
||||
55 | 'throw_exception_on_redeclare' => true, |
||||
56 | // whether to throw on exception when trying to |
||||
57 | // bind to an in-existent queue/exchange |
||||
58 | 'throw_exception_on_bind_fail' => true, |
||||
59 | // no ideea what it represents - @todo - find a documentation that states it's role |
||||
60 | 'ticket' => null |
||||
61 | ]; |
||||
62 | |||||
63 | /** |
||||
64 | * @var AMQPConnection |
||||
65 | */ |
||||
66 | protected $connection; |
||||
67 | |||||
68 | /** |
||||
69 | * @var string |
||||
70 | */ |
||||
71 | protected $aliasName; |
||||
72 | |||||
73 | /** |
||||
74 | * @var array |
||||
75 | */ |
||||
76 | protected $attributes; |
||||
77 | |||||
78 | /** |
||||
79 | * @var int |
||||
80 | */ |
||||
81 | protected $prefetchCount = 1; |
||||
82 | |||||
83 | /** |
||||
84 | * @var null|string|MessageProcessorInterface |
||||
85 | */ |
||||
86 | protected $messageProcessor = null; |
||||
87 | |||||
88 | /** |
||||
89 | * @var int |
||||
90 | */ |
||||
91 | protected $limitMessageCount; |
||||
92 | |||||
93 | /** |
||||
94 | * @var int |
||||
95 | */ |
||||
96 | protected $limitSecondsUptime; |
||||
97 | |||||
98 | /** |
||||
99 | * @var int |
||||
100 | */ |
||||
101 | protected $limitMemoryConsumption; |
||||
102 | |||||
103 | /** |
||||
104 | * @var double |
||||
105 | */ |
||||
106 | protected $startTime = 0; |
||||
107 | |||||
108 | /** |
||||
109 | * @var int |
||||
110 | */ |
||||
111 | protected $retryCount = 0; |
||||
112 | /** |
||||
113 | * @var bool |
||||
114 | 19 | */ |
|||
115 | protected $globalPrefetch = true; |
||||
116 | 19 | ||||
117 | /** |
||||
118 | * @param AMQPConnection $connection |
||||
119 | 19 | * @param string $aliasName |
|||
120 | * @param array $queueDetails |
||||
121 | * @return QueueEntity |
||||
122 | */ |
||||
123 | public static function createQueue(AMQPConnection $connection, string $aliasName, array $queueDetails) |
||||
124 | { |
||||
125 | return new static( |
||||
126 | 3 | $connection, |
|||
127 | $aliasName, |
||||
128 | 3 | array_merge(self::DEFAULTS, $queueDetails) |
|||
129 | ); |
||||
130 | } |
||||
131 | |||||
132 | /** |
||||
133 | * @return string |
||||
134 | */ |
||||
135 | public function getAliasName(): string |
||||
136 | { |
||||
137 | return $this->aliasName; |
||||
138 | 19 | } |
|||
139 | |||||
140 | 19 | /** |
|||
141 | 19 | * ExchangeEntity constructor. |
|||
142 | 19 | * |
|||
143 | 19 | * @param AMQPConnection $connection |
|||
144 | * @param string $aliasName |
||||
145 | * @param array $attributes |
||||
146 | */ |
||||
147 | public function __construct(AMQPConnection $connection, string $aliasName, array $attributes = []) |
||||
148 | { |
||||
149 | 2 | $this->connection = $connection; |
|||
150 | $this->aliasName = $aliasName; |
||||
151 | 2 | $this->attributes = $attributes; |
|||
152 | 2 | } |
|||
153 | |||||
154 | /** |
||||
155 | * @param int $prefetchCount |
||||
156 | * @return ConsumerInterface |
||||
157 | */ |
||||
158 | public function setPrefetchCount(int $prefetchCount): ConsumerInterface |
||||
159 | 2 | { |
|||
160 | $this->prefetchCount = $prefetchCount; |
||||
161 | 2 | return $this; |
|||
162 | 2 | } |
|||
163 | |||||
164 | /** |
||||
165 | * @param string $messageProcessor |
||||
166 | * @return ConsumerInterface |
||||
167 | */ |
||||
168 | 10 | public function setMessageProcessor(string $messageProcessor): ConsumerInterface |
|||
169 | { |
||||
170 | 10 | $this->messageProcessor = $messageProcessor; |
|||
171 | return $this; |
||||
172 | } |
||||
173 | |||||
174 | /** |
||||
175 | * @param bool $globalPrefetch |
||||
176 | 10 | * @return ConsumerInterface |
|||
177 | */ |
||||
178 | 10 | public function setGlobalPrefetch(bool $globalPrefetch): ConsumerInterface |
|||
179 | { |
||||
180 | $this->globalPrefetch = $globalPrefetch; |
||||
181 | |||||
182 | return $this; |
||||
183 | } |
||||
184 | 4 | ||||
185 | /** |
||||
186 | * @return AMQPConnection |
||||
187 | 4 | */ |
|||
188 | 4 | protected function getConnection(): AMQPConnection |
|||
189 | 4 | { |
|||
190 | 4 | return $this->connection; |
|||
191 | 4 | } |
|||
192 | 4 | ||||
193 | 4 | /** |
|||
194 | 4 | * @return AMQPChannel |
|||
195 | 4 | */ |
|||
196 | protected function getChannel(): AMQPChannel |
||||
197 | 2 | { |
|||
198 | return $this->getConnection()->getChannel(); |
||||
199 | 2 | } |
|||
200 | 1 | ||||
201 | /** |
||||
202 | * Create the Queue |
||||
203 | 1 | */ |
|||
204 | public function create() |
||||
205 | 3 | { |
|||
206 | try { |
||||
207 | 4 | $this->getChannel() |
|||
208 | ->queue_declare( |
||||
209 | 4 | $this->attributes['name'], |
|||
210 | 1 | $this->attributes['passive'], |
|||
211 | $this->attributes['durable'], |
||||
212 | 3 | $this->attributes['exclusive'], |
|||
213 | $this->attributes['auto_delete'], |
||||
214 | 3 | $this->attributes['nowait'], |
|||
215 | 3 | EntityArgumentsInterpreter::interpretArguments( |
|||
216 | 3 | $this->attributes['arguments'] |
|||
217 | 3 | ), |
|||
218 | 3 | $this->attributes['ticket'] |
|||
219 | ); |
||||
220 | 1 | } catch (AMQPProtocolChannelException $e) { |
|||
221 | // 406 is a soft error triggered for precondition failure (when redeclaring with different parameters) |
||||
222 | 1 | if (true === $this->attributes['throw_exception_on_redeclare'] || $e->amqp_reply_code !== 406) { |
|||
223 | 1 | throw $e; |
|||
224 | } |
||||
225 | 2 | // a failure trigger channels closing process |
|||
226 | $this->reconnect(); |
||||
227 | } |
||||
228 | 2 | } |
|||
229 | |||||
230 | public function bind() |
||||
231 | { |
||||
232 | if (!isset($this->attributes['bind']) || empty($this->attributes['bind'])) { |
||||
233 | 1 | return; |
|||
234 | } |
||||
235 | 1 | foreach ($this->attributes['bind'] as $bindItem) { |
|||
236 | 1 | try { |
|||
237 | $this->getChannel() |
||||
238 | ->queue_bind( |
||||
239 | $this->attributes['name'], |
||||
240 | $bindItem['exchange'], |
||||
241 | 1 | $bindItem['routing_key'] ?? '' |
|||
242 | ); |
||||
243 | 1 | } catch (AMQPProtocolChannelException $e) { |
|||
244 | 1 | // 404 is the code for trying to bind to an non-existing entity |
|||
245 | if (true === $this->attributes['throw_exception_on_bind_fail'] || $e->amqp_reply_code !== 404) { |
||||
246 | throw $e; |
||||
247 | } |
||||
248 | $this->reconnect(); |
||||
249 | } |
||||
250 | } |
||||
251 | } |
||||
252 | |||||
253 | /** |
||||
254 | 4 | * Delete the queue |
|||
255 | */ |
||||
256 | 4 | public function delete() |
|||
257 | 1 | { |
|||
258 | 1 | $this->getChannel()->queue_delete($this->attributes['name']); |
|||
259 | } |
||||
260 | |||||
261 | /** |
||||
262 | 4 | * {@inheritdoc} |
|||
263 | 4 | */ |
|||
264 | 4 | public function reconnect() |
|||
265 | 4 | { |
|||
266 | 4 | $this->getConnection()->reconnect(); |
|||
267 | 4 | } |
|||
268 | |||||
269 | 3 | /** |
|||
270 | 2 | * Publish a message |
|||
271 | 2 | * |
|||
272 | * @param string $message |
||||
273 | 2 | * @param string $routingKey |
|||
274 | 2 | * @param array $properties |
|||
275 | 2 | * @return mixed|void |
|||
276 | 1 | * @throws AMQPProtocolChannelException |
|||
277 | */ |
||||
278 | 1 | public function publish(string $message, string $routingKey = '', array $properties = []) |
|||
279 | { |
||||
280 | 3 | if ($this->attributes['auto_create'] === true) { |
|||
281 | $this->create(); |
||||
282 | $this->bind(); |
||||
283 | } |
||||
284 | |||||
285 | try { |
||||
286 | $this->getChannel() |
||||
287 | ->basic_publish( |
||||
288 | new AMQPMessage( |
||||
289 | $message, |
||||
290 | EntityArgumentsInterpreter::interpretProperties( |
||||
291 | $this->attributes, |
||||
292 | $properties |
||||
293 | ) |
||||
294 | ), |
||||
295 | '', |
||||
296 | $this->attributes['name'], |
||||
297 | true |
||||
298 | ); |
||||
299 | $this->retryCount = 0; |
||||
300 | } catch (AMQPChannelClosedException $exception) { |
||||
301 | $this->retryCount++; |
||||
302 | // Retry publishing with re-connect |
||||
303 | if ($this->retryCount < self::MAX_RETRIES) { |
||||
304 | $this->getConnection()->reconnect(); |
||||
305 | $this->publish($message, $routingKey); |
||||
306 | |||||
307 | return; |
||||
308 | } |
||||
309 | throw $exception; |
||||
310 | } |
||||
311 | } |
||||
312 | |||||
313 | /** |
||||
314 | * {@inheritdoc} |
||||
315 | * |
||||
316 | * @param int $messages |
||||
317 | * @param int $seconds |
||||
318 | * @param int $maxMemory |
||||
319 | * @return int |
||||
320 | */ |
||||
321 | public function startConsuming(int $messages, int $seconds, int $maxMemory) |
||||
322 | { |
||||
323 | $this->setupConsumer($messages, $seconds, $maxMemory); |
||||
324 | while (false === $this->shouldStopConsuming()) { |
||||
325 | try { |
||||
326 | $this->getChannel()->wait(null, false, $seconds); |
||||
327 | } catch (AMQPTimeoutException $e) { |
||||
328 | if ($this->shouldStopConsuming()) { |
||||
329 | break; |
||||
330 | } |
||||
331 | usleep(1000); |
||||
332 | $this->getConnection()->reconnect(); |
||||
333 | $this->setupChannelConsumer(); |
||||
334 | } catch (\Throwable $e) { |
||||
335 | // stop the consumer |
||||
336 | $this->stopConsuming(); |
||||
337 | $this->logger->notice(sprintf( |
||||
0 ignored issues
–
show
|
|||||
338 | "Stopped consuming: %s in %s:%d", |
||||
339 | get_class($e) . ' - ' . $e->getMessage(), |
||||
340 | (string)$e->getFile(), |
||||
341 | (int)$e->getLine() |
||||
342 | )); |
||||
343 | return 1; |
||||
344 | } |
||||
345 | } |
||||
346 | return 0; |
||||
347 | } |
||||
348 | |||||
349 | /** |
||||
350 | * @return bool |
||||
351 | */ |
||||
352 | protected function shouldStopConsuming(): bool |
||||
353 | { |
||||
354 | if ((microtime(true) - $this->startTime) > $this->limitSecondsUptime) { |
||||
355 | $this->logger->debug( |
||||
356 | "Stopped consumer", |
||||
357 | [ |
||||
358 | 'limit' => 'time_limit', |
||||
359 | 'value' => sprintf("%.2f", microtime(true) - $this->startTime) |
||||
360 | ] |
||||
361 | ); |
||||
362 | return true; |
||||
363 | } |
||||
364 | if (memory_get_peak_usage(true) >= ($this->limitMemoryConsumption * 1048576)) { |
||||
365 | $this->logger->debug( |
||||
366 | "Stopped consumer", |
||||
367 | [ |
||||
368 | 'limit' => 'memory_limit', |
||||
369 | 'value' => (int)round(memory_get_peak_usage(true) / 1048576, 2) |
||||
370 | ] |
||||
371 | ); |
||||
372 | return true; |
||||
373 | } |
||||
374 | |||||
375 | if ($this->getMessageProcessor()->getProcessedMessages() >= $this->limitMessageCount) { |
||||
376 | $this->logger->debug( |
||||
377 | "Stopped consumer", |
||||
378 | ['limit' => 'message_count', 'value' => (int)$this->getMessageProcessor()->getProcessedMessages()] |
||||
379 | ); |
||||
380 | return true; |
||||
381 | } |
||||
382 | return false; |
||||
383 | } |
||||
384 | |||||
385 | /** |
||||
386 | * Stop the consumer |
||||
387 | */ |
||||
388 | public function stopConsuming() |
||||
389 | { |
||||
390 | try { |
||||
391 | $this->getChannel()->basic_cancel($this->getConsumerTag(), false, true); |
||||
392 | } catch (\Throwable $e) { |
||||
393 | $this->logger->notice("Got " . $e->getMessage() . " of type " . get_class($e)); |
||||
394 | } |
||||
395 | } |
||||
396 | |||||
397 | /** |
||||
398 | * Setup the consumer |
||||
399 | * |
||||
400 | * @param int $messages |
||||
401 | * @param int $seconds |
||||
402 | * @param int $maxMemory |
||||
403 | */ |
||||
404 | protected function setupConsumer(int $messages, int $seconds, int $maxMemory) |
||||
405 | { |
||||
406 | $this->limitMessageCount = $messages; |
||||
407 | $this->limitSecondsUptime = $seconds; |
||||
408 | $this->limitMemoryConsumption = $maxMemory; |
||||
409 | |||||
410 | $this->startTime = microtime(true); |
||||
0 ignored issues
–
show
It seems like
microtime(true) can also be of type string . However, the property $startTime is declared as type double . Maybe add an additional type check?
Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable Either this assignment is in error or a type check should be added for that assignment. class Id
{
public $id;
public function __construct($id)
{
$this->id = $id;
}
}
class Account
{
/** @var Id $id */
public $id;
}
$account_id = false;
if (starsAreRight()) {
$account_id = new Id(42);
}
$account = new Account();
if ($account instanceof Id)
{
$account->id = $account_id;
}
![]() |
|||||
411 | |||||
412 | $this->setupChannelConsumer(); |
||||
413 | |||||
414 | $this->registerShutdownHandler(); |
||||
415 | $this->handleKillSignals(); |
||||
416 | } |
||||
417 | |||||
418 | private function setupChannelConsumer() |
||||
419 | { |
||||
420 | if ($this->attributes['auto_create'] === true) { |
||||
421 | $this->create(); |
||||
422 | $this->bind(); |
||||
423 | } |
||||
424 | |||||
425 | $this->getChannel() |
||||
426 | ->basic_qos(null, $this->prefetchCount, $this->globalPrefetch); |
||||
427 | |||||
428 | $this->getChannel() |
||||
429 | ->basic_consume( |
||||
430 | $this->attributes['name'], |
||||
431 | $this->getConsumerTag(), |
||||
432 | false, |
||||
433 | false, |
||||
434 | false, |
||||
435 | false, |
||||
436 | [ |
||||
437 | $this, |
||||
438 | 'consume' |
||||
439 | ] |
||||
440 | ); |
||||
441 | } |
||||
442 | |||||
443 | /** |
||||
444 | * Handle shutdown - Usually in case "Allowed memory size of x bytes exhausted" |
||||
445 | */ |
||||
446 | private function registerShutdownHandler() |
||||
447 | { |
||||
448 | $consumer = $this; |
||||
449 | register_shutdown_function(function () use ($consumer) { |
||||
450 | $consumer->stopConsuming(); |
||||
451 | }); |
||||
452 | } |
||||
453 | |||||
454 | /** |
||||
455 | * Register signals |
||||
456 | */ |
||||
457 | protected function handleKillSignals() |
||||
458 | { |
||||
459 | 1 | if (extension_loaded('pcntl')) { |
|||
460 | pcntl_signal(SIGTERM, [$this, 'catchKillSignal']); |
||||
461 | 1 | pcntl_signal(SIGINT, [$this, 'catchKillSignal']); |
|||
462 | |||||
463 | if (function_exists('pcntl_signal_dispatch')) { |
||||
464 | // let the signal go forward |
||||
465 | pcntl_signal_dispatch(); |
||||
466 | } |
||||
467 | 1 | } |
|||
468 | } |
||||
469 | |||||
470 | /** |
||||
471 | * Handle Kill Signals |
||||
472 | * @param int $signalNumber |
||||
473 | */ |
||||
474 | 1 | public function catchKillSignal(int $signalNumber) |
|||
475 | { |
||||
476 | $this->stopConsuming(); |
||||
477 | 1 | $this->logger->debug(sprintf("Caught signal %d", $signalNumber)); |
|||
478 | 1 | } |
|||
479 | |||||
480 | /** |
||||
481 | * It is the tag that is listed in RabbitMQ UI as the consumer "name" |
||||
482 | * |
||||
483 | * @return string |
||||
484 | */ |
||||
485 | private function getConsumerTag(): string |
||||
486 | { |
||||
487 | return sprintf("%s_%s_%s", $this->aliasName, gethostname(), getmypid()); |
||||
488 | } |
||||
489 | |||||
490 | /** |
||||
491 | * @return MessageProcessorInterface |
||||
492 | */ |
||||
493 | 1 | private function getMessageProcessor(): MessageProcessorInterface |
|||
494 | { |
||||
495 | if (!($this->messageProcessor instanceof MessageProcessorInterface)) { |
||||
496 | $this->messageProcessor = app($this->messageProcessor); |
||||
0 ignored issues
–
show
It seems like
app($this->messageProcessor) can also be of type Illuminate\Contracts\Foundation\Application or Illuminate\Foundation\Application . However, the property $messageProcessor is declared as type NeedleProject\LaravelRab...orInterface|null|string . Maybe add an additional type check?
Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable Either this assignment is in error or a type check should be added for that assignment. class Id
{
public $id;
public function __construct($id)
{
$this->id = $id;
}
}
class Account
{
/** @var Id $id */
public $id;
}
$account_id = false;
if (starsAreRight()) {
$account_id = new Id(42);
}
$account = new Account();
if ($account instanceof Id)
{
$account->id = $account_id;
}
![]() |
|||||
497 | if ($this->messageProcessor instanceof AbstractMessageProcessor) { |
||||
498 | $this->messageProcessor->setLogger($this->logger); |
||||
0 ignored issues
–
show
It seems like
$this->logger can also be of type null ; however, parameter $logger of NeedleProject\LaravelRab...eProcessor::setLogger() does only seem to accept Psr\Log\LoggerInterface , maybe add an additional type check?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
![]() |
|||||
499 | } |
||||
500 | } |
||||
501 | return $this->messageProcessor; |
||||
502 | } |
||||
503 | |||||
504 | /** |
||||
505 | * @param AMQPMessage $message |
||||
506 | * @throws \Throwable |
||||
507 | */ |
||||
508 | public function consume(AMQPMessage $message) |
||||
509 | { |
||||
510 | try { |
||||
511 | $this->getMessageProcessor()->consume($message); |
||||
512 | $this->logger->debug("Consumed message", ['message' => $message->getBody()]); |
||||
513 | } catch (\Throwable $e) { |
||||
514 | $this->logger->notice( |
||||
515 | sprintf( |
||||
516 | "Got %s from %s in %d", |
||||
517 | $e->getMessage(), |
||||
518 | (string)$e->getFile(), |
||||
519 | (int)$e->getLine() |
||||
520 | ) |
||||
521 | ); |
||||
522 | // let the exception slide, the processor should handle |
||||
523 | // exception, this is just a notice that should not |
||||
524 | // ever appear |
||||
525 | throw $e; |
||||
526 | } |
||||
527 | } |
||||
528 | } |
||||
529 |
This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.
This is most likely a typographical error or the method has been renamed.