Skip to content

mysql proxy: refactor BUILD struct#15885

Merged
lizan merged 5 commits intoenvoyproxy:mainfrom
qinggniq:struct
Apr 9, 2021
Merged

mysql proxy: refactor BUILD struct#15885
lizan merged 5 commits intoenvoyproxy:mainfrom
qinggniq:struct

Conversation

@qinggniq
Copy link
Copy Markdown
Contributor

@qinggniq qinggniq commented Apr 8, 2021

Signed-off-by: qinggniq livewithblank@gmail.com

This pull request is the part of #15485, help reduce the work of review.

  • refactor the BUILD struct of mysql proxy
  • replace auth related fields of message from string to vector, avoid \0 character leading message to be partial.

Commit Message: refactor BUILD struct of mysql proxy
Additional Description:
Risk Level: Low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

qinggniq added 2 commits April 8, 2021 16:55
Signed-off-by: qinggniq <livewithblank@gmail.com>

fix typo

Signed-off-by: qinggniq <livewithblank@gmail.com>
Signed-off-by: qinggniq <livewithblank@gmail.com>
@qinggniq qinggniq requested a review from mattklein123 as a code owner April 8, 2021 09:12
@qinggniq
Copy link
Copy Markdown
Contributor Author

qinggniq commented Apr 8, 2021

cc @lizan

Signed-off-by: qinggniq <livewithblank@gmail.com>
@lizan lizan self-assigned this Apr 8, 2021
qinggniq added 2 commits April 9, 2021 14:37
Signed-off-by: qinggniq <livewithblank@gmail.com>

fix header order

Signed-off-by: qinggniq <livewithblank@gmail.com>
Signed-off-by: qinggniq <livewithblank@gmail.com>
@qinggniq
Copy link
Copy Markdown
Contributor Author

qinggniq commented Apr 9, 2021

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #15885 (comment) was created by @qinggniq.

see: more, trace.

@lizan lizan merged commit 575fe8a into envoyproxy:main Apr 9, 2021
@qinggniq qinggniq deleted the struct branch April 14, 2021 12:35
Monkeyanator pushed a commit to Monkeyanator/envoy that referenced this pull request Apr 20, 2021
This pull request is the part of envoyproxy#15485, help reduce the work of review.
- refactor the BUILD struct of mysql proxy
- replace auth related fields of message from `string` to `vector`, avoid `\0` character leading message to be partial.

Commit Message: refactor BUILD struct of mysql proxy
Additional Description:
Risk Level: Low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: qinggniq <livewithblank@gmail.com>
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