keylime-agent/src: update API version to 2.1 to consistent with https//github.com/keylime/keylime/blob/master/docs/rest_apis.rst

[hustliyilin]

Fixes #589

Signed-off-by: YiLin.Li [email protected]

[aplanas]

In the 2.1 changelog I can see that they removed the vmask parameter, but I still can see some references in the quotes_handler.rs test code (and some comments).

The other change seems to affect the verifier, so I am not sure that has an impact here.

@ansasaki do you know if beside the quotes_handlers change, more changes are required?

[hustliyilin]

Thanks @aplanas. Currently, the rust-keylime only supports RESTful API v2.0. When using the https://github.com/keylime/keylime/blob/master/docs/rest_apis.rst, it will be an error like #588 .

So we need to update the API version in the rust-keylime agent or add corresponding instructions to the corresponding REST API documentation. I'm not sure which way is better.

[codecov]

Codecov Report

Merging #588 (c141442) into master (1f32de1) will decrease coverage by 0.09%.
The diff coverage is 50.00%.

Additional details and impacted files

Flag	Coverage Δ
e2e-testsuite	58.90% <50.00%> (-0.10%)	⬇️
upstream-unit-tests	58.56% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
keylime-agent/src/common.rs	70.62% <0.00%> (ø)
keylime-agent/src/errors_handler.rs	53.96% <ø> (ø)
keylime-agent/src/quotes_handler.rs	51.08% <100.00%> (ø)

... and 1 file with indirect coverage changes

[THS-on]

@aplanas yeah the only change for the agent should be that we no longer send the vmask parameter.

[hustliyilin]

@aplanas yeah the only change for the agent should be that we no longer send the vmask parameter.

Thanks @THS-on . @aplanas Could you review this PR again?

[ansasaki]

LGTM!