Skip to content
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

remove padding in ssl request packet #1

Open
wants to merge 1 commit into
base: 2.5
Choose a base branch
from

Conversation

ThomasDai
Copy link
Owner

Demo how to remove the padding from the SSL request packet.

Copy link

@rascasoft rascasoft left a comment

Choose a reason for hiding this comment

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

Tested with MaxScale 2.5 and Azure MariaDB implementation. This is really needed to make MaxScale work in Azure, please make it happen.

@rascasoft
Copy link

@ThomasDai do you think this can become a pull request for the main MaxScale repo? Is there something I can do to help?

@ThomasDai
Copy link
Owner Author

@ThomasDai do you think this can become a pull request for the main MaxScale repo? Is there something I can do to help?

Hi @rascasoft , sorry for the late reply. There are many law documents need to be signed to create a PR to the MaxScale main repo and I could not sign them on my own since I am in an organization. But I think let any other people who can sign these documents to create a same PR to the main MaxScale repo should be totally valid since my PR is a discussion of the issue and is just free speech, which is not restricted by any licenses.

Thanks for your understanding...

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.

2 participants