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

sparko help fix #2

Closed
wants to merge 5 commits into from
Closed

sparko help fix #2

wants to merge 5 commits into from

Conversation

evd0kim
Copy link
Collaborator

@evd0kim evd0kim commented Dec 4, 2019

No description provided.

t/ru.go Outdated
@@ -341,7 +341,7 @@ ssh -p{{.SSHPort}} {{.SSHUser}}@{{.IP}}</pre>{{end}}
IP: <code>{{.IP}}</code>
{{if .UserPort }}App порт: <code>{{.UserPort}}</code>
{{end}}{{if .SSHPort}}SSH: <code>ssh -p{{.SSHPort}} {{.SSHUser}}@{{.IP}}</code>
{{end}}{{with .Sparko}}<a href="{{.}}">Sparko</a>: <code>curl -X POST {{.}}/rpc -d '{"method": "getinfo"}' -H 'X-Access: grabyourkeyinside'</code>
{{end}}{{with .Sparko}}<a href="{{.}}">Sparko</a>: <code>curl -k -u user:password -X POST {{.}}/rpc -d '{"method": "getinfo"}' -H 'X-Access: grabyourkeyinside'</code>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sparko requires either the key or the username/password. Supplying both isn't necessary.
Also I want to discourage people from using username/password in API calls, reserve that for the wallet interface.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sparko requires either the key or the username/password. Supplying both isn't necessary.

I will check it again but id didn't work without authentication.

Copy link
Collaborator Author

@evd0kim evd0kim Dec 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirm, without -u id doesn't work and shows

< HTTP/1.1 401 Unauthorized

With -u it works without masterkey.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you sure you're using the correct key?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now I can say, I tried them all.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, if i try to request Sparko locally 0.0.0.0 it also shows 401 error.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fiatjaf ok. I removed -u and left only -k. Maybe I'm wrong but until Blazh will resolve issues with certificates, this option makes sense.

@evd0kim evd0kim closed this Dec 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants