The outcome is pretty much the same as in the issue: Some people think it’s better to keep the params, and the rest agree that at least it’s better to keep them for now
A perspective on the options: With the params, it puts more responsibility (and potentially pressure) on the maintainers, if they are removed the users have to do much more due diligence which snapshot is ok to use. But the thread to the users is a much more practical attack, at least it seems like it shortly.
Removing the params takes away the chance for users to skip the background sync entirely in the future (launching into a pruned node state). Not everyone agrees that this would be a useful feature any time soon. In its current state, this would also screw up nchaintx.