Skip to content

fix: allow # in channel names#3

Open
FearLessSaad wants to merge 1 commit into
sockudo:mainfrom
FearLessSaad:fix/allow-hash-in-channel-name
Open

fix: allow # in channel names#3
FearLessSaad wants to merge 1 commit into
sockudo:mainfrom
FearLessSaad:fix/allow-hash-in-channel-name

Conversation

@FearLessSaad

Copy link
Copy Markdown

Description

The send_to_user method is currently broken out of the box. It constructs a channel name using the # character, but the internal channel name validation regex explicitly rejects the # character, resulting in a validation error before the request is even sent.

Updated the CHANNEL_NAME_PATTERN regex in src/channel.rs to include the # character:

static CHANNEL_NAME_PATTERN: LazyLock<Regex> =
    LazyLock::new(|| Regex::new(r"^[#A-Za-z0-9_\-=@,.;]+$").unwrap());

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.

1 participant