Skip to content

IOutboundParameterTransformer: add support for QueryString parameters #27646

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

Closed
alienwareone opened this issue Nov 9, 2020 · 4 comments
Closed
Labels
affected-very-few This issue impacts very few customers area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-routing severity-major This label is used by an internal tool
Milestone

Comments

@Pilchie Pilchie added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Nov 9, 2020
@pranavkm
Copy link
Contributor

@alienwareone it looks like you were able to configure outbound transformation to use query strings. Was there a specific issue that I'm not seeing here?

@pranavkm pranavkm added the Needs: Author Feedback The author of this issue needs to respond in order for us to continue investigating this issue. label Nov 10, 2020
@alienwareone
Copy link
Author

alienwareone commented Nov 10, 2020

@pranavkm No. Unfortunately it will get encoded:
/Germany/Smartphone%3Fq%3Ddual%20sim
Should be:
/Germany/Smartphone?q=dual%20sim

This is actually the expected behavior. See here:
https://github.com/alienwareone/ComplexOutboundParameterTransformerSample/blob/aece2391ca8094d3467466a70c776403efda8f4f/WebApplication2/Controllers/HomeController.cs#L17-L23

@ghost ghost added Needs: Attention 👋 This issue needs the attention of a contributor, typically because the OP has provided an update. and removed Needs: Author Feedback The author of this issue needs to respond in order for us to continue investigating this issue. labels Nov 10, 2020
@mkArtakMSFT mkArtakMSFT added enhancement This issue represents an ask for new feature or an enhancement to an existing one and removed Needs: Attention 👋 This issue needs the attention of a contributor, typically because the OP has provided an update. labels Nov 13, 2020
@mkArtakMSFT mkArtakMSFT added this to the Backlog milestone Nov 13, 2020
@ghost
Copy link

ghost commented Nov 13, 2020

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@javiercn javiercn added affected-very-few This issue impacts very few customers severity-major This label is used by an internal tool labels Mar 19, 2021 — with ASP.NET Core Issue Ranking
@mkArtakMSFT
Copy link
Contributor

Hi. Thanks for contacting us.
We're closing this issue as there was not much community interest in this ask for quite a while now.
You can learn more about our triage process and how we handle issues by reading our Triage Process writeup.

@mkArtakMSFT mkArtakMSFT closed this as not planned Won't fix, can't repro, duplicate, stale Oct 12, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Nov 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affected-very-few This issue impacts very few customers area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-routing severity-major This label is used by an internal tool
Projects
None yet
Development

No branches or pull requests

5 participants