Skip to content
This repository has been archived by the owner on Sep 5, 2020. It is now read-only.

Enable HTTP RPC sockets #871

Merged
merged 21 commits into from
Aug 3, 2016
Merged

Enable HTTP RPC sockets #871

merged 21 commits into from
Aug 3, 2016

Conversation

hiddentao
Copy link
Contributor

@hiddentao hiddentao commented Jun 22, 2016

This incorporates and replaces #420.

  • It replaces the --ipcpath CLI parameter with --rpc, allowing for an HTTP RPC hostname (e.g. http://localhost:8545) to be provided. If omitted then IPC is used as the default.
  • README updated with instructions
  • Introduce usage of --node- prefix for CLI parameters that are to be passed onto geth/eth.
  • Warns user that the HTTP option is less secure than IPC
  • Fix a small bug in solidity compilation error reporting
  • Incorporates Fixes the node start IPC connection #841 as I needed that to test this properly.

@hiddentao
Copy link
Contributor Author

Ready for review.

@hiddentao
Copy link
Contributor Author

Todo:

  • Add warning pop-up

@hiddentao
Copy link
Contributor Author

Dialog msg added.

screen shot 2016-06-28 at 12 25 58

@frozeman
Copy link
Contributor

haha :)
Some text improvements:

"....This is less secure than using a local IPC socket - your account passwords will be send in plain text!

Only do this when you secured your HTTP connection, or you know what you are doing."

# Conflicts:
#	modules/ethereumNode.js
#	modules/settings.js
#	modules/sockets.js
@hiddentao
Copy link
Contributor Author

@luclu Fixed directly in develop branch.

@hiddentao hiddentao modified the milestones: 0.8.1, 0.8 Jul 11, 2016
@@ -136,8 +137,13 @@ for (let optIdx in argv) {
}

// some options are shared
<<<<<<< HEAD
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be removed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@frozeman
Copy link
Contributor

frozeman commented Jul 11, 2016

There seems to be an issue, if i put web3.eth.accounts into the interface it doesn't return, and mist wont quit.
Then sometimes when quitting, it will show all the RPC messages, because it can't parse it properly.

I will look more deeply what could be the problem, but if you have any idea @hiddentao please let me know.

EDIT: This seems to happen not all the time, but sometimes that sync requests don't get returned.

@hiddentao
Copy link
Contributor Author

@frozeman Yep, haven't been able to reproduce it yet. If I see it happen I'll investigate further.

# Conflicts:
#	main.js
#	modules/ipc/getIpcPath.js
#	modules/settings.js
@luclu luclu modified the milestones: 0.8.1, 1.0, 0.8.2 Jul 17, 2016
# Conflicts:
#	modules/ethereumNode.js
#	modules/settings.js
@hiddentao
Copy link
Contributor Author

This is ready to be merged for 0.8.2.

@hiddentao hiddentao merged commit 2f612cd into ethereum:develop Aug 3, 2016
@lock
Copy link

lock bot commented Mar 31, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked and limited conversation to collaborators Mar 31, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants