aquiladev/remix-mythx-plugin

Submit ASTs instead of sources

muellerberndt opened this issue · 7 comments

Description

MythX works more reliably based on ASTs and use of source code will be deprecated shortly.

Submit ASTs instead of sources in the sources dict of the request data. For an example how to construct a request with ASTs see:

https://github.com/b-mueller/sabre/blob/9ceb49218645cbcf700f4eea708a958ad306e16b/lib/client.js#L7

I'll take a look

we pushed a PR ethereum/remix#1236, in remix-alpha probably tomorrow (Tuesday)

I think the fix should already be in remix-alpha...

@yann300 I checked today, still ast is not there. I checked both environments, remix and remix-alpha

and now? (we just updated a few hours ago)

yes remix-alpha has ast

when AST is present, plugin uses it, otherwise it uses source