r/application_gateway: clarify default value of require_sni in an http_listener

[Supermathie]

Is there any reason we can't just make the default for require_sni to be true when the listener is HTTPS?

[tombuildsstuff]

LGTM - thanks for this 👍

[tombuildsstuff]

@Supermathie

Thanks for this PR :)

Is there any reason we can't just make the default for require_sni to be true when the listener is HTTPS?

In general we either have a default value or make it required, rather than changing the value dynamically, but possibly in the future - unfortunately either way we're currently blocked on any enhancements to Application Gateway because of #1576.

In the interim we're planning on rewriting the documentation for the Application Gateway resource in the near future to bring it into line with the other resources too, so that we can make this clearer.

Thanks!

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!