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

Bug: Unable to send text with new lines as "\n" #1202

Closed
2 tasks done
bartoszpijet opened this issue Sep 23, 2022 · 2 comments
Closed
2 tasks done

Bug: Unable to send text with new lines as "\n" #1202

bartoszpijet opened this issue Sep 23, 2022 · 2 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@bartoszpijet
Copy link

Describe the bug

New lines written as "\n" cause database to response with "Parse error".
What's weird is that even sending "\n" it will still throw same error.
Only sending new line as real new line (pressing enter) will be accepted.

Problem is pretty annoying while sending JSON data from JS sdk.

Steps to reproduce

Command that will throw error:
CREATE test SET value = "sometext\nwithnewline"; CREATE test SET value = "sometext\\nwithnewline";

Expected behaviour

Save values with "\n" as text, allow to unescape it or just parse it as new line.

SurrealDB version

v1.0.0-beta.7 for linux amd64

Contact Details

No response

Is there an existing issue for this?

  • I have searched the existing issues

Code of Conduct

  • I agree to follow this project's Code of Conduct
@bartoszpijet bartoszpijet added bug Something isn't working triage This issue is new labels Sep 23, 2022
@bartoszpijet bartoszpijet changed the title Bug: Bug: Unable to send text with new lines as "\n" Sep 23, 2022
@tobiemh
Copy link
Member

tobiemh commented Sep 23, 2022

Hi @bartoszpijet thanks for this! This is covered in #90, #137, and #245.

We have a fix coming really soon for this 😀👍!

@tobiemh tobiemh removed the triage This issue is new label Sep 23, 2022
@bartoszpijet
Copy link
Author

Hi, thanks for quick response. Sorry for unwanted bug report, wasn't able to find this issue here.

Have a great day!

@tobiemh tobiemh added this to the v1.0.0-beta.8 milestone Sep 24, 2022
@tobiemh tobiemh self-assigned this Sep 24, 2022
tobiemh added a commit that referenced this issue Sep 26, 2022
tobiemh added a commit that referenced this issue Sep 26, 2022
Closes #90
Closes #137
Closes #245
Closes #1202
Closes #1212
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants