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

Caddy - Add the ability to create redirects #4221

Closed
The-Istar opened this issue Sep 4, 2024 · 3 comments · Fixed by #4263
Closed

Caddy - Add the ability to create redirects #4221

The-Istar opened this issue Sep 4, 2024 · 3 comments · Fixed by #4263
Labels
feature Adding new functionality help wanted Contributor missing

Comments

@The-Istar
Copy link

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
this feature request is requested here:
There is currently no way to create redirects in the GUI, though it is a useful feature.

Describe the solution you'd like
A clear and concise description of what you want to happen.
(e.g. I would like an input field in the /ui/firewall/alias which would add .... to ....)
I would like to be able to create redirects in the GUI

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.
See here:

@Monviech
Copy link
Member

Monviech commented Sep 4, 2024

Hey there,

could you fill out the template a little more?

What should be the scope of that feature? Free form like e.g., the HTTP Headers menu?

Or is the scope just these nextcloud redirections as of this reddit thread: https://www.reddit.com/r/opnsense/s/sIjv5Plx2p

Give an example of the redirects you need and the results you expect.

@The-Istar
Copy link
Author

HTTP Headers menu seems fine. Basically it would be to integrate the redir directive, which is a simple enough command.

redir original_path new_path redirect_code

I suggest Original_path and New_path to be freeform and redirect_code to be a drop down?
That way it would work for all other situations that require a redirect as well.

Here is the example from the Nextcloud docs:

@Monviech Monviech self-assigned this Sep 4, 2024
@Monviech Monviech added feature Adding new functionality help wanted Contributor missing labels Sep 4, 2024
@Monviech Monviech removed their assignment Sep 5, 2024
@Monviech
Copy link
Member

Monviech commented Sep 5, 2024

I gonna keep this open if somebody wants to offer a PR here. I currently lack the time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding new functionality help wanted Contributor missing
Development

Successfully merging a pull request may close this issue.

2 participants