Summary
A logic error in the main summaly function causes the allowRedirects option to never be passed to any plugins, and as a result, isn't enforced.
Details
In the main summaly function, a new scrapingOptions object is created and passed to either the matched plugin, if any, or the default summarize function. The issue here is that the new scrapingOptions object is not provided the allowRedirects property of opts.
PoC
- Publish a post containing a link to any URL that redirects on Misskey.
- A preview will be generated for the target of the redirect, despite Misskey passing
allowRedirects: false.
Impact
Misskey will follow redirects, despite explicitly requesting not to.
References
Summary
A logic error in the main
summalyfunction causes theallowRedirectsoption to never be passed to any plugins, and as a result, isn't enforced.Details
In the main
summalyfunction, a newscrapingOptionsobject is created and passed to either the matched plugin, if any, or the default summarize function. The issue here is that the newscrapingOptionsobject is not provided theallowRedirectsproperty ofopts.PoC
allowRedirects: false.Impact
Misskey will follow redirects, despite explicitly requesting not to.
References