Skip to content

Add support for permanent redirects in Files service#3767

Merged
robjtede merged 4 commits intoactix:masterfrom
francesco-gaglione:configurable-redirect
Sep 15, 2025
Merged

Add support for permanent redirects in Files service#3767
robjtede merged 4 commits intoactix:masterfrom
francesco-gaglione:configurable-redirect

Conversation

@francesco-gaglione
Copy link
Copy Markdown
Contributor

@francesco-gaglione francesco-gaglione commented Sep 11, 2025

PR Type

Feature

PR Checklist

  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • A changelog entry has been made for the appropriate packages.
  • Format code with the latest stable rustfmt.
  • (Team) Label with affected crates and semver status.

Overview

Current behavior redirect only with FOUND.
This pull request:

  • Change default redirect to temporary redirect
  • Add support for permanent redirects in Files service

Close #3543

@JohnTitor JohnTitor requested a review from robjtede September 15, 2025 08:22
@JohnTitor JohnTitor added A-files project: actix-files B-semver-patch labels Sep 15, 2025
Copy link
Copy Markdown
Member

@robjtede robjtede left a comment

Choose a reason for hiding this comment

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

looks pretty good, thanks 👍🏻

@robjtede robjtede enabled auto-merge September 15, 2025 08:24
@robjtede robjtede added this pull request to the merge queue Sep 15, 2025
Merged via the queue into actix:master with commit 6e6c249 Sep 15, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-files project: actix-files B-semver-patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Files::redirect_to_slash_directory() should use HTTP status code 308

3 participants