Skip to content

Adjust cornucopia redirect(s) #352

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

Merged
merged 3 commits into from
Jun 8, 2025
Merged

Conversation

kingthorin
Copy link
Contributor

No description provided.

@kingthorin
Copy link
Contributor Author

Nudge @mamicidal 😀

@sydseter
Copy link

Question. I have never seen this before.

These take into account that

copi.owasp.org is the permanent domain for Copi - the Cornucopia game engine and that cornucopia.owasp.org is the permanent home of the Cornucopia website right?

@sydseter
Copy link

sydseter commented May 27, 2025

Reson I ask is because I see the redirect_from: /copi (which is it’s own site)

And I wonder wether that means there will be a redirect away from copi.owasp.org?

@kingthorin
Copy link
Contributor Author

kingthorin commented May 27, 2025

owasp.org/copi currently redirects to: https://owasp.org/www-project-cornucopia/ that stays in place.
owasp.org/cornucopia currently 404s (doesn't exist), but after this will redirect to the same project page.

If you'd rather owasp.org/copi redirect to copi.owasp.org I can update the PR, just lemme know.

@sydseter
Copy link

Sure, that seems perfectly fine.

@sydseter
Copy link

owasp.org/copi should redirect to copi.owasp.org

@sydseter
Copy link

copi is part of cornucopia, but it’s less confusing, that way, I think?

@kingthorin
Copy link
Contributor Author

Adjusted

@kingthorin
Copy link
Contributor Author

@mamicidal 😀
I believe this is good to go now.

@mamicidal mamicidal merged commit 1f97fab into OWASP:main Jun 8, 2025
@kingthorin kingthorin deleted the corn-redir branch June 8, 2025 21:03
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.

4 participants