cloudflare-go: Exit gracefully when role is unimplemented

xvzcf / tls-interop-runner

Interoperability testing of TLS implementations.

Other

10 stars 13 forks source link

cloudflare-go: Exit gracefully when role is unimplemented #39

Closed cjpatton closed 3 years ago

cjpatton commented 3 years ago

Resolves an issue discussed in #36.

The cloudflare-go server currently does not implement the "dc" test case. The expected behavior is for the endpoint to exit with code 64. This PR refactors cloudflare-go to allow for this behavior.

cjpatton commented 3 years ago

cc/ @claucece, in case you are planning to make conflicting changes to this :)

xvzcf commented 3 years ago

@chris-wood to continue the discussion on the question posed here, we could just have the endpoints return 64 for any inputs that they don't recognize/that are invalid. That would save them the trouble of keeping track of what is and isn't valid for the runner (for example, we could possibly have a role in the future called fuzzer).