Skip to content

upgrading pallet projects modules e.g. werkzeug 2.0#1160

Merged
ReimarBauer merged 5 commits intoOpen-MSS:developfrom
ReimarBauer:i970
Aug 20, 2021
Merged

upgrading pallet projects modules e.g. werkzeug 2.0#1160
ReimarBauer merged 5 commits intoOpen-MSS:developfrom
ReimarBauer:i970

Conversation

@ReimarBauer
Copy link
Member

No description provided.

@ReimarBauer ReimarBauer marked this pull request as draft August 19, 2021 17:55
@ReimarBauer ReimarBauer requested a review from joernu76 August 20, 2021 06:36
@ReimarBauer ReimarBauer marked this pull request as ready for review August 20, 2021 06:36
@ReimarBauer ReimarBauer linked an issue Aug 20, 2021 that may be closed by this pull request
@ReimarBauer ReimarBauer requested a review from Marilyth August 20, 2021 07:47
joernu76
joernu76 previously approved these changes Aug 20, 2021
sio.on('chat-message-client', handler=handle_incoming_message)
sio.connect(self.url)
sio.emit('start', response)
sio.on('chat-message-client', handler=handle_incoming_message, namespace="/")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This namespace for each call seems pretty dangerous and verbose. Is this really necessary or can this somehow be configured more smartly?

According to "https://python-socketio.readthedocs.io/en/latest/server.html#namespaces" "/" is the default namespace, so this seems superfluous?

Alternatively, one can use class based namespaces.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see
https://flask-socketio.readthedocs.io/en/latest/upgrading.html

We do have an event based communication, which is secured by a token.
On any http request the Url is verbose.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

example for using more than one namespace.
https://socket.io/docs/v3/namespaces/index.html

separation, improving, organisation can be done in a different issue.

@ReimarBauer ReimarBauer changed the title upgrading first part of flask based modules upgrading pallet projects modules e.g. werkzeug 2.0 Aug 20, 2021
joernu76
joernu76 previously approved these changes Aug 20, 2021
@ReimarBauer ReimarBauer merged commit 9a873bb into Open-MSS:develop Aug 20, 2021
@ReimarBauer ReimarBauer deleted the i970 branch August 20, 2021 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor for updated pallets projects

2 participants