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

MEDIUM: api: add support for custom response code redirects #53

Merged
merged 1 commit into from
Apr 10, 2023

Conversation

daniel-corbett
Copy link
Contributor

This commit adds support for performing 301 and 302 redirects to a defined url using custom response codes.

It also updates the tests for the default blocking code as it now supports changing the default from 406 to 301,302, or 400-599.

This commit adds support for performing 301 and 302 redirects to a
defined url using custom response codes.

It also updates the tests for the default blocking code as it now
supports changing the default from 406 to 301,302, or 400-599.
daniel-corbett added a commit to daniel-corbett/terraform-provider-sigsci that referenced this pull request Apr 6, 2023
This commit allows updating the default WAF blocking code. It supports
blocking using 301 or 302 with a custom URL as well as 400-599. It also
extends support for the existing site rule blocking codes to allow for
the same response codes.

This commit requires the merge of signalsciences/go-sigsci#53
@shawnps shawnps merged commit 17c8f12 into main Apr 10, 2023
daniel-corbett added a commit to signalsciences/terraform-provider-sigsci that referenced this pull request Apr 10, 2023
This commit allows updating the default WAF blocking code. It supports
blocking using 301 or 302 with a custom URL as well as 400-599. It also
extends support for the existing site rule blocking codes to allow for
the same response codes.

This commit requires the merge of signalsciences/go-sigsci#53
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