-
-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[feat] Validate proxies list before protocol decoder #6
Comments
@getlarge Would you mind to submit a PR for this? |
Yes i might make a proposal next week. |
Would you mind to open a new issue about that? Just to keep track of it. Maybe @mcollina could help to point us in the right direction to fix this
Daniel
… On 23 Jan 2020, at 20:39, getlarge ***@***.***> wrote:
Yes i might make a proposal next week.
By the way, there is still an issue in the with the protocol decoder when Aedes broker is used behind a TCP proxy which handles SSL termination.
The packet is splitted in two, on one side the protocol proxy header and the TCP buffer on the other.
Any help to solve this would be appreciated.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
@getlarge should this be moved to aedes-server-factory or protocol decoder libs instead? |
Good question, protocol-decoder might be a better place. We could still reuse that logic inside the server-factory via an option that could be overridden by user like |
Moved to protocol decoder |
Would you mind removing this todo and opening an issue instead?
Originally posted by @mcollina in moscajs/aedes#334
The text was updated successfully, but these errors were encountered: