Skip to content

Update README.md (small typos) #910

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.

Sign up for GitHub

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update README.md (small typos) #910

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b1d2f47
Update README.md (small typos)
mweeden2 Apr 5, 2024
9f4aede
Update README.md (typo in key-pair auth section)
mweeden2 Apr 5, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,7 @@ The following examples show how you can include different types of special chara
);
```

Note that previously you needed to use a double equal sign (==) to escape the character. However, beginning with version 2.0.18, you can use a single equal size.
Note that previously you needed to use a double equals sign (==) to escape the character. However, beginning with version 2.0.18, you can use a single equals sign.

### Other Authentication Methods

Expand Down Expand Up @@ -274,7 +274,7 @@ If you are using a different method for authentication, see the examples below:

where:

- `{pathToThePrivateKeyFile}` is the path to the file containing the unencrypted private key.
- `{pathToThePrivateKeyFile}` is the path to the file containing the encrypted private key.
Copy link
Collaborator

@sfc-gh-mhofman sfc-gh-mhofman Apr 8, 2024

Choose a reason for hiding this comment

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

There is below an example showing it is not encrypted.

Copy link
Author

@mweeden2 mweeden2 Apr 8, 2024

Choose a reason for hiding this comment

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

Hmm. I'm not sure what example you mean.

The section just below this line does use an unencrypted key, but this line seems to be about the example just above ("Specify the file containing an encrypted private key").

Also, why would you need to also provide the private_key_pwd if the private key were not encrypted?

Let me know if I'm missing something.

Copy link
Collaborator

@sfc-gh-mhofman sfc-gh-mhofman Oct 6, 2025

Choose a reason for hiding this comment

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

@mweeden2
I think the same was spot here:
https://github.com/snowflakedb/snowflake-connector-net/pull/1053/files
Is this clear in this way?

- `{passwordForDecryptingThePrivateKey}` is the password for decrypting the private key.

- Specify an unencrypted private key (read from a file):
Expand Down