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

add http options command #9365

Merged
merged 5 commits into from Jun 9, 2023
Merged

add http options command #9365

merged 5 commits into from Jun 9, 2023

Conversation

WindSoilder
Copy link
Collaborator

@WindSoilder WindSoilder commented Jun 5, 2023

Description

closes: #9344

Different to other http commands, http options command always returns header, according to MDN, the response information is included in header.

User-Facing Changes

Tests + Formatting

After Submitting

@fdncred
Copy link
Collaborator

fdncred commented Jun 5, 2023

nice work. seems to work as advertised when i tested it.

@fdncred fdncred merged commit 1edd3e7 into nushell:main Jun 9, 2023
15 of 16 checks passed
@WindSoilder WindSoilder deleted the http_options branch August 2, 2023 22:05
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.

Add support for HTTP OPTIONS method
3 participants