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.

Sign up for GitHub

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

Jump to bottom

meta/mysql: dont escape for mysql password #4069

Merged
merged 5 commits into from
Sep 27, 2023
Merged

meta/mysql: dont escape for mysql password #4069

merged 5 commits into from
Sep 27, 2023

Conversation

zhijian-pro
Copy link
Contributor

No description provided.

@zhijian-pro zhijian-pro changed the title dont escape for mysql password meta/mysql: dont escape for mysql password Sep 26, 2023
@zhijian-pro zhijian-pro added backport-1.0 Backport to release-1.0 branch backport-1.1 labels Sep 26, 2023
davies
davies reviewed Sep 26, 2023
View reviewed changes
cmd/format.go Outdated Show resolved Hide resolved
@zhijian-pro
Revert "improve desc"
d03f362 
This reverts commit ede82c7.
@github-actions
Copy link

Mutate Test Report
pkg/meta/sql_test.go-2-4: score:0.34 failed:134, passed:70, compile error:0, out of coverage:220, skip by comment:0, others:0
Job detail: https://github.com/juicedata/juicefs/actions/runs/6321817758/job/17166450050

pkg/meta/sql_test.go-1-4: score:0.53 failed:153, passed:176, compile error:0, out of coverage:95, skip by comment:0, others:0
Job detail: https://github.com/juicedata/juicefs/actions/runs/6321817758/job/17166449919

pkg/meta/sql_test.go-4-4: score:0.49 failed:158, passed:152, compile error:0, out of coverage:110, skip by comment:0, others:0
Job detail: https://github.com/juicedata/juicefs/actions/runs/6321817758/job/17166450314

pkg/meta/sql_test.go-3-4: score:0.41 failed:224, passed:157, compile error:0, out of coverage:43, skip by comment:0, others:0
Job detail: https://github.com/juicedata/juicefs/actions/runs/6321817758/job/17166450191

Usage: https://github.com/juicedata/juicefs/blob/main/.github/scripts/mutate/how_to_use_mutate_test.md

@github-actions
Copy link

Mutate Test Report

Usage: https://github.com/juicedata/juicefs/blob/main/.github/scripts/mutate/how_to_use_mutate_test.md

@github-actions
Copy link

Mutate Test Report
pkg/meta/sql_test.go-1-4: score:0.52 failed:159, passed:175, compile error:0, out of coverage:90, skip by comment:0, others:0
Job detail: https://github.com/juicedata/juicefs/actions/runs/6321832768/job/17166480505

pkg/meta/sql_test.go-2-4: score:0.33 failed:138, passed:68, compile error:0, out of coverage:218, skip by comment:0, others:0
Job detail: https://github.com/juicedata/juicefs/actions/runs/6321832768/job/17166480610

pkg/meta/sql_test.go-4-4: score:0.49 failed:158, passed:150, compile error:0, out of coverage:112, skip by comment:0, others:0
Job detail: https://github.com/juicedata/juicefs/actions/runs/6321832768/job/17166480832

pkg/meta/sql_test.go-3-4: score:0.41 failed:222, passed:157, compile error:0, out of coverage:45, skip by comment:0, others:0
Job detail: https://github.com/juicedata/juicefs/actions/runs/6321832768/job/17166480714

Usage: https://github.com/juicedata/juicefs/blob/main/.github/scripts/mutate/how_to_use_mutate_test.md

@davies davies merged commit 0314e38 into main Sep 27, 2023
36 of 38 checks passed
@davies davies deleted the fix-mysql-escape branch September 27, 2023 13:57
SandyXSD pushed a commit that referenced this pull request Nov 24, 2023
@zhijian-pro @SandyXSD
meta/mysql: dont escape for mysql password (#4069)
ac1ee76
davies pushed a commit that referenced this pull request Nov 24, 2023
@zhijian-pro @davies
meta/mysql: dont escape for mysql password (#4069)
056a8fc
SandyXSD pushed a commit that referenced this pull request Nov 24, 2023
@zhijian-pro @SandyXSD
meta/mysql: dont escape for mysql password (#4069)
29b8cc2
davies pushed a commit that referenced this pull request Nov 25, 2023
@zhijian-pro @davies
meta/mysql: dont escape for mysql password (#4069)
e1a4c4d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SandyXSD SandyXSD SandyXSD approved these changes

@davies davies Awaiting requested review from davies

Assignees
No one assigned
Labels
backport-1.0 Backport to release-1.0 branch backport-1.1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zhijian-pro @davies @SandyXSD