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

In some cases backslashes "\" are being removed from chat messages #13653

Open
binford6000 opened this issue Oct 28, 2024 · 4 comments
Open

In some cases backslashes "\" are being removed from chat messages #13653

binford6000 opened this issue Oct 28, 2024 · 4 comments

Comments

@binford6000
Copy link

Steps to reproduce

  1. Send a message with following string \\some\network\share\_Documentation
  2. Message is shown \some\network\share_Documentation in the chat.

Expected behaviour

Message in chat will be displayed like the original. Especially mandatory for windows network shares in general like \\server\share\folder
and specifically for \\server\share\_folder.

Actual behaviour

Doubled backslashes are reduced to one and leading backslash of an underscore "_"is removed.

Image

Talk app

Talk app version: 20.0.1

Custom Signaling server configured: yes (2.0.0)

Custom TURN server configured: yes

Custom STUN server configured: yes

Browser

Microphone available: yes

Camera available: yes

Operating system: macOS Sequoia 15.0.1

Browser name: Brave

Browser version: [Version 1.71.118 Chromium: 130.0.6723.70 (Offizieller Build) (arm64)]

@Antreesy
Copy link
Contributor

Antreesy commented Oct 28, 2024

@nickvergessen
Copy link
Member

nickvergessen commented Oct 28, 2024

Posting as code block works:
Image


GitHub comment for reference

Code block:

\\some\network\share\_Documentation

Code line:

\\some\network\share\_Documentation

Simple text:
\some\network\share_Documentation


Conclusion

Very same result in GitHub, so instead of "fixing" something, we might notify the user that we do markdown:

  • by showing formatting options on the input (can't find the ticket but it seems far away)
  • by a note when \\ is used in the text input.

@Antreesy
Copy link
Contributor

  • by a note when \\ is used in the text input.

Or any other combination detected as backslash + character from https://www.markdownguide.org/basic-syntax/#characters-you-can-escape

@SystemKeeper
Copy link
Contributor

We also discussed at one point to have an option to send raw messages, without markdown parsing, but seems we don't have a ticket for that?

  • can't find the ticket but it seems far away

nextcloud-libraries/nextcloud-vue#4355

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants