rustls / rustls-platform-verifier

A certificate verification library for rustls that uses the operating system's verifier
Apache License 2.0
57 stars 18 forks source link

ci: use default Android emulator target #74

Closed cpu closed 4 months ago

cpu commented 4 months ago

Previously the Android emulator based CI job was using the larger googles_api target instead of the slimmer default. This was done to ensure network access worked as intended for the real-world test suite with a TODO to revisit after an upstream bug was fixed:

https://github.com/rustls/rustls-platform-verifier/blob/d68c2edd4c5fb2aa4cde57f472de059e6966f64c/.github/workflows/ci.yml#L147-L153

In the meantime the upstream bug has been closed and we've upgraded the emulator action version. Let's try the default target again and hopefully get a small CI speedup. It appears to run without error in my testing.

cpu commented 4 months ago

Thanks for the review :-)