8000 messenger:consume async --time-limit=# always rise an error · Issue #48196 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

messenger:consume async --time-limit=# always rise an error #48196

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
circulosmeos opened this issue Nov 11, 2022 · 1 comment
Closed

messenger:consume async --time-limit=# always rise an error #48196

circulosmeos opened this issue Nov 11, 2022 · 1 comment
Labels
Bug Status: Needs Review

Comments

@circulosmeos
Copy link
circulosmeos commented Nov 11, 2022 •
edited
Loading

Symfony version(s) affected

4.4.49-DEV

Description

969f19b commit (or maybe other?) makes --time-limit=# to always rise an error.
4.4.47 is checked as fine.

How to reproduce

php bin/console messenger:consume async -t 3600

Possible Solution

No response

Additional Context

No response
@alamirault alamirault mentioned this issue Nov 11, 2022
Merged
@alamirault
Copy link
Contributor

Hi @circulosmeos

Thank you for your report, I created PR #48198 in order to fix it.

nicolas-grekas added a commit that referenced this issue Nov 14, 2022
@nicolas-grekas
bug #48198 [Messenger] Fix time-limit check exception (alamirault)
63221f0 
This PR was squashed before being merged into the 4.4 branch.

Discussion
----------

[Messenger] Fix time-limit check exception

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | Fix #48196  <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->
| License       | MIT
| Doc PR        | symfony/symfony-docs#... <!-- required for new features -->

Check on `time-limit` was invalid.

(Unrelated fabbot failure)

Commits
-------

a13b41a [Messenger] Fix time-limit check exception
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@nicolas-grekas @circulosmeos @alamirault @carsonbot
0