feat(ingressnginx): implement redirect annotation support

[asklokesh]

Summary

• Add support for nginx-ingress redirect annotations
• permanent-redirect with optional permanent-redirect-code (default 301)
• temporal-redirect (default 302)
• force-ssl-redirect enables SSL redirect even without TLS configuration
• Validates port range (1-65535)
• Warns when query string/fragment in redirect URL will be ignored (Gateway API limitation)

Test plan

• Added unit tests for createRedirectFilter with various URL formats
• Added unit tests for redirectFeature feature parser
• Added test for force-ssl-redirect annotation
• Added test for invalid port validation
• All existing tests pass

Fixes #268

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: asklokesh / name: Lokesh Mure (5c8e329)

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: asklokesh
Once this PR has been reviewed and has the lgtm label, please assign stevenjin8 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[k8s-ci-robot]

Hi @asklokesh. Thanks for your PR.

I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[asklokesh]

/easycla

[jgreeer]

Hey @asklokesh, I have an PR open for nginx.ingress.kubernetes.io/permanent-redirect and nginx.ingress.kubernetes.io/temporal-redirect here. The status codes won't be supported until gwapi v1.5, so we'd either want to wait to merge or emit a warning for those. I don't parse nginx.ingress.kubernetes.io/force-ssl-redirect so you could keep this PR for that.

[asklokesh]

Closing this PR as there is already an existing PR #299 by @jgreeer that implements redirect annotations (permanent-redirect and temporal-redirect). The force-ssl-redirect annotation was already implemented in PR #290.