-
Notifications
You must be signed in to change notification settings - Fork 20
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 HEAD requests in RESTDataSource Class #27
Comments
datasource-rest
Is there any progress on this topic? I would love to use this enhancement :) |
Is there any update on this? I also need to make HEAD method request using RESTDataSource but there is no support for that. |
@abernix Any update? |
it’s not the best solution but if you need a fast solution just use fetch and do a HEAD request |
@abernix Any update on this? We recently ran into this issue. |
I don't have the precise answer as to the exact "why", but I could certainly believe that the omission of That said, data obviously comes in various forms and it's entirely possible to return certain valuable data as header values, so I think the need resonates with me too. Thanks for opening the pull request, @Phoenix1405. |
I just passed the header into my returned data as a symbol |
I believe |
Ok, having familiarized myself with
Based on this information, I'm going to assume people want access to the response headers when the make a I propose 2 possible options (with the option of 1, then later 2):
I'd be interested in hearing input on these options from anyone on this thread. I'm leaning towards 2 myself since it offers a more robust and consistent API. |
Some endpoints of my REST API can be requested via a HEAD request but head is not implemented yet.
The text was updated successfully, but these errors were encountered: