| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
| |
Add missing DEL, RPUSH, RPOP and SREM commands to the list of commands to prefix.
Also, this commit refactors the prefixing logic a bit to make it simpler.
|
|
|
|
| |
Co-authored-by: Steven Joseph <steven@pointzi.com>
Co-authored-by: Steven Joseph <steven@stevenjoseph.in>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Introduce global key prefix for redis transport
Co-authored-by: Matus Valo <matusvalo@users.noreply.github.com>
* refactor: use a custom redis client
As per the suggestions, refactor the redis key prefixing to use a custom
redis client that prefixes the keys it uses.
The custom client implementation does not prefix every key by default as
the way of prefixing keys may differ for some redis commands, instead it
lists those keys that will be prefixed. In case of commands, where
multiple keys can be passed as an argument, the custom client defines
where the arg positions are starting and ending for the given command.
* test: fix unit tests by moving import statement
* fix: wrap redis.parse_response to remove key prefixes
Co-authored-by: Matus Valo <matusvalo@users.noreply.github.com>
* fix: typo
* fix: lint
Co-authored-by: Antonin Delpeuch <antonin@delpeuch.eu>
Co-authored-by: Matus Valo <matusvalo@users.noreply.github.com>
Co-authored-by: Jillian Vogel <jill@opencraft.com>
|
|
|
|
|
|
| |
updates:
- [github.com/asottile/pyupgrade: v2.23.3 → v2.24.0](https://github.com/asottile/pyupgrade/compare/v2.23.3...v2.24.0)
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
|
|
|
|
| |
Previously, calling `reject` when `predefined_queues` was not configured would cause `AttributeError` to be raised from `_extract_backoff_policy_configuration_and_message`. That exception could crash the whole Celery worker and force it to exit early because `AttributeError` is not excepted in the nearby call stack.
|
| |
|
|
|
| |
Only use simplejson if it's absolutely necessary - Python's built-in json module is better if it's available.
|
|
|
|
|
| |
updates:
- [github.com/asottile/pyupgrade: v2.23.1 → v2.23.3](https://github.com/asottile/pyupgrade/compare/v2.23.1...v2.23.3)
|
|
|
|
| |
for more information, see https://pre-commit.ci
|
| |
|
|
|
|
|
|
| |
updates:
- [github.com/asottile/pyupgrade: v2.23.0 → v2.23.1](https://github.com/asottile/pyupgrade/compare/v2.23.0...v2.23.1)
- [github.com/pycqa/isort: 5.9.2 → 5.9.3](https://github.com/pycqa/isort/compare/5.9.2...5.9.3)
|
| |
|
|
|
|
|
|
|
| |
updates:
- [github.com/asottile/pyupgrade: v2.21.2 → v2.23.0](https://github.com/asottile/pyupgrade/compare/v2.21.2...v2.23.0)
- https://gitlab.com/pycqa/flake8 → https://github.com/PyCQA/flake8
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* enable pre-commit
* use extend-ignore for flake8
* manual flake8 fixes
* [pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
* [pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
* Update kombu/__init__.py
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
|
| |
|
|
|
|
|
| |
* feat: add support for setting redis username
* tests: add redis connparams credentials tests
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Move pytest.importorskip to run earlier
Currently pytest fails to collect tests when `azure` is not present. Moving the pytest.importorskip clause before `from kombu.transport import azureservicebus` fixes it.
```
==================================== ERRORS ====================================
__________ ERROR collecting t/unit/transport/test_azureservicebus.py ___________
ImportError while importing test module '/build/python-kombu/src/kombu-5.1.0/t/unit/transport/test_azureservicebus.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/lib/python3.9/importlib/__init__.py:127: in import_module
return _bootstrap._gcd_import(name[level:], package, level)
t/unit/transport/test_azureservicebus.py:12: in <module>
from kombu.transport import azureservicebus
kombu/transport/azureservicebus.py:66: in <module>
import azure.core.exceptions
E ModuleNotFoundError: No module named 'azure'
```
* Add # noqa to make linter happy
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
* Fix broken nested() after #1320
* Remove backward compatible code not needed anymore
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
KeyError (#1343)
* Fix broken nested() after #1320
* prepare_accept_content now raises SerializerNotInstalled when wrong serializer alias is passed
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* add accept parameter to SimpleQueue class
* Fixed missing accept for get_nowait() and added unittests
* Remove unused **kwargs from SimpleQueue.__init__
* Use self.consumer.accept instead of new attribute in SimpleQueue
* Add tests for simple interface when accept=[]
Co-authored-by: Matus Valo <matusvalo@gmail.com>
|
|
|
| |
This reverts commit 3b6cd13d3b6da22f86a23b93b66c49133776cc75.
|
|
|
|
|
| |
* chore: refactor code quality issues
* remove .deepsource.toml config file
|
|
|
|
|
| |
We are ignoring kombu.transport.librabbitmq because apicheck is failing
due missing librabbitmq library.
|
| |
|
|
|
|
|
| |
* Make flake8 happy
* Make pydocstyle happy
|
|
|
|
|
| |
Celery does not coerce configuration values into the right type (See celery/celery#6696).
This is a workaround. This bug will be fixed in Celery NextGen when we will refactor our configuration subsystem.
|
|
|
| |
it was decided long ago and it let the unit and integrations tests run fast for faster feedback loop
|
| |
|
| |
|
|
|
|
|
|
|
| |
* Update before installing system dependencies.
* Actually run the linters using tox.
There are no environments named py-flake8, py-pydocstyle and py-apicheck.
|
|
|
|
|
|
|
|
|
| |
* fixed character replace table according to the comment above - dots replaced by dashes, other punctuations replaced by underscores
* optimised with precalculated punctuation set - according to @thedrow suggestion
* queue name tests
* cleanup
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Use a thread-safe implementation of cached_property.
* Restore setter and deleter.
* Restore tests.
* Fix __get__ signature for backport.
* Cleanup.
Co-authored-by: Asif Saif Uddin <auvipy@gmail.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* handle sqs sts
* add doc
* use string format
* improve docs
* Update kombu/transport/SQS.py
Co-authored-by: Omer Katz <omer.drow@gmail.com>
* add helper method and tests
* changer to private
Co-authored-by: galcohen <gal.cohen@autodesk.com>
Co-authored-by: Asif Saif Uddin <auvipy@gmail.com>
Co-authored-by: Omer Katz <omer.drow@gmail.com>
|
|
|
| |
- AMQPLAIN is only retained for backwards compatability and has become non-standard
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Clean up grammar in paragraph about callbacks
* Structure event draining examples into subsections
* Make code example make more sense
… and bring it more in line with text above.
* Fix code example
* Fix another code example
typo in method name
|
|
|
|
|
| |
https://docs.python.org/3.9/whatsnew/3.0.html#library-changes
> A common pattern in Python 2.x is to have one version of a module implemented in pure Python, with an optional accelerated version implemented as a C extension; for example, pickle and cPickle. This places the burden of importing the accelerated version and falling back on the pure Python version on each user of these modules. In Python 3.0, the accelerated versions are considered implementation details of the pure Python versions. Users should always import the standard version, which attempts to import the accelerated version and falls back to the pure Python version. The pickle / cPickle pair received this treatment.
|
|
|
|
|
|
|
|
|
| |
* Fix tiny mistake in introduction
… at least I _think_ it's a mistake.
* Remove unused import in code example
* Fix minor grammatical errors
|
| |
|
|
|
|
|
| |
Based on my headaches with silent revert to `localhost` I submit this PR.
The developer should be notified if their host settings are not found.
Details on the issue are here: https://github.com/celery/celery/issues/6661
|
|
|
|
|
| |
Protocol version 4 was added in Python 3.4 and is supported by all Python versions that kombu supports.
https://docs.python.org/3.9/library/pickle.html#data-stream-format
|