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

Switch request reflector backend #111

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

athomason
Copy link
Contributor

Requests to http-me.glitch.me occasionally fail with 500 errors, leading to test flakes [1]. @BrooksCunningham suggests using the nearly compatible http.edgecompute.app [2] instead, which seems to be more reliable and substantially faster thanks to being hosted directly on Fastly Compute.

One required change is not sending invalid UTF-8 in the request body, which results in a 500 Internal Server Error.

  1. https://github.com/fastly/compute-sdk-go/actions/runs/8839087224/job/24271648933
  2. https://github.com/BrooksCunningham/http-me-rust

Requests to http-me.glitch.me occasionally fail with 500 errors, leading
to test flakes [1]. @BrooksCunningham suggests using the nearly
compatible http.edgecompute.app [2] instead, which seems to be more
reliable and substantially faster thanks to being hosted directly on
Fastly Compute.

One required change is not sending invalid UTF-8 in the request body,
which results in a 500 Internal Server Error.

1. https://github.com/fastly/compute-sdk-go/actions/runs/8839087224/job/24271648933
2. https://github.com/BrooksCunningham/http-me-rust
@athomason athomason marked this pull request as ready for review April 26, 2024 00:01
Copy link
Member

@dgryski dgryski left a comment

Choose a reason for hiding this comment

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

LGTM. What about all the other occurrences of http-me.glitch.me in the repo?

@joeshaw
Copy link
Member

joeshaw commented Apr 26, 2024

I bulk-changed httpbin.com at one point to http-me.glitch.me, so a similar bulk change to http.edgecompute.app would be a good idea.

@athomason
Copy link
Contributor Author

It would be easy to change the examples with the exception of e.g. "https://http-me.glitch.me/drip=2?wait=3000", // delay 3s + stream body 2s = 5s; the delay functionality isn't in http.edgecompute.app. I'll check how much work that would be to add.

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.

3 participants