Token to request ChatGPT thru a proxy

[ThierryBachmann]

A request to ChatGPT thru the proxy discord.pawan doesn't work.

the token prefix sess- is retained in the bearer.

To check, a brutal action by removing the 5 first characters has corrected the issue.

headers.Add("Authorization", Rest.GetBearerOAuthToken(Authentication.Info.ApiKey.Substring(5)));

Perhaps, there is something more cleverly to correct.

[StephenHodgson]

I designed the proxy service to reflect a session token based mechanism similar to the one openAI uses for chat gpt.

More info here https://github.com/RageAgainstThePixel/OpenAI-DotNet/blob/main/OpenAI-DotNet-Proxy/Readme.md

If this is a proxy service you control, I'd prefer you use the sess- prefix. If it is not, then maybe contact the maintainer and request that they support it. This is probably not something I will be changing.

[StephenHodgson]

@ThierryBachmann Checkout the latest release. #185

[ThierryBachmann]

it's a good practice, sk- for chatGpt, pk- for pawan.
But, why adding another prefix before the prefix?
Why create a dependency between your API and pawan?
Clearly, no one will be able to use your framework with pawan or any others proxies.

[StephenHodgson]

I don't mind adding the ability to set a different prefix. This should be possible I think.

Clearly, no one will be able to use your framework with pawan or any others proxies.

I built this specifically for people managing their own proxies, not necessarily using a 3rd party one.
Personally, I would be apprehensive to do so for security concerns.

[ThierryBachmann]

In OpenAIClient.cs we have this check:

(!Authentication.Info.ApiKey.Contains(OpenAIAuthInfo.SecretKeyPrefix) &&
 !Authentication.Info.ApiKey.Contains(OpenAIAuthInfo.SessionKeyPrefix))))

So only key starting with sess- or sk- are authorized.
One possibilities is to open this choices with your API to others prefixes (pk- for pawan proxy)

[StephenHodgson]

Sounds good. I think being able to customize the prefix should be possible.

[ThierryBachmann]

Finally, Pawan proxy seems accept prefix like sess-pk-
I got another issue from Pawan (created attribut is not in second and not suit in an integer)
Obviously, I will check with Pawan for this issue.
thanks