sekwah41 / Advanced-Portals

An advanced portals plugin for bukkit
GNU Lesser General Public License v3.0
157 stars 65 forks source link

fix: Access to portal warp command locked behind wrong perm #275

Closed sekwah41 closed 3 years ago

sekwah41 commented 3 years ago

What Changed

The issue is with the current perm its both for the internal command code and spigot's system. So you get too much access.

📦 Published PR to discord as canary version: 0.7.0--canary.275.895fcc6

sekwah41 commented 3 years ago

Will be merged once its tested by the user which was having the issue.

Kamology commented 3 years ago

So, here's the series of events:

I downloaded the new jar, typed the command, as you can see it auto predicts so it's properly registered, keep in mind i already had advancedportals.warp.warzone granted. image

This is the response I then got, same as before. image

I then made the following changes just to see if that changed anything: image

And then got this: image

sekwah41 commented 3 years ago

Remember to replace your username for this

but if you run the following commands

/luckperms user sekwah41 permission check advancedportals.warp.warzone
/luckperms user sekwah41 permission check advancedportals.warp

What are the outputs?

github-actions[bot] commented 3 years ago

:rocket: PR was released in v0.7.1 :rocket: