Arnau

Results 18 comments of Arnau

This PR fixes issues #68 . @markokr , could you merge this PR, please? Thanks in advance!

Same issue here! @sirrus233 fix (PR #69 ) works for me.

I have a PR to fix this: #78 , but not merged yet. They will wait until celery 5, because it is not backward compatible.

@ZviBaratz , you're right, not working :( And this is because of #91, which was submitted after #78 but merged first. #91 gave `kwargsrepr` (which is already casted to string...

@ZviBaratz , the solution I've found, without changing anything from `django-celery-results`, is calling all tasks like this: ```python3 from celery import shared_task @shared_task def debug_task(x, y=0, s=True): if s: return...

I've come up with some solutions: 1. Do nothing, just always call tasks adding `argsrepr=args, kwargsrepr=kwargs` (keep in mind that we're duplicating data and messages have a max length...), or...

> I still can't seem to be able to return a dict. I know. It only works with `task_protocol=1` which is not the default. As I said in my previous...

@auvipy These are my suggestions: > > 2. Add a setting to `Celery` to be able to disable the argsrepr/kwargsrepr feature. Acceptable if you don't have sensitive data, and you...

And I think that the settings I suggest should be used here: https://github.com/celery/celery/blob/8c5e9888ae10288ae1b2113bdce6a4a41c47354b/celery/app/amqp.py#L314 and in some other place like https://github.com/celery/celery/blob/7288147d65a32b726869ed887d99e4bfd8c070e2/celery/worker/strategy.py#L81, maybe...

@auvipy , @ZviBaratz I've done this PR: https://github.com/celery/celery/pull/6595 Not tested. Just to show my suggestion, and get feedback from you and other users.