Command options are ignored when extra config is set about swarrotbundle HOT 9 CLOSED

Hi.
Could you please paste your whole configuration?
When you want to add extras config, it should be put under the right processor. In your case, I think you're adding extra configuration for your consumer (which doesn't know how to deal with those options).
You can take a at the configuration reference to have an example.

from swarrotbundle.

Hi !

My config :

swarrot:
    provider: pecl
    default_connection: rabbitmq
    default_command: swarrot.command.base
    connections:
        rabbitmq:
            host: "%rabbitmq_host%"
            port: "%rabbitmq_port%"
            login: "%rabbitmq_user%"
            password: "%rabbitmq_password%"
            vhost: "%rabbitmq_vhost%"
    consumers:
        voip_valorize_call:
            processor: voip.processor.valorize_call
            middleware_stack: # order matter
                - configurator: swarrot.processor.signal_handler
                - configurator: swarrot.processor.max_messages
                - configurator: swarrot.processor.max_execution_time
                - configurator: swarrot.processor.exception_catcher
                - configurator: app.processor_configurator.sentry_exception_catcher
                - configurator: swarrot.processor.ack
            extras:
                max_messages: 100
                max_execution_time: 300

The "extras" config works: when I execute the swarrot command without options 100 messages are handled.
But when I want to override this config with the --max-messages 1 option of the swarrot command, that doesn't works : 100 messages are handled, not 1

from swarrotbundle.

Your config should looks like this:

swarrot:
    provider: pecl
    default_connection: rabbitmq
    default_command: swarrot.command.base
    connections:
        rabbitmq:
            host: "%rabbitmq_host%"
            port: "%rabbitmq_port%"
            login: "%rabbitmq_user%"
            password: "%rabbitmq_password%"
            vhost: "%rabbitmq_vhost%"
    consumers:
        voip_valorize_call:
            processor: voip.processor.valorize_call
            middleware_stack: # order matter
                - configurator: swarrot.processor.signal_handler
                - configurator: swarrot.processor.max_messages
                  extras:
                      max_messages: 100
                - configurator: swarrot.processor.max_execution_time
                  extras:
                      max_execution_time: 300
                - configurator: swarrot.processor.exception_catcher
                - configurator: app.processor_configurator.sentry_exception_catcher
                - configurator: swarrot.processor.ack

Each extra config should be attached to the relevant configurator.
Can you confirm it works with this config?

I'll try to figure out why those options partly work when they are put under the customer instead of the right configurator.

BTW, I see that you're using a sentry_exception_catcher. Is it possible to share your code? Maybe in a gist if you think it's not something which can be merged in swarrot?

from swarrotbundle.

@odolbeau That works, thanks for your help !

For sentry, the code is here : https://gist.github.com/notFloran/c0e5f6290c498c5bc18ae188ecd389f8

Do you think the code is ok for a PR ?

from swarrotbundle.

Sorry, just found some time to take a look at your gist.

The code looks good to me.
I never used Sentry mysel. Is the $data a custom format you choose or is it documented somewhere?
Is there a limit a size for the data you sent?

Don't hesitate to open a PR for this Processor. :)

from swarrotbundle.

Sorry, just found some time to take a look at your gist.

no problem

The code looks good to me. I never used Sentry mysel. Is the $data a custom format you choose or is it documented somewhere?

$data is documented here -> https://docs.sentry.io/clients/php/usage/#optional-attributes

tags are used to find events in sentry and extra are additional data for the context.

Is there a limit a size for the data you sent?

In the doc I find this : Extra contextual data is limited to 4096 characters.

I will do some tests at work next week.

from swarrotbundle.

Thanks. :)

from swarrotbundle.

sentry-php cut $data when the lenght > 1024 -> https://github.com/getsentry/sentry-php/blob/master/lib/Raven/Serializer.php#L97

I will do a MR for the sentry processor.

from swarrotbundle.

Perfect. :)
Don't hesitate to add all those useful links into your MR. :)

from swarrotbundle.