Skip to content

Escape app name - #82

Merged
Samze merged 1 commit into
masterfrom
escape_app_name
Nov 12, 2024
Merged

Escape app name#82
Samze merged 1 commit into
masterfrom
escape_app_name

Conversation

@Samze

@Samze Samze commented Nov 12, 2024

Copy link
Copy Markdown
Contributor

Current behaviour:

cf change-stack static#blue cflinuxfs3
Attempting to change stack to cflinuxfs3 for perf-space-5403ca46-a079-11ef-bb1e-02420a0a0007/static#blue/...

2024/11/12 10:04:46 a problem occurred: no app found with name static#blue
$ cf curl "/v3/apps?names=static#blue"
{"pagination":{"total_results":0,"total_pages":1,"first":
{"href":"https://api.sys.tas.z3c9041a1.shepherd.lease/v3/apps?names=staticblue\u0026page=1\u0026per_page=50"}
,"last":{"href":"https://api.sys.tas.z3c9041a1.shepherd.lease/v3/apps?names=staticblue\u0026page=1\u0026per_page=50"},"next":null,"previous":null},"resources":[]}

With proper escaping:

cf change-stack static#blue cflinuxfs4
Attempting to change stack to cflinuxfs4 for perf-space-5403ca46-a079-11ef-bb1e-02420a0a0007/static#blue/...

Restoring prior application state: STARTED
Application static#blue was successfully changed to Stack cflinuxfs4
cf curl "/v3/apps?names=static%23blue"
{"pagination":{"total_results":1,"total_pages":1,"first":
{"href":"https://api.sys.tas.z3c9041a1.shepherd.lease/v3/apps?names=static%23blue\u0026page=1\u0026per_page=50"}
,"last":{"href":"https://api.sys.tas.z3c9041a1.shepherd.lease/v3/apps?names=static%23blue\u0026page=1\u0026per_page=50"},"next":null,"previous":null},"resources":[{"guid":"903ee4b0-138d-48d7-9a17-081272812342","created_at":"2024-11-12T16:42:30Z","updated_at":"2024-11-12T16:54:20Z","name":"static#blue","state":"STARTED","lifecycle":{"type":"buildpack","data":{"buildpacks":["staticfile_buildpack"],"stack":"cflinuxfs3"}},"relationships":{"space":{"data":{"guid":"5403ca41-a079-11ef-.....

Looked at the rest of the CF client in stack auditor and can find no other need for escaping. All other query params are either integers through pages or guids.

@Samze
Samze marked this pull request as ready for review November 12, 2024 17:07

@gururajsh gururajsh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Samze
Samze merged commit 89f5fb3 into master Nov 12, 2024
@joaopapereira
joaopapereira deleted the escape_app_name branch November 12, 2024 17:11
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.

2 participants