technically sending malformed version message

Are you sure that the field is required? The BIP implies that the field is optional:

If missing or true, no change in protocol behaviour occurs.

@MarcoFalke Yes - you're correct. I didn't read the BIP closely enough.

Still, I think mininode should specify the fRelay field. Even better: we should have a bloom filter test in qa which explicitly tests the field.

It makes sense for the test framework to support this.
utACK d7bf55c

Even better: we should have a bloom filter test in qa which explicitly tests the field.

Sure.

Please also make sure to deserialize nRelay and mention it in the repr.

Agree with @MarcoFalke, you should extend the change to the deserialization method.

@jnewbery Are you still working on this?

@MarcoFalke : deserialization and repr methods updated.

Thanks, utACK e5d682f