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

Corrected minor typos for ease of understanding #1098

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

robertjamison
Copy link

Not a significant update. I'm really just trying to make my first contribution ever on GitHub.
Feel free to add to the next update.

Not a significant update, but should be included in the next distro.
//
// From SQLITE3 - user-auth.txt
// The sqlite_user.pw field is encoded by a built-in SQL function
// "sqlite_crypt(X,Y)". The two arguments are both BLOBs. The first argument
// is the plaintext password supplied to the sqlite3_user_authenticate()
// interface. The second argument is the sqlite_user.pw value and is supplied
// so that the function can extract the "salt" used by the password encoder.
// The result of sqlite_crypt(X,Y) is another blob which is the value that
// The result of sqlite_crypt(X,Y) is another blob, which is the value that
Copy link
Collaborator

@rittneje rittneje Oct 18, 2022

Choose a reason for hiding this comment

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

These paragraphs are copied directly from here: https://www.sqlite.org/src/doc/trunk/ext/userauth/user-auth.txt

@@ -13,24 +13,24 @@ import (

// This file provides several different implementations for the
// default embedded sqlite_crypt function.
// This function is uses a caesar-cypher by default
// This function includes a caesar-cypher by default
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// This function includes a caesar-cypher by default
// This function uses a Caesar cipher by default

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.

2 participants