-
Notifications
You must be signed in to change notification settings - Fork 3
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
Dev/1.15.2 #69
Dev/1.15.2 #69
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think my only request was to remove body for methods that do not support a body (according to the RFCs)
* | ||
* @param url URL to make request | ||
* @param header request header | ||
* @param body request body |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
options request does not have a body. I think it makes the most sense to remove parameter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed (replaced with "")
* | ||
* @param url URL to make request | ||
* @param header request header | ||
* @param body request body |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
options request does not have a body
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed (replaced with "")
* | ||
* @param url URL to make request | ||
* @param header request header | ||
* @param body request body |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
HEAD request does not have a body
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed (replaced with "")
* | ||
* @param url URL to make request | ||
* @param header request header | ||
* @param body request body |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nobody
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed (replaced with "")
As included in Ewon documentation, I have added the remaining supported HTTP(s) method which were not already included in SCHttpUtility.java.
This allows for access within the HTTP(s) request info package.
All methods supported by the Ewon RequestHTTPX system have been supported by SCHttpUtility, thus are supported using the request info classes/objects.
closes #67. The CHANGELOG.md file was previously migrated to the web documentation system, but the file persisted unexpectedly.
No description provided.