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.

Sign up for GitHub

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

Jump to bottom

Allow disabling redirect following in actix-test #3356

Merged
merged 2 commits into from
Jun 7, 2024
Merged

Allow disabling redirect following in actix-test #3356

merged 2 commits into from
Jun 7, 2024

Conversation

mpalmer
Copy link
Contributor

@mpalmer mpalmer commented May 7, 2024 •
edited by robjtede

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

If you're testing that redirects are being properly generated, then it's useful to not have the client go off on a wild goose chase of its own.

@mpalmer
Allow disabling redirect following in actix-test
1c5d7b5 
If you're testing that redirects are being properly generated, then it's
useful to not have the client go off on a wild goose chase of its own.
@robjtede robjtede added B-semver-minor A-test-server project: test-server labels Jun 7, 2024
@robjtede robjtede enabled auto-merge June 7, 2024 14:23
robjtede
robjtede approved these changes Jun 7, 2024
View reviewed changes
Copy link
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.

nice addition, thanks 👍🏻

@robjtede robjtede added this pull request to the merge queue Jun 7, 2024
Merged via the queue into actix:master with commit 8b4d23a Jun 7, 2024
13 checks passed
@mpalmer mpalmer deleted the actix-test-disable-redirects branch June 8, 2024 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@robjtede robjtede robjtede approved these changes

Assignees
No one assigned
Labels
A-test-server project: test-server B-semver-minor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mpalmer @robjtede