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

v3.5.0 #1140

Merged
merged 158 commits into from
Sep 1, 2024
Merged

v3.5.0 #1140

merged 158 commits into from
Sep 1, 2024

Conversation

gantoine
Copy link
Member

No description provided.

gantoine and others added 30 commits July 15, 2024 18:27
* Add typing for nested objects in Tinfoil and Webrcade feed schemas.
* Do not send Rom background and thumbnail, if not available.
* Correctly build URLs using `starlette` utils.
* Deprecate the `ROMM_HOST` setting, no longer needed.
* Fix FastAPI custom router to prefer routes without trailing slash.
* Fix Webrcade background URL pointing to gallery screenshot.
misc: Improve typing for feed schemas
[ROMM-1063] Add Amiga CD32 to emulatorjs game list
fix: Readable setup text on white theme
Currently, the `request.url_for` and `URLPath.make_absolute_url` methods
always build URLs with "http" scheme, even when the original requested
URL is using "https".

The reason for this is that Gunicorn does not allow IPs other than
127.0.0.1 to set secure headers by default. As regular RomM
installations don't know which frontend IPs will try to set security
headers in advance, we can disable this validation, and fix URL
building.

A simple way to test this change is to access any of the `feed` endpoints,
which generate URLs using the mentioned methods. Accessing the endpoint
using "https" scheme must generate "https" URLs.

Reference:
* encode/starlette#538 (comment)
* https://docs.gunicorn.org/en/stable/settings.html#forwarded-allow-ips
This matches the behavior of the Webrcade feed.
gantoine and others added 27 commits August 29, 2024 10:09
[ROMM-1107] Add env variable to disable in-browser emulation
Default both fast forward and rewind to enabled
Co-authored-by: Georges-Antoine Assi <3247106+gantoine@users.noreply.github.com>
Copy link

github-actions bot commented Aug 30, 2024

Test Results

81 tests   81 ✅  21s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 72fc6d8.

♻️ This comment has been updated with latest results.

@gantoine gantoine marked this pull request as ready for review September 1, 2024 01:32
@gantoine gantoine merged commit 410007b into release Sep 1, 2024
11 checks passed
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