Skip to content

Fix typo 'securityOptions' => 'secureOptions'#3198

Open
scovetta wants to merge 1 commit into
request:masterfrom
scovetta:patch-1
Open

Fix typo 'securityOptions' => 'secureOptions'#3198
scovetta wants to merge 1 commit into
request:masterfrom
scovetta:patch-1

Conversation

@scovetta
Copy link
Copy Markdown

PR Checklist:

(No code was changed; this was a minor typo in the README.)

  • I have run npm test locally and all tests are passing.
  • I have added/updated tests for any new behavior.
  • If this is a significant change, an issue has already been created where the problem / solution was discussed: [N/A, or add link to issue here]

PR Description

I think this is a typo (nodejs/node-v0.x-archive#8608 and elsewhere in the request codebase). Passing in as 'securityOptions' would probably be a no-op, at least from first glance.

I think this is a typo (nodejs/node-v0.x-archive#8608 and elsewhere in the request codebase). Passing in as 'securityOptions' would probably be a no-op, at least from first glance.
@stale
Copy link
Copy Markdown

stale Bot commented Aug 25, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale Bot added the stale label Aug 25, 2020
@scovetta
Copy link
Copy Markdown
Author

Could this get merged? It's just a doc update.

@stale stale Bot removed the stale label Aug 25, 2020
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.

1 participant