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.

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

insert zero width space in nicks on irc #175

Closed
NikkyAI opened this issue May 23, 2017 · 11 comments
Closed

insert zero width space in nicks on irc #175

NikkyAI opened this issue May 23, 2017 · 11 comments

Comments

@NikkyAI
Copy link
Contributor

NikkyAI commented May 23, 2017

Please answer the following questions.

Which version of matterbridge are you using?

run matterbridge -version
version: 0.12.0 8c7549a

Please describe the expected behavior.

prefixed nicknames on irc should have a no ping character or zero width space U+200B

in the following line a zero width space is inserted at position 1 of the nickname
<N​ickname> some message

Please describe the actual behavior.

there is no inserted zwsp and there seem to be no options to enable such behaviour

Please add your configuration file

(be sure to exclude or anonymize private data (tokens/passwords))

[irc]
    [irc.esper]
    Server="irc.esper.net:6697"
    Nick="ruby" #effectively "ruby_" while testing
    
    UseTLS=true
    UseSASL=true

    NickServNick="username"
    NickServPassword="hunter2"

    IgnoreNicks="cord spammer"

[discord]
    [discord.chat]
    Token="token"
    Server="servername"
    EditDisable=false
    EditSuffix=" (edited)"
    IgnoreNicks="[cord] spammer"
    RemoteNickFormat="<{NICK}> "
    ShowJoinPart=false

[[gateway]]
#REQUIRED and UNIQUE
name="cord"
enable=true
    [[gateway.inout]]
    account="irc.esper"
    channel="#cord"

    [[gateway.inout]]
    account="discord.chat"
    channel="cord"
@42wim
Copy link
Owner

42wim commented May 23, 2017

Sorry, I don't really understand.
You want to insert a ZWSP after the nick ? and it doesn't work by using the RemoteNickFormat, is that the issue ?

@NikkyAI
Copy link
Contributor Author

NikkyAI commented May 23, 2017

i am not sure how the remotenickformat would allow me to split the string and insert something in the middle

and not after the nickname.. inside it

so that Nickname turns into N ickname for example.. just obviously not visible

@42wim
Copy link
Owner

42wim commented May 23, 2017

You want to insert a ZWSP inside a nick ? why?
RemoteNickFormat="<{NICK}> " is what I am talking about

@NikkyAI
Copy link
Contributor Author

NikkyAI commented May 23, 2017

because people who are on both platforms ping themself when chatting .. and yes their irc clients are stupid but that seems to be the easiest way to fix this

@NikkyAI
Copy link
Contributor Author

NikkyAI commented May 23, 2017

and ignoring the bot would not really help because then other users that send through the bot cannot ping you

@42wim
Copy link
Owner

42wim commented May 23, 2017

Ok, I understand now.
Let me think about it.

@42wim
Copy link
Owner

42wim commented May 29, 2017

Ok, I've added this feature in master.
You can now use {NOPINGNICK}, this will insert a ZWSP inside of the nick.

@42wim 42wim closed this as completed in c791423 May 29, 2017
Tjzabel added a commit to RITlug/teleirc that referenced this issue May 17, 2020
…ross platforms.

ResolveUserName is now used instead of user.String() for retrieving username information.
For additional details, see: 42wim/matterbridge#175.
Tjzabel added a commit to RITlug/teleirc that referenced this issue May 23, 2020
…ross platforms.

ResolveUserName is now used instead of user.String() for retrieving username information.
For additional details, see: 42wim/matterbridge#175.
jwflory pushed a commit to RITlug/teleirc that referenced this issue May 24, 2020
* Add zero width space between Telegram usernames to prevent pinging across platforms.

ResolveUserName is now used instead of user.String() for retrieving username information.
For additional details, see: 42wim/matterbridge#175.

* Add config check for ZWSP.
Helper methods now better reflect their actions.

* Make return values more concise.
@Mikaela
Copy link

Mikaela commented Aug 20, 2020

In case it's of interest here, so far the clients I have been told about that don't support ZWSP are:

@uduntuntu
Copy link

In case it's of interest here, so far the clients I have been told about that don't support ZWSP are:

image
Neat example the situation of my nick

I think the ZWSP is copied here from my Windows Terminal window.
19:02:47 <+MI1> [T] <A​riMartti>

@42wim
Copy link
Owner

42wim commented Aug 23, 2020

Yes, noticed it too, anyone an idea for workaround ?:)

@Juliaria08
Copy link

Juliaria08 commented Dec 7, 2021

Just ask them to fix it, should be that a better workaround, because this is the use of zwsp, no?

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

No branches or pull requests

5 participants