This is an automated email from the ASF dual-hosted git repository.
Aias00 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shenyu.git
The following commit(s) were added to refs/heads/master by this push:
new 9f57e93016 refactor(admin): rename misspelled loged column/field to
logged in selector and rule (#7589) (#7677)
9f57e93016 is described below
commit 9f57e930169404dd2d2e50b838bd651a18833924
Author: konlue <[email protected]>
AuthorDate: Wed Oct 21 10:40:16 2026 +0800
refactor(admin): rename misspelled loged column/field to logged in selector
and rule (#7589) (#7677)
* refactor(admin): rename misspelled loged column/field to logged in
selector and rule (#7589)
* fix(common): accept legacy loged key on the sync wire and make log
printing null-safe (#7589)
---
db/upgrade/upgrade-guide.md | 3 +++
.../java/org/apache/shenyu/plugin/base/AbstractShenyuPluginTest.java | 4 ++--
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/db/upgrade/upgrade-guide.md b/db/upgrade/upgrade-guide.md
index dbddf8da8a..9e7e1f2cad 100644
--- a/db/upgrade/upgrade-guide.md
+++ b/db/upgrade/upgrade-guide.md
@@ -14,6 +14,9 @@ contract. Upgrade all bootstrap instances before, or together
with, Admin:
an Admin emitting `logged` is not understood by older bootstraps. Stop Admin
instances before running the script, then restart them after it completes.
Fresh installations create the renamed columns directly.
+The MySQL script uses `ALTER TABLE ... RENAME COLUMN`, which requires
+MySQL 8.0 or later; on MySQL 5.7 apply the same rename with the equivalent
+`ALTER TABLE ... CHANGE COLUMN loged logged ...` syntax instead.
- 2.7.2-upgrade-2.7.3-mysql.sql
diff --git
a/shenyu-plugin/shenyu-plugin-base/src/test/java/org/apache/shenyu/plugin/base/AbstractShenyuPluginTest.java
b/shenyu-plugin/shenyu-plugin-base/src/test/java/org/apache/shenyu/plugin/base/AbstractShenyuPluginTest.java
index 9795844bbd..ce7c74adeb 100644
---
a/shenyu-plugin/shenyu-plugin-base/src/test/java/org/apache/shenyu/plugin/base/AbstractShenyuPluginTest.java
+++
b/shenyu-plugin/shenyu-plugin-base/src/test/java/org/apache/shenyu/plugin/base/AbstractShenyuPluginTest.java
@@ -319,7 +319,7 @@ public final class AbstractShenyuPluginTest {
try (LogCapture capture =
LogCapture.attach(AbstractShenyuPlugin.class)) {
StepVerifier.create(testShenyuPlugin.execute(exchange,
shenyuPluginChain)).expectSubscription().verifyComplete();
assertTrue(capture.infoMessages().stream().noneMatch(message ->
message.contains("rule")),
- "no rule match line may reach INFO when the rule's loged
flag is false, got: " + capture.infoMessages());
+ "no rule match line may reach INFO when the rule's logged
flag is false, got: " + capture.infoMessages());
}
}
@@ -342,7 +342,7 @@ public final class AbstractShenyuPluginTest {
.filter(message -> message.contains("rule"))
.collect(Collectors.toList());
assertEquals(1, ruleLines.size(),
- "exactly one rule match line is expected at INFO when
loged is true, got: " + ruleLines);
+ "exactly one rule match line is expected at INFO when
logged is true, got: " + ruleLines);
assertTrue(ruleLines.get(0).contains("rule success match"),
"the surviving line should be the one that carries the
rule name, got: " + ruleLines.get(0));
}