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

Bug/feat: shielded-sync still requires --node or is this a feature? #2867

Closed
zenodeapp opened this issue Mar 9, 2024 · 8 comments
Closed
Labels
bug Something isn't working

Comments

@zenodeapp
Copy link
Contributor

zenodeapp commented Mar 9, 2024

Is it normal for shielded-sync to require the --node field. This is probably still a leftover issue from 0.31.6?

Or are shielded sync's going to be linked to RPCs? (a potential solution for having to keep syncing like this and allowing shielded accounts to be 'accessible' outside of the CLI?).

Or like perhaps it's an idea to create an entire class of its own. Not RPCs, but having nodes that are in charge of keeping viewing keys up-to-speed with the blockchain.

@zenodeapp zenodeapp added the bug Something isn't working label Mar 9, 2024
@quangtuyen88
Copy link

It should work without --node also this already discussed in this ticket #2705

@zenodeapp
Copy link
Contributor Author

It should work without --node also this already discussed in this ticket #2705

Yeah, except for the fact that it doesn't.

@quangtuyen88
Copy link

What do you mean? I'm running command without --node

@zenodeapp
Copy link
Contributor Author

What do you mean? I'm running command without --node

Is your node on the default port? It might be due to this that it works for you but doesn't for me.

@quangtuyen88
Copy link

Yes default port and my local server not public RPC. if you change port you need to point namadac to your special port for every command.

@zenodeapp
Copy link
Contributor Author

zenodeapp commented Mar 10, 2024

Yes default port and my local server not public RPC. if you change port you need to point namadac to your special port for every command.

This is not the case for a while. Can do all sorts of transactions without --node. So found it interesting that shielded-sync is the only command for me needing to add it.

Think back then they made it parse the port from the config in order to use the correct port. I might be mistaken to what they did exactly, but I'm on a custom port, don't have a variable set for this and don't need to use the option/arg except for shielded sync. Thus why I wonder and opened this issue. Plus it also gave me an idea hehe.

@cwgoes
Copy link
Contributor

cwgoes commented May 24, 2024

@zenodeapp Is this still an issue?

@zenodeapp
Copy link
Contributor Author

@zenodeapp Is this still an issue?

Oh wow, I'm only seeing this just now. Apologies!
This is no longer an issue, I just checked this out on Campfire.

I'll close this, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants