Custom Site for viewing a Gauntlet's levels #233

2022-05-27T11:58:51-04:00

DerOwnerHD commented

2022-05-27 11:58:51 -04:00

(Migrated from github.com)

This PR adds a GD-like menu for viewing the levels in a Gauntlet.

It's accessible when clicking on a Gauntlet in the Gauntlets menu, which previously redirected to the search menu (bc Colon was too lazy to add such a menu).
The search menu has not been messed with and it still allows Gauntlets to be shown.

If a Gauntlet is unknown, e.g. a custom one from a private server, it will redirect to the search menu (just like the old days)

This PR adds a GD-like menu for viewing the levels in a Gauntlet. It's accessible when clicking on a Gauntlet in the Gauntlets menu, which previously redirected to the search menu (bc Colon was too lazy to add such a menu). The search menu has not been messed with and it still allows Gauntlets to be shown. If a Gauntlet is unknown, e.g. a custom one from a private server, it will redirect to the search menu (just like the old days)

GDColon commented

2022-05-29 13:35:40 -04:00

(Migrated from github.com)

Oh this is cool! Will have to think about it though, since I find the current system to be a lot more convenient

github-advanced-security[bot] (Migrated from github.com) reviewed 2022-07-25 10:53:57 -04:00

index.js

					
				@ -270,6 +270,7 @@ app.get("/mappacks", function(req, res) { res.status(200).sendFile(__dirname + "

				app.get("/messages", function(req, res) { res.status(200).sendFile(__dirname + "/html/messages.html") })

				app.get("/search", function(req, res) { res.status(200).sendFile(__dirname + "/html/filters.html") })

				app.get("/search/:text", function(req, res) { res.status(200).sendFile(__dirname + "/html/search.html") })

				app.get("/gauntlet/:id", function(req, res) { res.status(200).sendFile(__dirname + "/html/gauntlet.html") })

github-advanced-security[bot] (Migrated from github.com) commented

2022-07-25 10:53:57 -04:00

Missing rate limiting

This route handler performs a file system access, but is not rate-limited.

Show more details

## Missing rate limiting This route handler performs [a file system access](1), but is not rate-limited. [Show more details](https://github.com/GDColon/GDBrowser/security/code-scanning/68)

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin DerOwnerHD/master:DerOwnerHD/master

git checkout DerOwnerHD/master

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout master

git merge --no-ff DerOwnerHD/master

git checkout DerOwnerHD/master

git rebase master

git checkout master

git merge --ff-only DerOwnerHD/master

git checkout DerOwnerHD/master

git rebase master

git checkout master

git merge --no-ff DerOwnerHD/master

git checkout master

git merge --squash DerOwnerHD/master