rap not adding text comments as expected

[bannsec]

I'm attempting to use rap protocol to add comments to a remote r2 server. The setup is:

• r2 ./binary
• =&:12345

Then in a second window (in python):

r2 = r2pipe.open("rap://127.0.0.1:12345")
r2.cmd('T test')

Oddly enough, when I do a r2.cmd('T') in the python window, it shows the text. But when i just do T in the r2 window, it does not show the text.

I'd like to be able to send text/other remote commands to the r2 server that update that r2 instance.

[radare]

See this python implementation which ive been waiting for a month to get the PR done.. https://github.com/jroimartin/pyrebox/commits/dev-r2-rap <https://github.com/jroimartin/pyrebox/commits/dev-r2-rap> /cc @jroimartin

…

On 18 Jun 2018, at 03:53, bannsec ***@***.***> wrote: I'm attempting to use rap protocol to add comments to a remote r2 server. The setup is: r2 ./binary =&:12345 Then in a second window (in python): r2 = r2pipe.open("rap://127.0.0.1:12345") r2.cmd('T test') Oddly enough, when I do a r2.cmd('T') in the python window, it shows the text. But when i just do T in the r2 window, it does not show the text. I'd like to be able to send text/other remote commands to the r2 server that update that r2 instance. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#10378>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA3-lig68DlzKHGngqDFn3Y7UUQ1-IhKks5t9wg3gaJpZM4UrEHr>.

[radare]

nvm that pr is unrelated.

the problem is that if you do r2pipe.open("rap://...") you are creating a new instance of r2 that talks to the remote instance instead of making a direct connection to the remote rap server in python. if you see ps you'll see 2 instances.

therefor.. when you do "T..." that runs in the local instance under r2pipe, instead of the remote. you can hack it by using the =!= or !=! commands to make r2 forward all commands to the remote io.. but ideally it will be better to make r2pipe python api handle rap:// natively instead.

or just use the http server which is more tested

[bannsec]

Yeah, i switched to using the http server for now, so this isn't a huge deal for me. Would still probably be a good thing to implement at some point.

[radare]

Please open an issue in radare2-pipe repo and close this one

…

On 18 Jun 2018, at 17:16, bannsec ***@***.***> wrote: Yeah, i switched to using the http server for now, so this isn't a huge deal for me. Would still probably be a good thing to implement at some point. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.