Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NEO.CLI] custom plugins won't shown by command plugins #3268

Closed
lingyido opened this issue May 23, 2024 · 9 comments · Fixed by #3269
Closed

[NEO.CLI] custom plugins won't shown by command plugins #3268

lingyido opened this issue May 23, 2024 · 9 comments · Fixed by #3269
Assignees
Labels
Bug Used to tag confirmed bugs Easy-to-do Help Wanted We suggest a careful read at https://docs.neo.org/. However, fell free to further discuss the topic.

Comments

@lingyido
Copy link
Contributor

Describe the bug
After upgrading to neo v3.7.4, the plugins command in neo-cli won't show custom installed plugins.

To Reproduce
Steps to reproduce the behavior:

  1. Open the project, install any custom plugin
  2. Type 'plugins'

Expected behavior
show custom plugin's information

Screenshots
Screenshot 2024-05-23 at 13 38 39

Platform:

  • Version [e.g. neo-cli 3.7.4]

(Optional) Additional context
This should be related to #3092.

I don't want to hurt someone here. But for NEO's stability, I personally suggest that you should pay more attention to https://github.com/cschuchardt88's PR. IIRC, neo-project/neo-modules#897 also find some bugs introduced by cschuchardt88. He has introduced so many big PRs that changes a lot, especially there are many opening now. Let's move carefully.

@cschuchardt88
Copy link
Member

That's a good point to bring. In #3092 the already existing code that was updated. Did display other plugins, only the plugins from github. We are going to be reworking this anyways. Because after this release plugins will not be at that URL anymore or that repo.

@cschuchardt88 cschuchardt88 added Bug Used to tag confirmed bugs Help Wanted We suggest a careful read at https://docs.neo.org/. However, fell free to further discuss the topic. Easy-to-do labels May 23, 2024
@shargon shargon self-assigned this May 23, 2024
@shargon
Copy link
Member

shargon commented May 23, 2024

neo> plugins
[Not Installed]  ApplicationLogs
[Installed]      DBFTPlugin       @3.7.4  Consensus plugin with dBFT algorithm.
[Not Installed]  LevelDBStore
[Not Installed]  MPTTrie
[Not Installed]  OracleService
[Not Installed]  RocksDBStore
[Not Installed]  RpcClient
[Not Installed]  RpcServer
[Not Installed]  SQLiteWallet
[Not Installed]  StatesDumper
[Not Installed]  StateService
[Not Installed]  StorageDumper
[Not Installed]  TokensTracker

it works for me, could you give us more information?

@shargon
Copy link
Member

shargon commented May 23, 2024

I have an idea, I will fix it

@cschuchardt88
Copy link
Member

cschuchardt88 commented May 23, 2024

Its for custom plugins, I'm guessing he has his only plugin that he made. Below should be the other way around.

var plugins = GetPluginListAsync().GetAwaiter().GetResult();
if (plugins == null) return;
plugins
.Order()
.ForEach(f =>
{
var installedPlugin = Plugin.Plugins.SingleOrDefault(pp => string.Equals(pp.Name, f, StringComparison.CurrentCultureIgnoreCase));

@shargon
Copy link
Member

shargon commented May 23, 2024

The problem is when the plugin list can't be retrieved, because it's wrong configured or a network error., fixed in #3269

@lingyido
Copy link
Contributor Author

lingyido commented Jun 6, 2024

The problem is when the plugin list can't be retrieved, because it's wrong configured or a network error., fixed in #3269

I think that @cschuchardt88 fully understood what problem I faced. I'm not sure if this PR fixed that problem. Apart from official plugins, I have my own plugins.

Anyway, thanks for fixing this. I'll reopen it if the problem still exist in the next version.

@vncoelho
Copy link
Member

vncoelho commented Jun 6, 2024

I see, @lingyido .

let's re-open in this case.

@vncoelho vncoelho reopened this Jun 6, 2024
@lingyido
Copy link
Contributor Author

lingyido commented Jun 6, 2024

@cschuchardt88 If you knew and tested that this issue has been fixed, please tell us and close this. I'm not sure that if @shargon get my point.

@vncoelho
Copy link
Member

vncoelho commented Jun 6, 2024

I will let you reopen.

@vncoelho vncoelho closed this as completed Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Used to tag confirmed bugs Easy-to-do Help Wanted We suggest a careful read at https://docs.neo.org/. However, fell free to further discuss the topic.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants