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 ability to handle non-JSON endpoints #24

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

Conversation

aogail
Copy link

@aogail aogail commented Jan 5, 2024

Add an option to pass arbitrary HTTP request headers in the raw command, for endpoints such as [app]/sso/saml/metadata that are not JSON. Also do not attempt to parse the body as JSON if the content type indicates something other than JSON.

Example usage:

$ okta-cli raw -H 'Accept: application/xml' 'apps/id/sso/saml/metadata'
<?xml version="1.0" encoding="UTF-8"?><md:EntityDescriptor entityID="http://www.okta.com/...

@flypenguin
Copy link
Owner

oh! love the MR, I kinda missed this. will try to resolve the conflicts soon :)

Add an option to pass arbitrary HTTP request headers in the `raw`
command, for endpoints such as `[app]/sso/saml/metadata` that are
not JSON. Also do not attempt to parse the body as JSON if the content
type indicates something other than JSON.
@aogail aogail force-pushed the feature/handleNonJsonEndpoints branch from a7bbc66 to 60f3357 Compare May 20, 2024 21:34
@aogail
Copy link
Author

aogail commented May 20, 2024

Hi @flypenguin, I rebased onto the current main to resolve the conflicts.

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