Fix https://github.com/usmannasir/cyberpanel/issues/842

[Lvl4Sword]

As said in #842 (comment) , there's no reason not to have this as a list.

[Lvl4Sword]

Noticed the ip file is read within listWebsitesJson, but is never closed.
with open() ensures the file is closed without needing to explicitly call .close()

[Lvl4Sword]

I'm unsure as to why checker is here, let alone why the ip file is even being read, since neither seem to really be used for anything. since it doesn't seem to be used. The ip is used, because that needs provided to the user.. lol

[xmok]

Though not directly related to your PR, Issue #1145 I've opened deals with fixing JSON in the cloudApi which does relate to your previously opened Issue #842. Would be nice to get some feedback from the team on this as I've observed fixing the JSON handling also fixed some other minor errors thrown in the dashboard.

[usmannasir]

I had to change the base for this merge, thanks.

[Lvl4Sword]

Would you also accept cleaning up the rest of the CLI file to catch issues like this? I'd make separate issues if needed as well.

[usmannasir]

You have any pending PR?

If now I've no problem reviewing new and merging.

thank you.

Would you also accept cleaning up the rest of the CLI file to catch issues like this? I'd make separate issues if needed as well.

[Lvl4Sword]

Not currently, but it'll be worked on this week. Glad to help! 👍

You have any pending PR?

If now I've no problem reviewing new and merging.

thank you.

Would you also accept cleaning up the rest of the CLI file to catch issues like this? I'd make separate issues if needed as well.

[usmannasir]

Here to help.