Skip to content

Testing web routes#328

Draft
carver wants to merge 2 commits intoethereum:masterfrom
carver:route-testing
Draft

Testing web routes#328
carver wants to merge 2 commits intoethereum:masterfrom
carver:route-testing

Conversation

@carver
Copy link
Contributor

@carver carver commented Oct 14, 2024

I wanted to add a test that routing to static files from the root path works.

I made some progress, but ran out of time so I have to pause work on it for now.

While adding this, I discovered that the fallback service was
overwritten by a 404 fallback, so static files were not being served
correctly from the root URL: http://glados/

This 404 handler was first added in
ethereum#64
but this seems to have been a mistake, since the fallback doesn't get
invoked in the use case for the issue.

I confirmed that 404s are still returned by a missing url like:
http://glados/UNSUPPORTED_PATH
@carver carver self-assigned this Oct 14, 2024
@mrferris mrferris moved this from In Progress to Prioritized Feature Backlog in Glados May 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Prioritized Feature Backlog

Development

Successfully merging this pull request may close these issues.

1 participant