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

[Bug]: Disable cache on web application's index.html #4127

Closed
2 tasks done
rbren opened this issue Sep 30, 2024 · 10 comments
Closed
2 tasks done

[Bug]: Disable cache on web application's index.html #4127

rbren opened this issue Sep 30, 2024 · 10 comments
Labels
bug Something isn't working fix-me Attempt to fix this issue with OpenHands severity:medium Affecting multiple users

Comments

@rbren
Copy link
Collaborator

rbren commented Sep 30, 2024

Is there an existing issue for the same bug?

Describe the bug

For the root path / and /index.html, we need to make sure that there is no caching set up. Otherwise the frontend breaks on update.

This will involve editing openhands/server/listen.py, specifically the part that serves the web app at the bottom of the file

Current OpenHands version

0.9.7

Installation and Configuration

N/A

Model and Agent

No response

Operating System

No response

Reproduction Steps

No response

Logs, Errors, Screenshots, and Additional Context

No response

@rbren rbren added bug Something isn't working fix-me Attempt to fix this issue with OpenHands labels Sep 30, 2024
Copy link
Contributor

OpenHands started fixing the issue! You can monitor the progress here.

Copy link
Contributor

The workflow to fix this issue encountered an error. Please check the workflow logs for more information.

@neubig neubig added fix-me Attempt to fix this issue with OpenHands and removed fix-me Attempt to fix this issue with OpenHands labels Sep 30, 2024
Copy link
Contributor

OpenHands started fixing the issue! You can monitor the progress here.

@neubig neubig added fix-me Attempt to fix this issue with OpenHands and removed fix-me Attempt to fix this issue with OpenHands labels Sep 30, 2024
Copy link
Contributor

OpenHands started fixing the issue! You can monitor the progress here.

Copy link
Contributor

A potential fix has been generated and a draft PR #4129 has been created. Please review the changes.

@amanape
Copy link
Member

amanape commented Oct 14, 2024

Does this issue still persist? Cache shouldn't be enabled by default.

@rbren
Copy link
Collaborator Author

rbren commented Oct 14, 2024

I frequently have to cmd+shift+R when loading up a new version of OpenHands...

@mamoodi
Copy link
Collaborator

mamoodi commented Oct 16, 2024

This would make a big difference for usability. I've had to do this way too many times.

@mamoodi
Copy link
Collaborator

mamoodi commented Nov 11, 2024

@rbren was this fixed by any chance? I thought I saw a PR go by related to this.

@rbren
Copy link
Collaborator Author

rbren commented Nov 11, 2024

Hypothetically yes, but I think I saw an issue w/ it again recently. Let's close this one for now and I'll keep an eye out

@rbren rbren closed this as completed Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fix-me Attempt to fix this issue with OpenHands severity:medium Affecting multiple users
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants