Re:Skin

Re:Skin

276k Downloads

Url whitelist blocking /setskin reset command

Kariaro opened this issue ยท 4 comments

commented

If this was an intended functionallity I would class this as a bug introduced by the url checking algorithm.

This error is generated not from the execute method but from the URLArgument type class.

commented

Because an old feature that was an alternative that I just hadn't fully removed. I should remove that code.

commented

See the following code

CustomSkinManager.setSkin(target, "reset");

That is why this exists.

commented

It's for internally tracking when it should be set back as it's not possible as a normal set string.

commented

private static int execute(CommandSource source, Collection<ServerPlayerEntity> targets, String skinUrl) {

Why is the setskin command skipping the whitelist check if the input url is reset then?