Thread

14 comments

JJustin

I believe for the API you need to use the Authorization Header

JJustin

@st_bokarev can probably help some more

ccvb941

okay, I changed it to Authorization, now I get this

Attachment

JJustin

Hey @cvb941 sorry for the delay. Do you think we can hop on a quick call and I can help debug?

JJustin

I think the path is wrong

JJustin

I think you need to do /v1/request/{id}/feedback

ccvb941

Hi, I tried that and this is the response now

Plain Text

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="utf-8">
    <title>Error</title>
</head>
<body>
<pre>ValidateError<br> &nbsp; &nbsp;at ExpressTemplateService.getValidatedArgs (/usr/src/app/valhalla/jawn/node_modules/@tsoa/runtime/dist/routeGeneration/templates/express/expressTemplateService.js:70:19)<br> &nbsp; &nbsp;at RequestController_feedbackRequest (/usr/src/app/valhalla/jawn/dist/valhalla/jawn/src/tsoa-build/public/routes.js:1705:45)<br> &nbsp; &nbsp;at Layer.handle [as handle_request] (/usr/src/app/valhalla/jawn/node_modules/express/lib/router/layer.js:95:5)<br> &nbsp; &nbsp;at next (/usr/src/app/valhalla/jawn/node_modules/express/lib/router/route.js:149:13)<br> &nbsp; &nbsp;at runAuthenticationMiddleware (/usr/src/app/valhalla/jawn/dist/valhalla/jawn/src/tsoa-build/public/routes.js:2680:17)<br> &nbsp; &nbsp;at process.processTicksAndRejections (node:internal/process/task_queues:95:5)</pre>
</body>
</html>

ccvb941

This is the URL for that request: https://api.helicone.ai/v1/request/dacfe304-2900-476a-920c-7f7b2914e809/feedback

ccvb941

I can sure hop on a call

ccvb941

all right my bad, it works now!

ccvb941

There are at least two wrong URLs in the docs then for fixing
https://docs.helicone.ai/features/advanced-usage/feedback

aassaf

Great timing! I just implemented rating this morning and got a 404 because I copied the example code at the bottom of https://docs.helicone.ai/features/advanced-usage/feedback
Docs are missing the /request/ part of the path

Attachment

ccvb941

of course the Helicone-auth header needs to be changed to Authorization in the examples

sst_bokarev

Thank you for flagging that! Gonna fix this

Helicone Community Page

Thread