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

potential fix to the bug im having #23622

Closed
wants to merge 1 commit into from

Conversation

ciaranmcguinness
Copy link

No description provided.

Copy link

Connected to Huly®: V_0.6-22046

@ciaranmcguinness
Copy link
Author

what? i hate github

@felipensp
Copy link
Member

what? i hate github

I have reopened it.

@spytheman
Copy link
Member

Can you please describe in more details what problem this PR solves, and potentially provide a regression test, that fails on master, but passes here?

mut status_code := ""
mut status_msg := ""
//this may be a hack, but it means i can get free completions from ai.hackclub.com
//fix sites that send malformed responses with leading newlines
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well... I think we must avoid handling things out of documented standards.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Returning an error would be more interesting to respond to a bad request error. IMHO

@ciaranmcguinness
Copy link
Author

this was supposed to be a pr on my own repo, this is not at all ready for prod

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.

4 participants