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

Support pushing the webserver to fly.io instead of heroku #919

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

Conversation

tresni
Copy link

@tresni tresni commented Apr 29, 2023

The old jsonpath.herokuapp.com appears to have gone offline when Heroku discontinued free services. Fly.io still offers a basic free service that is perfect for running this app.

I've set that up an organization on fly.io that I'm happy to move over to someone elses control. The benefit is that it has the domain https://jsonpath.fly.dev mirroring as close as possible the old herokuapp.com domain.

@coffee-coder99
Copy link

@tresni thank you for posting this. It was helpful to see what the server actually uses. Things don't work in jsonpath.com vs javainuse, but I see the same issues with jsonpath.fly.dev, inconsistent results on what I would expect. I am willing to bet your site is the closest. Is it still the most recent?

As an aside,
In the network tab, when I select Go! it makes a request to eval and it looks like your server is evaluating with all 4 types of evaluators. It might be a good optimization to only evaluate the current tabs evaluator.

@tresni
Copy link
Author

tresni commented Nov 5, 2023

There have been no updates to JsonPath since I pushed that image to fly.io, so what you see there matches the master branch. The code for the site and everything was already in place, I just put a new deployment method in place.,

@valfirst
Copy link
Contributor

json-path-web-test subproject was removed: #994

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