Skip to content

Conversation

@ishaan26
Copy link
Contributor

DEFINE LOGIN is incorrect. It should be DEFINE USER

`DEFINE LOGIN` is incorrect. It should be `DEFINE USER`
@ishaan26 ishaan26 changed the title fix: Corrected surrelql statement. fix: Corrected surrealql statement. Jun 21, 2025
@netlify
Copy link

netlify bot commented Jun 21, 2025

Deploy Preview for nostalgic-ptolemy-b01ab8 ready!

Name Link
🔨 Latest commit d53e435
🔍 Latest deploy log https://app.netlify.com/projects/nostalgic-ptolemy-b01ab8/deploys/6856df31ab38f40008f3cc04
😎 Deploy Preview https://deploy-preview-7291--nostalgic-ptolemy-b01ab8.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Rajakavitha1
Copy link
Collaborator

Thank you for the PR. Tested and validated the steps.

rajie@surreal:~$ surreal sql --conn http://localhost:8000 --user root --pass exampleRootPass --ns application --db todo --pretty
1 | DEFINE LOGIN exampleUser ON DATABASE PASSWORD 'examplePass';
# |        ^^^^^ 
#  Welcome to the SurrealDB SQL shell
#
#  How to use this shell:
#    - Different statements within a query should be separated by a (;) semicolon.
#    - To create a multi-line query, end your lines with a (\) backslash, and press enter.
#    - To exit, send a SIGTERM or press CTRL+C
#
#  Consult https://surrealdb.com/docs/cli/sql for further instructions
#
#  SurrealDB version: 2.3.6
#
		
application/todo> DEFINE USER exampleUser ON DATABASE PASSWORD 'examplePass';
-- Query 1 (execution time: 739.352µs)
NONE

application/todo> DEFINE LOGIN exampleUser ON DATABASE PASSWORD 'examplePass'; --< Parse error: Unexpected token `an identifier`, expected a define statement keyword
 --> [1:8]
  |
1 | DEFINE LOGIN exampleUser ON DATABASE PASSWORD 'examplePass';
  |        ^^^^^ 

Copy link
Collaborator

@Rajakavitha1 Rajakavitha1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
Tested and validated.

@jddocs jddocs added the Ready label Jun 30, 2025
@jddocs jddocs merged commit 7340916 into linode:develop Jun 30, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants