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.

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

[Feature] Rename flag to app-protocol in map-route #2231

Merged

Conversation

jdgonzaleza
Copy link
Contributor

When running the following command cf map-route my-app example.com --hostname myhost --app-protocol http2 the following output should be generated:

$ cf map-route my-app example.com --hostname myhost --app-protocol http2

Mapping route route my-app.example.com for org my-org / space my-space as admin...
OK

Currently the cli already has a flag named destination-protocol
and in order to have consistency we need to rename it to app-protocol

Co-authored-by: Hector Calderon <hecalderon@vmware.com>
@jdgonzaleza jdgonzaleza merged commit c80d8c1 into master Nov 12, 2021
jdgonzaleza added a commit that referenced this pull request Nov 16, 2021
Currently the cli already has a flag named destination-protocol
and in order to have consistency we need to rename it to app-protocol

Co-authored-by: Hector Calderon <hecalderon@vmware.com>
hjcalderon10 pushed a commit that referenced this pull request Feb 7, 2022
Currently the cli already has a flag named destination-protocol
and in order to have consistency we need to rename it to app-protocol

Co-authored-by: Hector Calderon <hecalderon@vmware.com>
@moleske moleske deleted the feature/rename-destination-protocol-cf-map-route branch August 24, 2023 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants