Solved Error with Developed Plugin

Hi all.
I'm running into a mess with my plugin.
I'll give you all the resources first:
IPCMain:

Show Spoiler Hide Spoiler

Code:java

package dev.flareline.ipcheck;
 
import java.io.File;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;
import java.util.logging.Logger;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Server;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.configuration.file.YamlConfiguration;
import org.bukkit.entity.Player;
import org.bukkit.event.player.PlayerLoginEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class IPCMain extends JavaPlugin {
 
    public static final String ANSI_RESET = "\u001B[0m";
    public static final String ANSI_CYAN = "\u001B[36m";
    public static final String ANSI_GREEN = "\u001B[32m";
    public static final String ANSI_RED = "\u001B[31m";
 
    static File configFile;
    static File loggerFile;
 
    static FileConfiguration config;
    static FileConfiguration logger;
 
    @Override
    public void onDisable() {
        saveYamls();
    }
 
    @Override
    public void onEnable() {
 
        getServer().getPluginManager().registerEvents(new IPCListener(), this);
 
        configFile = new File(getDataFolder(), "config.yml");
        loggerFile = new File(getDataFolder(), "logger.yml");
 
        try {
            firstRun();
        } catch (Exception e) {
            e.printStackTrace();
        }
        config = new YamlConfiguration();
        logger = new YamlConfiguration();
        loadYamls();
 
    }
 
    private void firstRun() throws Exception {
        if (!configFile.exists()) {
            configFile.getParentFile().mkdirs();
            copy(getResource("config.yml"), configFile);
        }
        if (!loggerFile.exists()) {
            loggerFile.getParentFile().mkdirs();
            copy(getResource("logger.yml"), loggerFile);
        }
    }
 
    private void copy(InputStream in, File file) {
        try {
            OutputStream out = new FileOutputStream(file);
            byte[] buf = new byte[1024];
            int len;
            while ((len = in.read(buf)) > 0) {
                out.write(buf, 0, len);
            }
            out.close();
            in.close();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
 
    public static void loadYamls() {
        try {
            config.load(configFile);
            logger.load(loggerFile);
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
 
    public static void saveYamls() {
        try {
            config.save(configFile);
            logger.save(loggerFile);
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
 
    public void writeLogs(Player player) {
        String playername = player.getName();
        logger.createSection(playername);
        saveYamls();
    }
 
    public boolean onCommand(CommandSender sender, Command cmd, String label,
            String[] args) {
        if (sender instanceof Player) {
            if (cmd.getName().equalsIgnoreCase("ipc")) {
                sender.sendMessage(ChatColor.GOLD + "IPC");
                return true;
            }
        } else {
            sender.sendMessage(ChatColor.GOLD + "You must be ingame to use IPC");
        }
        return false;
    }
 
    public static void checker(String playername, String address,
            PlayerLoginEvent event) {
        Server server = Bukkit.getServer();
        Logger console = server.getLogger();
        console.info(ANSI_CYAN + playername + "'s IP Address is: " + address
                + ANSI_RESET);
        loadYamls();
        String playeraddress = logger.getString(playername + ".join.address");
        if (playeraddress != null) {
            console.info(ANSI_GREEN + playername
                    + "'s Address already exists: " + playeraddress
                    + ANSI_RESET);
            saveYamls();
        } else {
            logger.set(playername + ".join.address", address);
            console.info(ANSI_RED + "Adding " + playername
                    + "'s IP in Logger: " + address + ANSI_RESET);
            saveYamls();
        }
        boolean allowed = configurationChecker(playername, address);
        Player player = server.getPlayer(playername);
        IPCMain.denylogin(player, address, event, allowed, playername);
    }
 
    public static boolean configurationChecker(String playername, String address) {       
        loadYamls();
        String player = config.getString("exceptions.addresses." + address);
        if (player != null) {
            if (playername.equals(player)) {
                return true;
            } else {
                return false;
            }
        }
        return false;
    }
 
    public static void denylogin(Player player, String address,
            PlayerLoginEvent event, boolean allowed, String playername) {
        Server server = Bukkit.getServer();
        Logger console = server.getLogger();
        String kickmessage = "Potential Account Theft. Denied Login.";
        console.info(ANSI_CYAN + "Testing Validity Of: " + playername
                + ANSI_RESET);
        String realaddress = logger.getString(playername + ".join.address");
        if (allowed = true) {
            console.info(ANSI_GREEN + "Address Matches Exception In config.yml"
                    + ANSI_RESET);
            console.info(ANSI_CYAN + "Allowing " + playername + " To Join!"
                    + ANSI_RESET);
        } else {
            if (address.equalsIgnoreCase(realaddress)) {
                console.info(ANSI_GREEN + "Address Matches!" + ANSI_RESET);
                console.info(ANSI_CYAN + "Allowing " + playername + " To Join!"
                        + ANSI_RESET);
            } else {
                console.info(ANSI_RED + "Address Does Not Match!" + ANSI_RESET);
                console.info(ANSI_RED + "Player: " + playername + ANSI_RESET);
                console.info(ANSI_RED + "Address Used: " + address + ANSI_RESET);
                console.info(ANSI_RED + "Address In Logger: " + realaddress
                        + ANSI_RESET);
                console.info(ANSI_RED + "Consider Looking Into This Issue!"
                        + ANSI_RESET);
 
                event.disallow(PlayerLoginEvent.Result.KICK_OTHER, kickmessage);
            }
        }
    }
 
}

IPCListener:

Show Spoiler Hide Spoiler

Code:java

package dev.flareline.ipcheck;
 
import org.bukkit.Bukkit;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerLoginEvent;
 
public class IPCListener implements Listener {
 
    IPCMain ipcheckinstance = new IPCMain();
 
    @EventHandler(priority = EventPriority.HIGHEST)
    public void onLogin(PlayerLoginEvent event) {
 
        Player player = event.getPlayer();
        String playername = player.getName();
        String address = event.getAddress().toString();
 
        Bukkit.getLogger().info(
                IPCMain.ANSI_CYAN + playername + " Has Joined The Server!"
                        + IPCMain.ANSI_RESET);
        Bukkit.getLogger().info(
                IPCMain.ANSI_CYAN + "Checking Information On: " + playername
                        + IPCMain.ANSI_RESET);
 
        IPCMain.checker(playername, address, event);
 
    }
}

config.yml

Show Spoiler Hide Spoiler

Code:

# IPC Configuration File
# Use this file to define exceptions for players.
# Define them using a YML String format.
# Every Different IP Address Needs A Separate Entry.
# Address Must Also Start With A Trailing FowardSlash.
# eg.
# exceptions:
#  addresses:
#    /999.999.999.999: FlareLine
#    /123.456.789.012: ExamplePlayer
exceptions:
  addresses:

logger.yml

Show Spoiler Hide Spoiler

# IPC Logger File
# This File Will Contain All Of The Logged Players Who
# Have Joined The Server Since IPC Was Installed And
# Any Relevant Information Collected About Them. This
# Information To Be Collected Will Soon Be Configurable
# In A Future Update.

It works, fine for the first time, however when I login a second time, it disregards the changes in logger.yml and states that the player was excepted by the config.yml, even though there are no entries there. Cheers,
FlareLine.

 

For one, you're using a separate instance of the main class in your listener:

Code:java

public class IPCListener implements Listener {
 
    IPCMain ipcheckinstance = new IPCMain();
 
}
 
// Should be...
 
public class IPCListener implements Listener {
 
    private final IPCMain ipcheckinstance;
 
    public IPCListener (IPCMain ipcheckinstance) {
        this.ipcheckinstance = ipcheckinstance;
    }
 
}
 
// And would be called by the main class as...
 
IPCListener listener = new IPCListener(this);

 

1Rogue This wouldn't cause such an error though.

 

It could if you're referencing different instances of the file. Have you fixed your listener yet?

 

1Rogue Yes, however it is still giving me the same error. Mind you it was working before with the opposite instance, and this instantiated class:

Code:java

IPCMain ipcheckinstance = new IPCMain();

is never used. Anywhere
Give me a moment and I'll post the specific error.

 

You're getting the same error from before, but it was working before....?

Do you have a specific error?

Also as a recommendation, I would say to use a BufferedWriter for the log file, it will take less CPU and you just need to make sure you would called writer.flush(); in onDisable();

 

What I meant was that it was working before I implemented the config.yml

On first join:

Show Spoiler Hide Spoiler

Code:

2013-10-07 12:55:10 [INFO] [36mFlareLine Has Joined The Server![0m
2013-10-07 12:55:10 [INFO] [36mChecking Information On: FlareLine[0m
2013-10-07 12:55:10 [INFO] [36mFlareLine's IP Address is: /127.0.0.1[0m
2013-10-07 12:55:10 [INFO] [32mFlareLine's Address already exists: /127.0.0.1[0m
2013-10-07 12:55:10 [INFO] [36mTesting Validity Of: FlareLine[0m
2013-10-07 12:55:10 [INFO] [32mAddress Matches Exception In config.yml[0m
2013-10-07 12:55:10 [INFO] [36mAllowing FlareLine To Join![0m
2013-10-07 12:55:10 [INFO] FlareLine[/127.0.0.1:52278] logged in with entity id 232 at ([Spawnworld] -4.889269125630935, 68.0, -49.86040489102292)

config.yml

Show Spoiler Hide Spoiler

Code:

# IPC Configuration File
# Use this file to define exceptions for players.
# Define them using a YML String format.
# Every Different IP Address Needs A Separate Entry.
# Address Must Also Start With A Trailing FowardSlash.
# eg.
# exceptions:
#  addresses:
#    /999.999.999.999: FlareLine
#    /123.456.789.012: ExamplePlayer
exceptions: {}

logger.yml

Show Spoiler Hide Spoiler

Code:

FlareLine:
  join:
    address: /127.0.0.1

After changing the IP in logger.yml to /127.0.0.12 , The console gives me this

Code:

2013-10-07 12:59:10 [INFO] [36mFlareLine Has Joined The Server![0m
2013-10-07 12:59:10 [INFO] [36mChecking Information On: FlareLine[0m
2013-10-07 12:59:10 [INFO] [36mFlareLine's IP Address is: /127.0.0.1[0m
2013-10-07 12:59:10 [INFO] [32mFlareLine's Address already exists: /127.0.0.13[0m  <--- Note the difference
2013-10-07 12:59:10 [INFO] [36mTesting Validity Of: FlareLine[0m
2013-10-07 12:59:10 [INFO] [32mAddress Matches Exception In config.yml[0m
2013-10-07 12:59:10 [INFO] [36mAllowing FlareLine To Join![0m
2013-10-07 12:59:10 [INFO] FlareLine[/127.0.0.1:52311] logged in with entity id 229 at ([Spawnworld] -4.889269125630935, 68.0, -49.86040489102292)

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

You have:

Code:java

if (allowed = true) {
    console.info(ANSI_GREEN + "Address Matches Exception In config.yml"
                        + ANSI_RESET);
    console.info(ANSI_CYAN + "Allowing " + playername + " To Join!"
                        + ANSI_RESET);
}

Which will always return true. Checks for values need to be == or just the boolean itself:

Code:java

if (allowed) {
    // ...
}
if (allowed == true) {
    // ...
}

That would be why this is triggering each time.

Edit: This is under your denyLogin method.

 

1Rogue Ah, so a single equals sign will compare 'allowed' to a boolean variable, and return true if allowed is a boolean?

 

No, a single equals sign sets a value, and a double equals sign compares the values and returns a boolean.

Code:java

boolean test1 = true == true; // Returns true
boolean test2 = true == false; // Returns false
boolean test3 = true != false; // Returns true
boolaen test4 = true != true; // Returns false

 

1Rogue Oh derp. Thankyou for your help anyway kind sir - it's working correctly now