Some deprecated fields are required

[fazo96]

chlu-protocol/protobuf/drspec/popr.proto

Line 56 in 7183308

required string vendor_encryption_key_location = 15;

[kulpreet]

This is the problem with the required fields in proto2. They simply suck.

I think we should upgrade to proto3, get rid of required/optional.

For the moment, should we put in an empty string as you were doing to bypass that proton issue?

[fazo96]

yeah either that or we make those optional.. what do you think

[fazo96]

Or maybe we should just rip the bandaid off and try to upgrade to proto3