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

Try It does not support * (asterisk) as a path parameter #2258

Closed
karatkep opened this issue Jul 25, 2023 · 8 comments
Closed

Try It does not support * (asterisk) as a path parameter #2258

karatkep opened this issue Jul 25, 2023 · 8 comments

Comments

@karatkep
Copy link

Bug description

Try It - Path parameter is omitted when * (asterisk) provided for parameter value.

Reproduction steps

  1. View an API which contains at least 1 path parameter - e.g. '/s/{id}/{subId}'
  2. Click on the 'Try it' button
  3. Enter * (asterisk) as a value for path parameter - e.g. '/s/*/12'
  4. Path parameter with * (asterisk) value is omitted in generated URL: '/s/12'. Please see screenshoot below:
    image

Expected behavior

Path parameter with * (asterisk) value is present in generated URL: '/s/*/12'.

Is your portal managed or self-hosted?

Managed

Release tag or commit SHA (if using self-hosted version)

n/a

API Management service name

contoso-api

Environment

  • Operating system: macOS
  • Browser: Google Chrome
  • Version: 114.0.5735.248 (Official Build) (x86_64)

Additional context

Add any other context about the problem here, including screenshots.

@ghost
Copy link

ghost commented Jul 25, 2023

@karatkep, thank you for opening this issue. We will triage it within the next few business days.

@malincrist
Copy link
Member

Hello @karatkep , thanks for reaching out. The issue you described was fixed (as part of this PR: #2137 ).
However, please note that it hasn't been deployed for managed portals yet (ETA is end of August).

@karatkep
Copy link
Author

karatkep commented Sep 5, 2023

Hello @malincrist, is there any update on deployment the fix to support * ?

@malincrist
Copy link
Member

Hello @karatkep , unfortunately, the release was postponed. New ETA is mid-September. Apologies for the delay and any inconveniences created.

@karatkep
Copy link
Author

Hello @malincrist , is there any luck with deployment?

@malincrist
Copy link
Member

Hwllo @karatkep , the deployment is in progress, please give it a few days to reach all the regions. Apologies for the delay.

@karatkep
Copy link
Author

Hello @malincrist, unfortunately, the reported issue is still reproducible. Can we reopen this bug?

@malincrist
Copy link
Member

Hello @karatkep , unfortunately, the deployment was cancelled due to found regressions. The new ETA for the release is 1 week. Apologies for the delay and the inconveniences caused by this.

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

No branches or pull requests

2 participants