Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

disable http-listen by default #29

Closed
boucadair opened this issue Mar 16, 2023 · 1 comment
Closed

disable http-listen by default #29

boucadair opened this issue Mar 16, 2023 · 1 comment
Labels
duplicate This issue or pull request already exists WGLC comment received during WGLC

Comments

@boucadair
Copy link
Contributor

boucadair commented Mar 16, 2023 •
edited

draft-ietf-alto-oam-yang/yang/ietf-alto.yang

Line 495 in 17687c2

case http {

I wonder whether transport-type can defined with a default set to the secure https mode. The http-listen is not supposed to be used for deployment. Having guards to avoid exposing that would be needed.
@boucadair boucadair added the WGLC comment received during WGLC label Mar 16, 2023
@boucadair boucadair linked a pull request May 12, 2023 that will close this issue
https as default #63
Closed
@boucadair boucadair added the duplicate This issue or pull request already exists label May 12, 2023
@boucadair
Copy link
Contributor Author

#27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate This issue or pull request already exists WGLC comment received during WGLC
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

https as default ietf-wg-alto/draft-ietf-alto-oam-yang
1 participant
@boucadair