search

discord / embedded-app-sdk

🚀 The Discord Embedded App SDK lets you build rich, multiplayer experiences as Activities inside Discord.
https://discord.com/developers/docs/activities/overview
MIT License
1.26k stars 176 forks source link

fix: prevent multiple remappings from applying the .proxy path twice #252

Closed matthova closed 1 month ago

matthova commented 1 month ago

We already do this in attemptRemap but we need to apply it in matchAndRewriteURL as well.

It's worth asking, why are we checking this twice? Why are we adding /.proxy in two places? In case the original url has no matching urls to map to, we still want to inject the /.proxy path.