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

docs: update options.encrypt default to true #1000

Merged
merged 1 commit into from Nov 5, 2019

Conversation

@davidjb
Copy link
Contributor

davidjb commented Nov 5, 2019

As of this commit (70ffe4a) the default is to encrypt by default, so update the documentation accordingly

As of this commit (70ffe4a) the default is to encrypt by default, so update the documentation accordingly
davidjb added a commit to davidjb/tedious that referenced this pull request Nov 5, 2019
This explains the encryption-by-default option whilst removing example that indicates that `{encrypt: true}` is necessary now.  See also tediousjs#1000
@arthurschreiber arthurschreiber merged commit 23fbdc3 into tediousjs:gh-pages Nov 5, 2019
1 check failed
1 check failed
LGTM analysis: JavaScript Analysis failed (could not build the merge commit and the base commit (3020b54))
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.