diff options
| author | Dirk-Jan C. Binnema <djcb@djcbsoftware.nl> | 2025-09-12 19:46:23 +0300 |
|---|---|---|
| committer | Dirk-Jan C. Binnema <djcb@djcbsoftware.nl> | 2025-09-12 19:06:21 +0300 |
| commit | 8209e802e3c331e2dd0eb67fce18aa2a3019fed4 (patch) | |
| tree | 519b16498bfe829a0f7c91e011b81cc75625efce /lib | |
| parent | 97d6a41823a76351ed26693b8db167cc6ee14881 (diff) | |
labels: tighten labels regexp
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/message/mu-labels.cc | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/lib/message/mu-labels.cc b/lib/message/mu-labels.cc index c3822dd..87b92e3 100644 --- a/lib/message/mu-labels.cc +++ b/lib/message/mu-labels.cc @@ -51,16 +51,16 @@ Mu::Labels::validate_label(const std::string &label) if (::iscntrl(uc)) return Err(Error{Error::Code::InvalidArgument, - "control character {} is not allowed", + "control character {} not allowed in label", static_cast<int>(uc)}); if (::isblank(uc)) return Err(Error{Error::Code::InvalidArgument, - "blank character {} is not allowed", + "blank character {} not allowed in label", static_cast<int>(uc)}); - if (uc == '"' || uc == '\'' || uc == '`' || + if (uc == '"' || uc == '\'' || uc == '`' || uc == ',' || uc == '\\' || uc == '/' || uc == '$') return Err(Error{Error::Code::InvalidArgument, - "character '{}' is not allowed", uc}); + "character '{}' not allowed in label", uc}); } return Ok(); @@ -217,11 +217,14 @@ test_validate_label() g_assert_true(!!validate_label("@raven+king")); g_assert_true(!!validate_label("operation:mindcrime")); g_assert_true(!!validate_label("😨")); + g_assert_true(!!validate_label("foo%bar+1")); g_assert_false(!!validate_label("norrell strange")); g_assert_false(!!validate_label("")); g_assert_false(!!validate_label("+")); g_assert_false(!!validate_label("-")); + g_assert_false(!!validate_label("foo$bar")); + g_assert_false(!!validate_label("foo,bar")); g_assert_false(!!validate_label("foo`bar")); g_assert_false(!!validate_label("\"quoted\"")); } |