Hi, I've been really happy with the basic usage of mqttasgi in my pr

Hi <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="

Clarify Worker API usage about mqttasgi HOT 3 CLOSED

SootyOwl commented on August 15, 2024

Clarify Worker API usage

from mqttasgi.

Comments (3)

sivulich commented on August 15, 2024 1

Thanks for your contribution! I missed that the create application is also used for the applications that already come from the configuration files. If you’d like you can create a PR with this changes so your contribution will appear on the project 😊

…

On Thu, 25 Aug 2022 at 10:25 Sooty ***@***.***> wrote: Hi @sivulich <https://github.com/sivulich> I tried the change above but was presented with another exception: Traceback (most recent call last): File "/usr/local/bin/mqttasgi", line 8, in <module> sys.exit(main()) File "/usr/local/lib/python3.10/site-packages/mqttasgi/cli.py", line 41, in main application = get_application(args.application) File "/usr/local/lib/python3.10/site-packages/mqttasgi/utils.py", line 13, in get_application return application.as_asgi() AttributeError: 'ProtocolTypeRouter' object has no attribute 'as_asgi' I managed to "solve" this problem by changing https://github.com/sivulich/mqttasgi/blob/bcf4262031f5155647a1acd36a283528f1199588/mqttasgi/utils.py#L12 to if hasattr(application, 'as_asgi'): return application.as_asgi() return application This doesn't present any errors, and a worker now successfully spawns! DEBUG [mqttasgi.cli:208] [mqttasgi][app][subscribe] - Subscription to dt/#:0 has 2 listeners — Reply to this email directly, view it on GitHub <#11 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEMYVUHAO4ZLVTGOPTBX7NTV243Z3ANCNFSM57PHMBTQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

from mqttasgi.

sivulich commented on August 15, 2024

Hi @SootyOwl thanks for your comments.

Your usage is correct, I think it's related to a possible bug here:
line 12 of utils.py

I think it should be:

    return application.as_asgi()

Let me know if that change works for you and I'll create a PR with the fix.

from mqttasgi.

SootyOwl commented on August 15, 2024

Hi @sivulich

I tried the change above but was presented with another exception:

Traceback (most recent call last):
  File "/usr/local/bin/mqttasgi", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.10/site-packages/mqttasgi/cli.py", line 41, in main
    application = get_application(args.application)
  File "/usr/local/lib/python3.10/site-packages/mqttasgi/utils.py", line 13, in get_application
    return application.as_asgi()
AttributeError: 'ProtocolTypeRouter' object has no attribute 'as_asgi'

I managed to "solve" this problem by changing

mqttasgi/mqttasgi/utils.py

Line 12 in bcf4262

return application

    if hasattr(application, 'as_asgi'):
        return application.as_asgi()
    return application

This doesn't present any errors, and a worker now successfully spawns!

DEBUG [mqttasgi.cli:208] [mqttasgi][app][subscribe] - Subscription to dt/#:0 has 2 listeners

from mqttasgi.

Clarify Worker API usage about mqttasgi HOT 3 CLOSED

Comments (3)

Related Issues (19)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent