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

Fixes #1835 - Handles unused well-known routes with a 404 #2578

Merged
merged 1 commit into from
Sep 14, 2018

Conversation

karlcow
Copy link
Member

@karlcow karlcow commented Sep 14, 2018

This PR fixes issue #1835

Proposed PR background

We had a some errors polluting the log file with errors.
This helps us to gracefully handles them, and avoid polluting the log.

The max-age is set to 1 year.

@karlcow karlcow requested a review from miketaylr September 14, 2018 00:33
Copy link
Member

@miketaylr miketaylr left a comment

Choose a reason for hiding this comment

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

Thanks, Karl!

the route /.well-known/{subpath} doesn't exist.

Nothing behind me, everything ahead of me, as is ever so on the road.
- Jack Kerouac, On the Road.

This comment was marked as abuse.

@miketaylr miketaylr merged commit d910868 into webcompat:master Sep 14, 2018
@karlcow karlcow deleted the 1835/1 branch October 17, 2018 23:47
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.

2 participants