LayerXcom / confidential-computing-modules

A Module for Privacy-preserving State Transitions with Verifiability
90 stars 9 forks source link

フィーチャーverify-mr-enclave-enableにおける抜け漏れ対応 #657

Closed jkcomment closed 3 years ago

jkcomment commented 3 years ago

Issueへのリンク

やったこと

やらないこと

動作検証

参考

jkcomment commented 3 years ago

確かに例のあのリポジトリの方では下記のような形でOn/Offをしてました

Cargo.toml

[features]
default = ["backup-enable", "runtime_enabled", "verify-mr-enclave-enable"]
backup-enable = [
  "anonify-enclave/backup-enable",
  "frame-runtime/backup-enable",
]
runtime_enabled = []
verify-mr-enclave-enable = [
  "anonify-enclave/verify-mr-enclave-enable",
  "frame-runtime/verify-mr-enclave-enable",
]
jkcomment commented 3 years ago

tests/integration以下はfeaturesを設定しない設計ですかね?

tests/integrationの方は元々backup-enableも設定されていないのでそういう意図かなと思い、特に対応はしていない感じです。 もしfeaturesを設定するとしたらbackup-enableも一緒にやった方がいいですかね