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

Parse step input url correctly #137

Closed
pytlesk4 opened this issue Mar 7, 2018 · 1 comment
Closed

Parse step input url correctly #137

pytlesk4 opened this issue Mar 7, 2018 · 1 comment
Assignees
@pytlesk4
Labels
t/bug
Milestone
Prism v3 Release

Comments

@pytlesk4
Copy link
Contributor

pytlesk4 commented Mar 7, 2018

If the step input url is http://{$$.env.todosHost}/todos, it won't be parsed correctly. Solution, don't parse the url until the variables are replaced.
@pytlesk4 pytlesk4 added the t/bug label Mar 7, 2018
@pytlesk4 pytlesk4 added this to the v3 Release milestone Mar 7, 2018
@pytlesk4 pytlesk4 self-assigned this Mar 7, 2018
@pytlesk4 pytlesk4 added the done label Mar 7, 2018
@pytlesk4
Copy link
Contributor Author

pytlesk4 commented Mar 7, 2018

image

Fixed, releasing soon.

@pytlesk4 pytlesk4 closed this as completed Mar 8, 2018
@marbemac marbemac removed the done label Mar 8, 2018
pytlesk4 added a commit that referenced this issue Mar 8, 2018
@pytlesk4
- fixes #137
2810bb8 
- fixes #136
@philsturgeon philsturgeon modified the milestones: v3 Release, Prism v3 Release Apr 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pytlesk4 pytlesk4

Labels
t/bug
Projects
None yet
Milestone
Prism v3 Release
Development

No branches or pull requests
3 participants
@philsturgeon @marbemac @pytlesk4